fixing tests and creating testng XML files to run tests off of

Author: andrewmkrug

.idea/runConfigurations/Run_Mac_Tests.xml

@@ -8,7 +8,7 @@ addons:
 
 script:
 - ./mac_install_browsers.sh
-- mvn clean test -Dgroups=mac
+- mvn clean test  -Dsurefire.suiteXmlFiles=mac-only.xml
 
 cache:
   directories:

.idea/runConfigurations/Run_Nunit_SampleTest_.xml renamed to .idea/runConfigurations/Run_Mac_Tests2.xml

@@ -4,29 +4,24 @@
 This project is using the latest Selenium Bindings.
 
 ## Setup
+
 Run these commands depending on your base OS.
 
 ### On Mac OS
 
-Install Java JDK
-
-Install Homebrew `/usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"`
-
-Install Caskroom `brew tap caskroom/cask`
-
-If you dont have the Java Development Kit (JDK) run this command `brew cask install java`
-
-Install Maven `brew install maven`
-
-Run `./mac_install_browsers.sh`
+1. Install Homebrew `/usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"`
+2. Install Caskroom `brew tap caskroom/cask`
+3. If you dont have the Java Development Kit (JDK) run this command `brew cask install java`
+4. Install Maven `brew install maven`
+5. Run `./mac_install_browsers.sh`
 
 ## Running Tests
 
 The following commands are instructions on how to run the tests
 
 ### From CLI
 
-`mvn test -Dgroups=mac` on a mac
+`mvn test  -Dsurefire.suiteXmlFiles=mac-only.xml ` on a mac
 
 ### Intellij Idea
 
@@ -36,7 +31,7 @@ There is a shared run configuration that has the maven target setup and another
 
 Safari requires the following steps to enable Selenium Webdriver tests to run:
 
-1. Open Safari Prefrences
+1. Open Safari Preferences
 2. Go to the Advanced Tab
 3. Check the box at the bottom "Show Develop menu in menu bar"
 4. Click on the Develop menu 

.idea/runConfigurations/Run_Windows_Tests.xml

@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd">
+<suite name="Mac Test Suite" parallel="none">
+    <test verbose="1" name="javajenkins">
+        <groups>
+            <run>
+                <include name="mac"/>
+            </run>
+        </groups>
+        <classes>
+            <class name="LocalBrowsers.Chrome"/>
+            <class name="LocalBrowsers.Edge"/>
+            <class name="LocalBrowsers.Firefox"/>
+            <class name="LocalBrowsers.PhantomJS"/>
+            <class name="LocalBrowsers.Safari"/>
+        </classes>
+    </test> <!-- javajenkins -->
+</suite> <!-- Custom suite -->

.travis.yml

@@ -5,13 +5,12 @@
 import org.openqa.selenium.chrome.ChromeOptions;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.AfterTest;
-import org.testng.annotations.BeforeTest;
-import org.testng.annotations.Test;
+import org.testng.annotations.*;
 
 /**
  * Created by andrew on 12/3/16.
  */
+@Test(groups = {"mac", "windows"})
 public class Chrome {
     WebDriver driver;
 
@@ -22,8 +21,7 @@ public void chromeSetup(){
         capabilities.setCapability(ChromeOptions.CAPABILITY, options);
         driver = new ChromeDriver(capabilities);
     }
-
-    @Test(groups = {"firefox", "chrome", "safari"})
+    @Test
     public void test(){
         driver.get("http://lazycoder.io/about.html");
         Assert.assertEquals(driver.getTitle(), "About");

README.md

@@ -7,24 +7,21 @@
 import org.openqa.selenium.edge.EdgeOptions;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.AfterTest;
-import org.testng.annotations.BeforeTest;
-import org.testng.annotations.Test;
+import org.testng.annotations.*;
 
 /**
  * Created by andrew on 12/3/16.
  */
 public class Edge {
     WebDriver driver;
 
-    @BeforeTest
+    @BeforeClass
     public void edgeSetup(){
         DesiredCapabilities capabilities = DesiredCapabilities.edge();
         EdgeOptions options = new EdgeOptions();
         capabilities.setCapability(ChromeOptions.CAPABILITY, options);
         driver = new EdgeDriver(capabilities);
     }
-
     @Test(groups = {"windows"})
     public void test(){
         driver.get("http://lazycoder.io/about.html");

mac-only.xml

@@ -6,16 +6,14 @@
 import org.openqa.selenium.firefox.FirefoxProfile;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.AfterTest;
-import org.testng.annotations.BeforeTest;
-import org.testng.annotations.Test;
+import org.testng.annotations.*;
 
 import java.io.File;
 
 /**
  * Created by andrew on 12/3/16.
  */
-
+@Test(groups = {"mac", "windows"})
 public class Firefox {
     WebDriver driver;
 
@@ -27,8 +25,7 @@ public void firefoxSetup() {
         FirefoxProfile ffp = new FirefoxProfile();
         driver = new FirefoxDriver(ff, ffp, capabilities);
     }
-
-    @Test(groups = {"mac", "windows"})
+    @Test
     public void test(){
         driver.get("http://lazycoder.io/about.html");
         Assert.assertEquals(driver.getTitle(), "About");

src/test/java/LocalBrowsers/Chrome.java

@@ -4,25 +4,23 @@
 import org.openqa.selenium.phantomjs.PhantomJSDriver;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.AfterTest;
-import org.testng.annotations.BeforeTest;
-import org.testng.annotations.Test;
+import org.testng.annotations.*;
 
 /**
  * Created by andrew on 12/3/16.
  */
+@Test(groups = {"mac", "windows"})
 public class PhantomJS {
     WebDriver driver;
 
     @BeforeTest
     public void phantomJSSetup(){
         //DesiredCapabilities capabilities = new DesiredCapabilities.phantomjs();
-        PhantomJSDriver d = new PhantomJSDriver();
+        driver = new PhantomJSDriver();
     }
-
-    @Test(groups = {"mac", "windows"})
+    @Test
     public void test(){
-        driver.get("http://lazycoder.io/about.html");
+        this.driver.get("http://lazycoder.io/about.html");
         Assert.assertEquals(driver.getTitle(), "About");
     }