Merge pull request #54 from healenium/feature/EPMHLM-295

EPMHLM-295

Author: Alex-Reif

pom.xml

@@ -10,7 +10,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>com.epam.healenium</groupId>
     <artifactId>healenium-appium</artifactId>
-    <version>1.4.0</version>
+    <version>1.5.0</version>
     <packaging>jar</packaging>
     <name>healenium-appium</name>
     <description>healenium appium client</description>
@@ -55,7 +55,7 @@
 
 
     <properties>
-        <healenium.version>3.3.1</healenium.version>
+        <healenium.version>3.4.0</healenium.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <typesafe.version>1.3.4</typesafe.version>
         <commonslang3.version>3.11</commonslang3.version>
@@ -65,8 +65,7 @@
         <lombok.version>1.18.20</lombok.version>
         <javassist.version>3.26.0-GA</javassist.version>
         <junit.version>5.7.1</junit.version>
-        <appium.version>8.2.0</appium.version>
-        <okhttp3.version>4.9.3</okhttp3.version>
+        <appium.version>8.3.0</appium.version>
     </properties>
 
     <profiles>
@@ -191,11 +190,6 @@
             <artifactId>healenium-web</artifactId>
             <version>${healenium.version}</version>
         </dependency>
-        <dependency>
-            <groupId>com.squareup.okhttp3</groupId>
-            <artifactId>okhttp</artifactId>
-            <version>${okhttp3.version}</version>
-        </dependency>
         <dependency>
             <groupId>org.apache.maven.plugins</groupId>
             <artifactId>maven-surefire-plugin</artifactId>

src/main/java/com/epam/healenium/appium/service/MobileNodeService.java

@@ -26,16 +26,12 @@ public class MobileNodeService extends NodeService {
     private static final int ONE_ELEMENT = 1;
     private static final int THE_ONLY_ELEMENT = 0;
 
-    public MobileNodeService(WebDriver driver) {
-        super(driver);
-    }
-
     @Override
-    public List<Node> getNodePath(WebElement element) {
-        return getHierarchyElements(element);
+    public List<Node> getNodePath(WebDriver driver, WebElement element) {
+        return getHierarchyElements(driver, element);
     }
 
-    private List<Node> getHierarchyElements(WebElement element) {
+    private List<Node> getHierarchyElements(WebDriver driver, WebElement element) {
         String xmlString = driver.getPageSource();
 
         Document doc = Jsoup.parse(xmlString, "", Parser.xmlParser());

src/main/java/com/epam/healenium/appium/wrapper/DriverWrapper.java

@@ -63,7 +63,7 @@ public static <T extends AppiumDriver> T wrap(T delegate, Config config) {
         }
         SelfHealingEngine engine = new MobileSelfHealingEngine(delegate, config);
         engine.setClient(new RestClient(engine.getConfig()));
-        engine.setNodeService(new MobileNodeService(delegate));
+        engine.setNodeService(new MobileNodeService());
         engine.setHealingService(new MobileHealingService(engine.getConfig(), delegate));
         engine.getClient().setMapper(new HealeniumMapper(new MobileStackTraceReader()));
         return create(engine);
@@ -88,7 +88,7 @@ public static <T extends AppiumDriver> T create(SelfHealingEngine engine){
                     new MobileSelfHealingProxyInvocationHandler(engine)
             );
         } catch (Exception ex){
-            log.error("Failed to create wrapper!", ex);
+            log.error("Failed to create wrapper! Exception: {0}", ex);
             return origin;
         }
     }