browser version capability should be a string when parsing command li…

…ne options for a node. fixes SeleniumHQ#3204
anilkumarn02 · Dec 5, 2016 · 465ef4d · 465ef4d
1 parent bd3f14c
commit 465ef4d
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/...penqa/grid/internal/utils/configuration/converters/BrowserDesiredCapabilityConverter.java b/...penqa/grid/internal/utils/configuration/converters/BrowserDesiredCapabilityConverter.java
@@ -20,6 +20,7 @@
 import com.beust.jcommander.IStringConverter;
 
 import org.openqa.selenium.Platform;
+import org.openqa.selenium.remote.CapabilityType;
 import org.openqa.selenium.remote.DesiredCapabilities;
 
 public class BrowserDesiredCapabilityConverter implements IStringConverter<DesiredCapabilities> {
@@ -30,6 +31,11 @@ public DesiredCapabilities convert(String value) {
       String[] pieces = cap.split("=");
       String capabilityName = pieces[0].trim();
       String capabilityValue = pieces[1].trim();
+      if (capabilityName.equals(CapabilityType.VERSION)) {
+        // store version as a string, DesiredCapabilities assumes version is a string
+        capabilities.setCapability(capabilityName, capabilityValue);
+        continue;
+      }
       try {
         final Long x = Long.parseLong(capabilityValue);
         capabilities.setCapability(capabilityName, x);