Skip to content

Commit f12a4c2

Browse files
committed
fix sample tests for Windows, fix global setup.
rename ApplicationPlatform to PlatformName for consistency
1 parent a34533d commit f12a4c2

File tree

6 files changed

+22
-24
lines changed

6 files changed

+22
-24
lines changed

src/main/java/aquality/appium/application/Application.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,10 +35,10 @@ public AppiumDriver getDriver() {
3535
}
3636

3737
/**
38-
* Returns platform of current application
38+
* Returns name of current platform
3939
* @return name
4040
*/
41-
public final ApplicationPlatform geApplicationPlatform() {
42-
return configuration.getApplicationProfile().getApplicationName();
41+
public final PlatformName getPlatformName() {
42+
return configuration.getApplicationProfile().getPlatformName();
4343
}
4444
}
Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package aquality.appium.application;
22

3-
public enum ApplicationPlatform {
3+
public enum PlatformName {
44
ANDROID,
5-
IOS
5+
IOS,
6+
WINDOWS
67
}

src/main/java/aquality/appium/configuration/IApplicationProfile.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
package aquality.appium.configuration;
22

3-
import aquality.appium.application.ApplicationPlatform;
3+
import aquality.appium.application.PlatformName;
44
import aquality.appium.configuration.driversettings.IDriverSettings;
55

66
import java.net.URL;
77

88
public interface IApplicationProfile {
99

10-
ApplicationPlatform getApplicationName();
10+
PlatformName getPlatformName();
1111

1212
boolean isRemote();
1313

src/main/java/aquality/appium/configuration/driversettings/IDriverSettings.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package aquality.appium.configuration.driversettings;
22

3-
import aquality.appium.application.ApplicationPlatform;
3+
import aquality.appium.application.PlatformName;
44
import aquality.appium.utils.JsonFile;
55
import org.openqa.selenium.Capabilities;
66

@@ -16,7 +16,7 @@ public interface IDriverSettings {
1616

1717
String getDownloadDirCapabilityKey();
1818

19-
ApplicationPlatform getApplicationPlatform();
19+
PlatformName getPlatformName();
2020

2121
JsonFile getSettingsFile();
2222

src/test/java/samples/BaseTest.java

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,27 +4,24 @@
44
import org.testng.annotations.AfterSuite;
55
import org.testng.annotations.BeforeSuite;
66

7-
import java.io.IOException;
8-
import java.net.MalformedURLException;
97
import java.net.URL;
108

119
public abstract class BaseTest {
1210
private static AppiumDriverLocalService service;
1311

1412
@BeforeSuite
15-
public void globalSetup () throws IOException {
16-
/* todo: fix
13+
public void globalSetup () {
1714
service = AppiumDriverLocalService.buildDefaultService();
18-
service.start();*/
15+
service.start();
1916
}
2017

2118
@AfterSuite
2219
public void globalTearDown () {
23-
//service.stop();
20+
service.stop();
2421
}
2522

26-
public URL getServiceUrl () throws MalformedURLException {
27-
return new URL(String.format("http://%1$s:%2$s/wd/hub", "0.0.0.0", "4723"));
28-
//return service.getUrl();
23+
public URL getServiceUrl () {
24+
// todo: this is for remote run return new URL(String.format("http://%1$s:%2$s/wd/hub", "127.0.0.1", "4723"));
25+
return service.getUrl();
2926
}
3027
}

src/test/java/samples/windows/WindowsDesktopAppTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ public class WindowsDesktopAppTest extends BaseTest {
1515
public static WindowsDriver<?> driver;
1616

1717
@BeforeTest
18-
public void setup() throws MalformedURLException {
18+
public void setup() {
1919
DesiredCapabilities caps = new DesiredCapabilities();
2020
caps.setCapability("platformVersion", "10");
2121
caps.setCapability("platformName", "Windows");
@@ -32,10 +32,10 @@ public void tearDown() {
3232

3333
@Test
3434
public void test() {
35-
driver.findElementByName("One").click();
36-
driver.findElementByName("Plus").click();
37-
driver.findElementByName("Two").click();
38-
driver.findElementByName("Equals").click();
39-
Assert.assertEquals(driver.findElementByAccessibilityId("CalculatorResults").getText(), "Display is 3");
35+
driver.findElementByXPath("//*[@AutomationId='num1Button']").click();
36+
driver.findElementByAccessibilityId("plusButton").click();
37+
driver.findElementByAccessibilityId("num2Button").click();
38+
driver.findElementByAccessibilityId("equalButton").click();
39+
Assert.assertTrue(driver.findElementByAccessibilityId("CalculatorResults").getText().endsWith("3"), "Display result is not match to expected");
4040
}
4141
}

0 commit comments

Comments
 (0)