[ISSUE alibaba#11255]Update PathEncoderManagerTest#testEncodeWithNonE…

…xistOs.
stone-98 · Oct 17, 2023 · 54e2632 · 54e2632
1 parent c9f77e5
commit 54e2632
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/common/src/test/java/com/alibaba/nacos/common/pathencoder/PathEncoderManagerTest.java b/common/src/test/java/com/alibaba/nacos/common/pathencoder/PathEncoderManagerTest.java
@@ -68,6 +68,18 @@ public void testInitWithWindows()
      */
     @Test
     public void testWindowsEncode() throws Exception {
+        // load static
+        PathEncoderManager instance = PathEncoderManager.getInstance();
+        String case1 = "aa||a";
+        String case2 = "aa%A9%%A9%a";
+        // try to encode if in windows
+        targetEncoder.set(instance, new WindowsEncoder());
+        Assert.assertEquals(PathEncoderManager.getInstance().encode(case1), case2);
+        Assert.assertEquals(PathEncoderManager.getInstance().decode(case2), case1);
+    }
+
+    @Test
+    public void testEncodeWithNonExistOs() throws Exception {
         // load static
         PathEncoderManager instance = PathEncoderManager.getInstance();
         // remove impl
@@ -77,10 +89,6 @@ public void testWindowsEncode() throws Exception {
         Assert.assertEquals(PathEncoderManager.getInstance().encode(case1), case1);
         String case2 = "aa%A9%%A9%a";
         Assert.assertEquals(PathEncoderManager.getInstance().decode(case2), case2);
-        // try to encode if in windows
-        targetEncoder.set(instance, new WindowsEncoder());
-        Assert.assertEquals(PathEncoderManager.getInstance().encode(case1), case2);
-        Assert.assertEquals(PathEncoderManager.getInstance().decode(case2), case1);
     }
 
     @Test