Upgraded versions.

Author: AutomatedOwl

README.md

@@ -1,6 +1,6 @@
 # selenium-java-pom-example
 
-This example currently supports Linux and Windows, both with Java 8 and above. Maven and Chrome browser are also required.
+This example currently supports Linux and Windows, both with Java 12 and above. Maven and Chrome browser are also required.
 
 To run on Windows or Linux, simply clone the repository and from root project diectory run:
 ```

pom.xml

@@ -5,7 +5,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>com.github.automatedowl</groupId>
     <artifactId>selenium-java-pom-example</artifactId>
-    <version>1.0.3</version>
+    <version>1.0.4</version>
     <properties>
         <aspectj.version>1.9.1</aspectj.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
@@ -21,23 +21,23 @@
                     <version>3.0.0-M3</version>
                     <configuration>
                         <suiteXmlFiles>
-                            <suiteXmlFile>default_tests.xml</suiteXmlFile>
+                            <suiteXmlFile>suites/e2e_tests.xml</suiteXmlFile>
                         </suiteXmlFiles>
                         <argLine>
                             -javaagent:"${settings.localRepository}/org/aspectj/aspectjweaver/${aspectj.version}/aspectjweaver-${aspectj.version}.jar"
                         </argLine>
-                   <forkCount>3</forkCount>
+                   <forkCount>1</forkCount>
                    <reuseForks>true</reuseForks>
                    <argLine>-Xmx1024m -XX:MaxPermSize=256m</argLine>
                    </configuration>
                 </plugin>
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-compiler-plugin</artifactId>
-                    <version>3.8.0</version>
+                    <version>3.8.1</version>
                     <configuration>
-                        <source>1.8</source>
-                        <target>1.8</target>
+                        <source>12</source>
+                        <target>12</target>
                     </configuration>
                 </plugin>
             </plugins>
@@ -47,12 +47,12 @@
         <dependency>
             <groupId>io.qameta.allure</groupId>
             <artifactId>allure-testng</artifactId>
-            <version>2.8.1</version>
+            <version>2.13.0</version>
         </dependency>
         <dependency>
             <groupId>org.testng</groupId>
             <artifactId>testng</artifactId>
-            <version>6.14.3</version>
+            <version>7.0.0</version>
         </dependency>
         <dependency>
             <groupId>javax.xml.bind</groupId>
@@ -75,17 +75,4 @@
             <version>3.6.0</version>
         </dependency>
     </dependencies>
-    <reporting>
-        <excludeDefaults>true</excludeDefaults>
-        <plugins>
-            <plugin>
-                <groupId>io.qameta.allure</groupId>
-                <artifactId>allure-maven</artifactId>
-                <configuration>
-                    <reportVersion>2.8.1</reportVersion>
-                </configuration>
-                <version>2.9</version>
-            </plugin>
-        </plugins>
-    </reporting>
 </project>

src/main/java/com/github/automatedowl/examples/drivers/DriverModule.java

@@ -2,7 +2,6 @@
 
 import com.google.inject.AbstractModule;
 import io.github.bonigarcia.wdm.WebDriverManager;
-import org.apache.tika.utils.SystemUtils;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.chrome.ChromeDriver;
 

src/main/java/com/github/automatedowl/examples/drivers/ThreadLocalDriver.java

@@ -9,8 +9,8 @@ public class ThreadLocalDriver {
     private static ThreadLocal<WebDriver> driver = new ThreadLocal<>();
 
     public ThreadLocalDriver() {
-        Injector i = Guice.createInjector(new DriverModule());
-        driver.set(i.getInstance(WebDriver.class));
+        Injector injector = Guice.createInjector(new DriverModule());
+        driver.set(injector.getInstance(WebDriver.class));
     }
 
     public WebDriver getDriver() {

src/main/java/com/github/automatedowl/examples/pages/Site88Page.java

@@ -1,4 +1,4 @@
-package com.github.automatedowl.examples;
+package com.github.automatedowl.examples.pages;
 
 import java.util.List;
 import org.openqa.selenium.By;
@@ -13,7 +13,7 @@ public class Site88Page {
 	/** Default timeout of waiting for web elements in the page. */
 	private final int DEFAULT_ELEMENT_TIMEOUT = 30;
 	private final String XPATH_CHILD_ELEMENTS_LOCATOR = ".//*";
-	private final String URL = "http://testjs.site88.net";
+	private final String URL = "https://testjs2.000webhostapp.com/";
 
 	/////////////////////////////////////////////////////////////////////////////////
 	/////////////////////////////// Constructor /////////////////////////////////////

src/main/java/com/github/automatedowl/examples/pages/Site88PageThreadLocal.java

@@ -1,19 +1,16 @@
-package com.github.automatedowl.examples.drivers;
+package com.github.automatedowl.examples.pages;
 
-import com.google.inject.Guice;
-import com.google.inject.Injector;
 import org.openqa.selenium.WebDriver;
 
-public class ThreadLocalDriver {
+public class Site88PageThreadLocal {
 
-    private static ThreadLocal<WebDriver> driver = new ThreadLocal<>();
+    private static ThreadLocal<Site88Page> site88Page = new ThreadLocal<>();
 
-    public ThreadLocalDriver() {
-        Injector i = Guice.createInjector(new DriverModule());
-        driver.set(i.getInstance(WebDriver.class));
+    public Site88PageThreadLocal(WebDriver driver) {
+        site88Page.set(new Site88Page(driver));
     }
 
-    public WebDriver getDriver() {
-        return this.driver.get();
+    public Site88Page getSite88Page() {
+        return this.site88Page.get();
     }
 }

src/test/java/com/github/automatedowl/examples/tests/WebHostTests.java

@@ -8,14 +8,6 @@ public class WebHostTests extends WebHostTestCase {
     @Test
     public void site88Test() throws InterruptedException {
 
-        System.out.println("Running site88Test");
-
-        System.out.println(getDriver().equals(null));
-
-        System.out.println(site88Page.equals(null));
-
-        System.out.println(site88Page.getSite88Page().equals(null));
-
         // Navigate to URL.
         getDriver().get(site88Page.getSite88Page().getURL());
 
@@ -25,32 +17,4 @@ public void site88Test() throws InterruptedException {
         // Wait before closing browser..
         waitBeforeClosingBrowser();
     }
-/*
-    @Test
-    public void failureTest() throws InterruptedException {
-        for (int i = 0; i < 10; i++) {
-            try {
-                Thread.sleep(1000);
-            } catch (InterruptedException e) {
-                e.printStackTrace();
-            }
-            System.out.println("TEST 2");
-        }
-        // Assert failure.
-        Assert.assertTrue(false);
-    }*/
-
-    @Test(enabled = false)
-    public void failureTwoTest() throws InterruptedException {
-        for (int i = 0; i < 10; i++) {
-            try {
-                Thread.sleep(1000);
-            } catch (InterruptedException e) {
-                e.printStackTrace();
-            }
-            System.out.println("TEST 2");
-        }
-        // Assert failure.
-        Assert.assertTrue(false);
-    }
 }

suites/e2e_tests.xml

@@ -1,9 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd">
-<suite name="Suite">
-    <test name="Test" time-out="3600000" parallel="methods" thread-count="2">
+<suite name="E2E Suite">
+    <test name="E2E Test" >
         <classes>
-            <class name="com.github.automatedowl.examples.WebHostTest"/>
+            <class name="com.github.automatedowl.examples.tests.WebHostTests"/>
         </classes>
     </test>
 </suite>