[java] fix aliases for windows 7 and vista platform name

SeleniumHQ · Sep 23, 2022 · 31190f8 · 31190f8
1 parent 0b3bc09
commit 31190f8
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/java/src/org/openqa/selenium/Platform.java b/java/src/org/openqa/selenium/Platform.java
@@ -58,7 +58,7 @@ public String toString() {
   /**
    * For versions of Windows that "feel like" Windows Vista.
    */
-  VISTA("windows 7", "win7") {
+  VISTA("windows vista", "Windows Server 2008") {
     @Override
     public Platform family() {
       return WINDOWS;

diff --git a/java/test/org/openqa/selenium/PlatformTest.java b/java/test/org/openqa/selenium/PlatformTest.java
@@ -100,7 +100,7 @@ public void testShouldIdentifyXPVariants() {
 
   @Test
   public void testShouldIdentifyVistaVariants() {
-    assertAllAre(Platform.VISTA, "Windows Vista", "windows server 2008", "Windows 7", "win7");
+    assertAllAre(Platform.VISTA, "Windows Vista", "windows server 2008");
   }
 
   @Test

diff --git a/java/test/org/openqa/selenium/remote/DesiredCapabilitiesTest.java b/java/test/org/openqa/selenium/remote/DesiredCapabilitiesTest.java
@@ -94,7 +94,7 @@ public void testExtractDebugLogLevelFromCapabilityMap() {
   @Test
   public void shouldAutomaticallyConvertPlatformFromStringToEnum() {
     DesiredCapabilities caps = new DesiredCapabilities();
-    caps.setCapability(CapabilityType.PLATFORM_NAME, "windows 7");
+    caps.setCapability(CapabilityType.PLATFORM_NAME, "Windows Server 2008");
     assertThat(caps.getCapability(CapabilityType.PLATFORM_NAME)).isEqualTo(Platform.VISTA);
     caps.setCapability(CapabilityType.PLATFORM_NAME, "win8.1");
     assertThat(caps.getCapability(CapabilityType.PLATFORM_NAME)).isEqualTo(Platform.WIN8_1);
-Original file line number
+Diff line change
@@ Expand Up / @@ -100,7 +100,7 @@ public void testShouldIdentifyXPVariants() { @@
       @Test
       public void testShouldIdentifyVistaVariants() {
-        assertAllAre(Platform.VISTA, "Windows Vista", "windows server 2008", "Windows 7", "win7");
+        assertAllAre(Platform.VISTA, "Windows Vista", "windows server 2008");
       }
       @Test
@@ Expand Down @@