Skip to content

cleaning up for codacy #15

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Dec 5, 2016
Merged

cleaning up for codacy #15

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0f0c756
cleaning up for codacy
andrewmkrug Dec 5, 2016
e13e59d
adding debug option to figure out why travis is failing
andrewmkrug Dec 5, 2016
1469d90
removing debug and updating TestNG XML files
andrewmkrug Dec 5, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions mac-only.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 -->
59 changes: 0 additions & 59 deletions src/test/java/LocalBrowsersSampleTest.java
View file
Open in desktop

This file was deleted.

8 changes: 5 additions & 3 deletions src/test/java/LocalBrowsers/Chrome.java → src/test/java/localbrowsers/Chrome.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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(){
Expand Down
9 changes: 5 additions & 4 deletions src/test/java/LocalBrowsers/Edge.java → src/test/java/localbrowsers/Edge.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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(){
Expand Down
8 changes: 5 additions & 3 deletions src/test/java/LocalBrowsers/Firefox.java → src/test/java/localbrowsers/Firefox.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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;

Expand All @@ -15,7 +17,7 @@
*/
@Test(groups = {"mac", "windows"})
public class Firefox {
WebDriver driver;
private WebDriver driver;

@BeforeTest
public void firefoxSetup() {
Expand Down
9 changes: 5 additions & 4 deletions src/test/java/LocalBrowsers/PhantomJS.java → src/test/java/localbrowsers/PhantomJS.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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(){
Expand Down
8 changes: 5 additions & 3 deletions src/test/java/LocalBrowsers/Safari.java → src/test/java/localbrowsers/Safari.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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(){
Expand Down
10 changes: 5 additions & 5 deletions windows-only.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 -->