fix: Fixed an issue with the screenplay webdriver integration

Browser windows were not closing correctly when there were more than one driver used in a single test.
serenity-bdd · Jun 2, 2016 · 0ea09c7 · 0ea09c7
1 parent 0222177
commit 0ea09c7
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/serenity-core/src/main/java/net/thucydides/core/webdriver/WebdriverInstances.java b/serenity-core/src/main/java/net/thucydides/core/webdriver/WebdriverInstances.java
@@ -1,8 +1,6 @@
 package net.thucydides.core.webdriver;
 
 import com.google.common.collect.Lists;
-import net.thucydides.core.guice.Injectors;
-import net.thucydides.core.util.EnvironmentVariables;
 import org.apache.commons.lang3.StringUtils;
 import org.openqa.selenium.WebDriver;
 
@@ -202,7 +200,9 @@ public InstanceRegistration(final String driverName) {
 
 
         public void forDriver(final WebDriver driver) {
-            driverMap.put(normalized(driverName), driver);
+            if (!driverMap.values().contains(driver)) {
+                driverMap.put(driverName, driver);
+            }
 //            currentDriver = normalized(driverName);
         }
     }