Merge pull request #15 from lazycoderio/codacy

cleaning up for codacy

Author: andrewmkrug

mac-only.xml

@@ -8,11 +8,11 @@
             </run>
         </groups>
         <classes>
-            <class name="LocalBrowsers.Chrome"/>
-            <class name="LocalBrowsers.Edge"/>
-            <class name="LocalBrowsers.Firefox"/>
-            <class name="LocalBrowsers.PhantomJS"/>
-            <class name="LocalBrowsers.Safari"/>
+            <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 -->

src/test/java/LocalBrowsersSampleTest.java

@@ -1,18 +1,20 @@
-package LocalBrowsers;
+package localbrowsers;
 
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.chrome.ChromeDriver;
 import org.openqa.selenium.chrome.ChromeOptions;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.*;
+import org.testng.annotations.AfterTest;
+import org.testng.annotations.BeforeTest;
+import org.testng.annotations.Test;
 
 /**
  * Created by andrew on 12/3/16.
  */
 @Test(groups = {"mac", "windows"})
 public class Chrome {
-    WebDriver driver;
+    private WebDriver driver;
 
     @BeforeTest
     public void chromeSetup(){

src/test/java/LocalBrowsers/Chrome.java renamed to src/test/java/localbrowsers/Chrome.java

@@ -1,19 +1,20 @@
-package LocalBrowsers;
+package localbrowsers;
 
 import org.openqa.selenium.WebDriver;
-import org.openqa.selenium.chrome.ChromeDriver;
 import org.openqa.selenium.chrome.ChromeOptions;
 import org.openqa.selenium.edge.EdgeDriver;
 import org.openqa.selenium.edge.EdgeOptions;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.*;
+import org.testng.annotations.AfterTest;
+import org.testng.annotations.BeforeClass;
+import org.testng.annotations.Test;
 
 /**
  * Created by andrew on 12/3/16.
  */
 public class Edge {
-    WebDriver driver;
+    private WebDriver driver;
 
     @BeforeClass
     public void edgeSetup(){

src/test/java/LocalBrowsers/Edge.java renamed to src/test/java/localbrowsers/Edge.java

@@ -1,12 +1,14 @@
-package LocalBrowsers;
+package localbrowsers;
 
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.firefox.FirefoxBinary;
 import org.openqa.selenium.firefox.FirefoxDriver;
 import org.openqa.selenium.firefox.FirefoxProfile;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.*;
+import org.testng.annotations.AfterTest;
+import org.testng.annotations.BeforeTest;
+import org.testng.annotations.Test;
 
 import java.io.File;
 
@@ -15,7 +17,7 @@
  */
 @Test(groups = {"mac", "windows"})
 public class Firefox {
-    WebDriver driver;
+    private WebDriver driver;
 
     @BeforeTest
     public void firefoxSetup() {

src/test/java/LocalBrowsers/Firefox.java renamed to src/test/java/localbrowsers/Firefox.java

@@ -1,17 +1,18 @@
-package LocalBrowsers;
+package localbrowsers;
 
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.phantomjs.PhantomJSDriver;
-import org.openqa.selenium.remote.DesiredCapabilities;
 import org.testng.Assert;
-import org.testng.annotations.*;
+import org.testng.annotations.AfterTest;
+import org.testng.annotations.BeforeTest;
+import org.testng.annotations.Test;
 
 /**
  * Created by andrew on 12/3/16.
  */
 @Test(groups = {"mac", "windows"})
 public class PhantomJS {
-    WebDriver driver;
+    private WebDriver driver;
 
     @BeforeTest
     public void phantomJSSetup(){

src/test/java/LocalBrowsers/PhantomJS.java renamed to src/test/java/localbrowsers/PhantomJS.java

@@ -1,18 +1,20 @@
-package LocalBrowsers;
+package localbrowsers;
 
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import org.openqa.selenium.safari.SafariDriver;
 import org.openqa.selenium.safari.SafariOptions;
 import org.testng.Assert;
-import org.testng.annotations.*;
+import org.testng.annotations.AfterTest;
+import org.testng.annotations.BeforeTest;
+import org.testng.annotations.Test;
 
 /**
  * Created by andrew on 12/3/16.
  */
 @Test//(groups = {"mac"})
 public class Safari {
-    WebDriver driver;
+    private WebDriver driver;
 
     @BeforeTest
     public void safariSetup(){

src/test/java/LocalBrowsers/Safari.java renamed to src/test/java/localbrowsers/Safari.java

@@ -8,11 +8,11 @@
             </run>
         </groups>
         <classes>
-            <class name="LocalBrowsers.Chrome"/>
-            <class name="LocalBrowsers.Edge"/>
-            <class name="LocalBrowsers.Firefox"/>
-            <class name="LocalBrowsers.PhantomJS"/>
-            <class name="LocalBrowsers.Safari"/>
+            <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 -->