Merge pull request #45 from healenium/IS-44

remove Log4j and leave only Sl4j logging #44
healenium · Mar 13, 2020 · c97d4b8 · c97d4b8
2 parents 39d0166 + f4202d2
commit c97d4b8
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 43 deletions.
diff --git a/build.gradle b/build.gradle
@@ -26,9 +26,7 @@ dependencies {
     compile group: 'com.fasterxml.jackson.core', name: 'jackson-databind', version: '2.10.0'
     compile group: 'org.apache.commons', name: 'commons-lang3', version: '3.8.1'
     compile group: 'commons-codec', name: 'commons-codec', version: '1.12'
-    compile group: 'org.slf4j', name: 'slf4j-simple', version: '1.7.25'
-    compile group: 'org.apache.logging.log4j', name: 'log4j-api', version: '2.12.1'
-    compile group: 'org.apache.logging.log4j', name: 'log4j-core', version: '2.12.1'
+    compile group: 'org.slf4j', name: 'slf4j-simple', version: '1.7.30'
     compile group: 'org.eclipse.jetty', name: 'jetty-server', version: '9.4.19.v20190610'
     compile group: 'org.jetbrains', name: 'annotations', version: '18.0.0'
     compileOnly group: 'org.projectlombok', name: 'lombok', version: '1.18.10'

diff --git a/src/main/java/com/epam/healenium/SelfHealingEngine.java b/src/main/java/com/epam/healenium/SelfHealingEngine.java
@@ -36,7 +36,7 @@
 import com.typesafe.config.Config;
 import lombok.Getter;
 import lombok.SneakyThrows;
-import lombok.extern.log4j.Log4j2;
+import lombok.extern.slf4j.Slf4j;
 import org.openqa.selenium.By;
 import org.openqa.selenium.JavascriptExecutor;
 import org.openqa.selenium.WebDriver;
@@ -55,7 +55,7 @@
 import java.util.Set;
 import java.util.stream.Collectors;
 
-@Log4j2
+@Slf4j
 public class SelfHealingEngine {
 
     /**

diff --git a/src/main/java/com/epam/healenium/data/FileSystemPathStorage.java b/src/main/java/com/epam/healenium/data/FileSystemPathStorage.java
@@ -22,7 +22,7 @@
 import com.fasterxml.jackson.databind.jsontype.TypeSerializer;
 import com.fasterxml.jackson.databind.module.SimpleModule;
 import com.typesafe.config.Config;
-import lombok.extern.log4j.Log4j2;
+import lombok.extern.slf4j.Slf4j;
 import org.apache.commons.codec.digest.DigestUtils;
 import org.openqa.selenium.By;
 
@@ -38,7 +38,7 @@
 import java.util.List;
 import java.util.Map;
 
-@Log4j2
+@Slf4j
 public class FileSystemPathStorage implements PathStorage {
 
     private static final int MAX_FILE_LENGTH = 128;

diff --git a/src/main/java/com/epam/healenium/handlers/proxy/SelfHealingProxyInvocationHandler.java b/src/main/java/com/epam/healenium/handlers/proxy/SelfHealingProxyInvocationHandler.java
@@ -14,16 +14,15 @@
 
 import com.epam.healenium.SelfHealingEngine;
 import com.epam.healenium.utils.ProxyFactory;
-import java.lang.reflect.Method;
-import java.lang.reflect.Proxy;
-import java.util.Optional;
-import lombok.extern.log4j.Log4j2;
+import lombok.extern.slf4j.Slf4j;
 import org.openqa.selenium.By;
-import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.WebDriver.TargetLocator;
 import org.openqa.selenium.WebElement;
 
-@Log4j2
+import java.lang.reflect.Method;
+import java.util.Optional;
+
+@Slf4j
 public class SelfHealingProxyInvocationHandler extends BaseHandler {
 
     public SelfHealingProxyInvocationHandler(SelfHealingEngine engine) {

diff --git a/src/main/resources/log4j2.xml b/src/main/resources/log4j2.xml
diff --git a/src/test/java/com/epam/healenium/AbsentLocatorTest.java b/src/test/java/com/epam/healenium/AbsentLocatorTest.java
@@ -13,13 +13,11 @@
 package com.epam.healenium;
 
 import org.junit.After;
-import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 import org.openqa.selenium.By;
 import org.openqa.selenium.NoSuchElementException;
 import org.openqa.selenium.WebDriver;
-import org.openqa.selenium.WebElement;
 import org.openqa.selenium.chrome.ChromeDriver;
 import org.openqa.selenium.chrome.ChromeOptions;