Fix ChromeDriver version detection

SeleniumGridExtras/src/main/java/com/groupon/seleniumgridextras/downloader/webdriverreleasemanager/ChromeDriverRelease.java

@@ -6,19 +6,22 @@
 
 public class ChromeDriverRelease extends WebDriverRelease {
 
+  private int buildVersion;
+
   public ChromeDriverRelease(String input) {
     super(input);
 
-    Matcher m = Pattern.compile("(\\d+)\\.(\\d+)").matcher(input);
+    Matcher m = Pattern.compile("(\\d+)\\.(\\d+)\\.(\\d+)\\.(\\d+)").matcher(input);
 
     if (m.find()){
       setMajorVersion(Integer.valueOf(m.group(1)));
       setMinorVersion(Integer.valueOf(m.group(2)));
+      setBuildVersion(Integer.valueOf(m.group(3)));
+      setPatchVersion(Integer.valueOf(m.group(4)));
     }
 
     setName("chromedriver");
     setRelativePath("index.html?path=" + getPrettyPrintVersion(".") + "/");
-
   }
 
   public String getPrettyPrintVersion(String separator){
@@ -27,7 +30,20 @@ public String getPrettyPrintVersion(String separator){
     stringBuilder.append(getMajorVersion());
     stringBuilder.append(separator);
     stringBuilder.append(getMinorVersion());
+    stringBuilder.append(separator);
+    stringBuilder.append(getBuildVersion());
+    stringBuilder.append(separator);
+    stringBuilder.append(getPatchVersion());
 
     return stringBuilder.toString();
   }
+
+  public int getBuildVersion() {
+    return buildVersion;
+  }
+
+  public void setBuildVersion(int buildVersion) {
+    this.buildVersion = buildVersion;
+  }
+
 }

SeleniumGridExtras/src/test/java/com/groupon/seleniumgridextras/downloader/WebDriverReleaseManagerTest.java

@@ -18,7 +18,7 @@ public class WebDriverReleaseManagerTest {
   @Before
   public void setUp() throws Exception {
       URL webDriverAndIEDriverURL = ClassLoader.getSystemResource("fixtures/selenium_release_manifest.xml");
-      URL chromeDriverVersionURL = ClassLoader.getSystemResource("fixtures/selenium_release_version.txt");
+      URL chromeDriverVersionURL = ClassLoader.getSystemResource("fixtures/chromedriver_release_version.txt");
       URL marionetteDriverVersionURL = ClassLoader.getSystemResource("fixtures/selenium_release_version.txt");
       releaseManager = new WebDriverReleaseManager(webDriverAndIEDriverURL, chromeDriverVersionURL, marionetteDriverVersionURL);
   }
@@ -33,7 +33,7 @@ public void testReleaseCounts() throws Exception {
   public void testGetLatestVersion() throws Exception {
     assertEquals("2.41.0", releaseManager.getWedriverLatestVersion().getPrettyPrintVersion("."));
     assertEquals("2.41.0", releaseManager.getIeDriverLatestVersion().getPrettyPrintVersion("."));
-    assertEquals("2.10", releaseManager.getChromeDriverLatestVersion().getPrettyPrintVersion("."));
+    assertEquals("2.10.13.42", releaseManager.getChromeDriverLatestVersion().getPrettyPrintVersion("."));
   }
 
   @Test

SeleniumGridExtras/src/test/java/com/groupon/seleniumgridextras/downloader/webdriverreleasemanager/WebDriverReleaseTest.java

@@ -88,11 +88,11 @@ public void testComparableVersion() throws Exception{
 
   @Test
   public void testChromeDriverRelease() throws Exception{
-    WebDriverRelease chromeRelease = new ChromeDriverRelease("2.10");
+    WebDriverRelease chromeRelease = new ChromeDriverRelease("2.10.13.42");
 
-    assertEquals("2.10", chromeRelease.getPrettyPrintVersion("."));
+    assertEquals("2.10.13.42", chromeRelease.getPrettyPrintVersion("."));
     assertEquals("chromedriver", chromeRelease.getName());
-    assertEquals("index.html?path=2.10/", chromeRelease.getRelativePath());
+    assertEquals("index.html?path=2.10.13.42/", chromeRelease.getRelativePath());
 
   }
 

SeleniumGridExtras/src/test/resources/fixtures/chromedriver_release_version.txt

@@ -0,0 +1 @@
+2.10.13.42