From dcb678fe68c1effecf650b5f407488c651e28ce5 Mon Sep 17 00:00:00 2001
From: lixiaoshuang <644968328@qq.com>
Date: Sun, 26 Sep 2021 22:16:26 +0800
Subject: [PATCH 1/2] add unit test for CryptoManager

---
 .../nacos/api/config/CryptoManagerTest.java   | 62 +++++++++++++++++++
 1 file changed, 62 insertions(+)
 create mode 100644 api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java

diff --git a/api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java b/api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java
new file mode 100644
index 00000000000..7afd36293c4
--- /dev/null
+++ b/api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java
@@ -0,0 +1,62 @@
+/*
+ *   Copyright 1999-2018 Alibaba Group Holding Ltd.
+ *
+ *   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.alibaba.nacos.api.config;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+/**
+ * CryptoManagerTest.
+ *
+ * @author lixiaoshuang
+ */
+public class CryptoManagerTest {
+    
+    @Test
+    public void testInstance() {
+        CryptoSpi aes = CryptoManager.instance("rsa");
+        Assert.assertNull(aes);
+    }
+    
+    @Test
+    public void testJoin() {
+        CryptoSpi cryptoSpi = new CryptoSpi() {
+            @Override
+            public String encrypt(String secretKey, String content) {
+                return null;
+            }
+            
+            @Override
+            public String decrypt(String secretKey, String content) {
+                return null;
+            }
+            
+            @Override
+            public String generateSecretKey() {
+                return null;
+            }
+            
+            @Override
+            public String named() {
+                return "aes";
+            }
+        };
+        CryptoManager.join(cryptoSpi);
+        CryptoSpi aes = CryptoManager.instance("aes");
+        Assert.assertNotNull(aes);
+    }
+}
\ No newline at end of file

From 5d19f888708ee5c696b5733f8c7177d8976b67aa Mon Sep 17 00:00:00 2001
From: lixiaoshuang <644968328@qq.com>
Date: Sun, 26 Sep 2021 22:18:58 +0800
Subject: [PATCH 2/2] add unit test for CryptoManager

---
 .../java/com/alibaba/nacos/api/config/CryptoManagerTest.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java b/api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java
index 7afd36293c4..39301093a92 100644
--- a/api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java
+++ b/api/src/test/java/com/alibaba/nacos/api/config/CryptoManagerTest.java
@@ -28,8 +28,8 @@ public class CryptoManagerTest {
     
     @Test
     public void testInstance() {
-        CryptoSpi aes = CryptoManager.instance("rsa");
-        Assert.assertNull(aes);
+        CryptoSpi rsa = CryptoManager.instance("rsa");
+        Assert.assertNull(rsa);
     }
     
     @Test