Merge branch 'master' into release-history

CHANGES.md

@@ -33,5 +33,7 @@ Apollo 2.0.0
 * [fix the json number display issue when it's longer than 16](https://github.com/apolloconfig/apollo/pull/4183)
 * [Bump client springboot version](https://github.com/apolloconfig/apollo/pull/4189)
 * [The release history of namespaces that are not properties will also show comments and release times](https://github.com/apolloconfig/apollo/pull/4198)
+* [Add unit tests for Utils](https://github.com/apolloconfig/apollo/pull/4193)
+
 ------------------
 All issues and pull requests are [here](https://github.com/ctripcorp/apollo/milestone/8?closed=1)

apollo-core/src/test/java/com/ctrip/framework/foundation/internals/UtilsTest.java

@@ -0,0 +1,88 @@
+/*
+ * Copyright 2021 Apollo Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ */
+package com.ctrip.framework.foundation.internals;
+
+import org.junit.After;
+import org.junit.Test;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+public class UtilsTest {
+    private final String actualOsName = System.getProperty("os.name");
+
+    @After
+    public void tearDown() {
+        System.setProperty("os.name", actualOsName);
+    }
+
+    @Test
+    public void isBlankTrueGivenNull() {
+        assertTrue(Utils.isBlank(null));
+    }
+
+    @Test
+    public void isBlankTrueGivenEmpty() {
+        assertTrue(Utils.isBlank(""));
+    }
+
+    @Test
+    public void isBlankTrueGivenWhitespace() {
+        assertTrue(Utils.isBlank("   "));
+    }
+
+    @Test
+    public void isBlankFalseGivenLoremIpsum() {
+        assertFalse(Utils.isBlank("Lorem Ipsum"));
+    }
+
+    @Test
+    public void isBlankFalseGivenWhitespacePadded() {
+        assertFalse(Utils.isBlank("   Lorem Ipsum   "));
+    }
+
+    @Test
+    public void isOsWindowsTrueGivenWindows10() {
+        System.setProperty("os.name", "Windows 10");
+        assertTrue(Utils.isOSWindows());
+    }
+
+    @Test
+    public void isOSWindowsFalseGivenMacOsX() {
+        System.setProperty("os.name", "Mac OS X");
+        assertFalse(Utils.isOSWindows());
+    }
+
+    @Test
+    public void isOSWindowsFalseGivenBlank() {
+        System.setProperty("os.name", "");
+        assertFalse(Utils.isOSWindows());
+    }
+
+    // Explicitly calling out case sensitivity; revisit if Microsoft changes naming convention
+    @Test
+    public void isOSWindowsFalseGivenAllUppercaseWindows() {
+        System.setProperty("os.name", "WINDOWS 10");
+        assertFalse(Utils.isOSWindows());
+    }
+
+    @Test
+    public void isOSWindowsFalseGivenAllLowercaseWindows() {
+        System.setProperty("os.name", "windows 10");
+        assertFalse(Utils.isOSWindows());
+    }
+}