Skip to content

Commit 60c3f81

Browse files
Alvaro LasernaAlvaro Laserna
authored andcommitted
fix ios and android drivers
1 parent 544d7ab commit 60c3f81

3 files changed

Lines changed: 29 additions & 4 deletions

File tree

src/main/java/testUI/TestUIDriver.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,18 @@ public static void removeDriver(int driver) {
130130
TestUIDriver.driver.set(appiumDrivers);
131131
}
132132

133+
public static void removeiOSDriver(int driver) {
134+
List<IOSDriver> appiumiOSDrivers = new ArrayList<>(getIOSDrivers());
135+
appiumiOSDrivers.remove(driver);
136+
TestUIDriver.IOSTestUIDriver.set(appiumiOSDrivers);
137+
}
138+
139+
public static void removeAndroidDriver(int driver) {
140+
List<AndroidDriver> appiumAndroidDrivers = new ArrayList<>(getAndroidDrivers());
141+
appiumAndroidDrivers.remove(driver);
142+
TestUIDriver.AndroidTestUIDriver.set(appiumAndroidDrivers);
143+
}
144+
133145
public static WebDriver getSelenideDriver() {
134146
return WebDriverRunner.getWebDriver();
135147
}

src/main/java/testUI/TestUIServer.java

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -278,11 +278,13 @@ public static void stop(int driver) {
278278
removeUsePort(driver - 1);
279279
removeUseBootstrapPort(driver - 1);
280280
if (Configuration.automationType.equals(IOS_PLATFORM)) {
281-
if (getDrivers().size() != 0)
282-
getDrivers().get(driver - 1).close();
283-
sleep(500);
281+
if (!getIOSDrivers().isEmpty())
282+
removeiOSDriver(driver - 1);
283+
} else if (Configuration.automationType.equals(ANDROID_PLATFORM)) {
284+
if (!getAndroidDrivers().isEmpty())
285+
removeAndroidDriver(driver - 1);
284286
}
285-
if (getDrivers().size() != 0) {
287+
if (!getDrivers().isEmpty()) {
286288
getDrivers().get(driver - 1).quit();
287289
removeDriver(driver - 1);
288290
}
@@ -369,6 +371,13 @@ public static void stop() {
369371
removeUseBootstrapPort(driver - 1);
370372
getDrivers().get(driver - 1).quit();
371373
removeDriver(driver - 1);
374+
if (Configuration.automationType.equals(IOS_PLATFORM)) {
375+
if (!getIOSDrivers().isEmpty())
376+
removeiOSDriver(driver - 1);
377+
} else if (Configuration.automationType.equals(ANDROID_PLATFORM)) {
378+
if (!getAndroidDrivers().isEmpty())
379+
removeAndroidDriver(driver - 1);
380+
}
372381
if (getAppiumServices() != null && getAppiumServices().size() >= driver) {
373382
getAppiumServices().get(driver - 1).stop();
374383
getAppiumServices().remove(driver - 1);

src/test/java/TestRunners/TestIOS.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import org.junit.Test;
55
import testUI.Configuration;
66

7+
import static testUI.TestUIDriver.getIOSDrivers;
78
import static testUI.TestUIServer.stop;
89
import static testUI.UIOpen.open;
910

@@ -15,8 +16,11 @@ public void testIOSBrowser() {
1516
Configuration.automationType = Configuration.IOS_PLATFORM;
1617
Configuration.serverLogLevel = "debug";
1718
Configuration.useNewWDA = false;
19+
// Configuration.appiumUrl = "http://IP:4723/wd/hub";
1820
open("https://www.facebook.com");
21+
System.out.println(getIOSDrivers().size());
1922
stop();
23+
System.out.println(getIOSDrivers().size());
2024
open("https://www.facebook.com");
2125
}
2226
}

0 commit comments

Comments
 (0)