From 115f7740d137bd04ddaf738f2b1ade357fe2a64d Mon Sep 17 00:00:00 2001
From: li-xiao-shuang <644968328@qq.com>
Date: Sat, 8 May 2021 10:59:49 +0800
Subject: [PATCH 1/2] add unit test for MetadataOperation

---
 .../v2/metadata/MetadataOperationTest.java    | 109 ++++++++++++++++++
 1 file changed, 109 insertions(+)
 create mode 100644 naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java

diff --git a/naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java b/naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java
new file mode 100644
index 00000000000..dba817dbaee
--- /dev/null
+++ b/naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java
@@ -0,0 +1,109 @@
+/*
+ * 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.naming.core.v2.metadata;
+
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.junit.MockitoJUnitRunner;
+
+@RunWith(MockitoJUnitRunner.class)
+public class MetadataOperationTest {
+    
+    private MetadataOperation<String> metadataOperation;
+    
+    @Before
+    public void setUp() {
+        metadataOperation = new MetadataOperation();
+    }
+    
+    @Test
+    public void getNamespace() {
+        String namespace = metadataOperation.getNamespace();
+        
+        Assert.assertNull(namespace);
+    }
+    
+    @Test
+    public void setNamespace() {
+        String namespace = "2398479283749823984";
+        metadataOperation.setNamespace(namespace);
+        
+        Assert.assertEquals(metadataOperation.getNamespace(), namespace);
+    }
+    
+    @Test
+    public void getGroup() {
+        String group = metadataOperation.getGroup();
+        
+        Assert.assertNull(group);
+    }
+    
+    @Test
+    public void setGroup() {
+        String group = "default";
+        metadataOperation.setGroup(group);
+        
+        Assert.assertEquals(metadataOperation.getGroup(), group);
+    }
+    
+    @Test
+    public void getServiceName() {
+        String serviceName = metadataOperation.getServiceName();
+        
+        Assert.assertNull(serviceName);
+    }
+    
+    @Test
+    public void setServiceName() {
+        String serviceName = "nacos";
+        metadataOperation.setServiceName(serviceName);
+        
+        Assert.assertEquals(metadataOperation.getServiceName(), serviceName);
+    }
+    
+    @Test
+    public void getTag() {
+        String tag = metadataOperation.getTag();
+        
+        Assert.assertNull(tag);
+    }
+    
+    @Test
+    public void setTag() {
+        String tag = "tag";
+        metadataOperation.setTag(tag);
+        
+        Assert.assertEquals(metadataOperation.getTag(), tag);
+    }
+    
+    @Test
+    public void getMetadata() {
+        Object metadata = metadataOperation.getMetadata();
+        
+        Assert.assertNull(metadata);
+    }
+    
+    @Test
+    public void setMetadata() {
+        String metadata = "metadata";
+        metadataOperation.setMetadata(metadata);
+        
+        Assert.assertEquals(metadataOperation.getMetadata(), metadata);
+    }
+}
\ No newline at end of file

From c5f62d1dad9fb903d3c458df845b7159c4639d66 Mon Sep 17 00:00:00 2001
From: li-xiao-shuang <644968328@qq.com>
Date: Sat, 8 May 2021 11:07:27 +0800
Subject: [PATCH 2/2] update method name

---
 .../v2/metadata/MetadataOperationTest.java    | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java b/naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java
index dba817dbaee..9d77e513974 100644
--- a/naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java
+++ b/naming/src/test/java/com/alibaba/nacos/naming/core/v2/metadata/MetadataOperationTest.java
@@ -33,14 +33,14 @@ public void setUp() {
     }
     
     @Test
-    public void getNamespace() {
+    public void testGetNamespace() {
         String namespace = metadataOperation.getNamespace();
         
         Assert.assertNull(namespace);
     }
     
     @Test
-    public void setNamespace() {
+    public void testSetNamespace() {
         String namespace = "2398479283749823984";
         metadataOperation.setNamespace(namespace);
         
@@ -48,14 +48,14 @@ public void setNamespace() {
     }
     
     @Test
-    public void getGroup() {
+    public void testGetGroup() {
         String group = metadataOperation.getGroup();
         
         Assert.assertNull(group);
     }
     
     @Test
-    public void setGroup() {
+    public void testSetGroup() {
         String group = "default";
         metadataOperation.setGroup(group);
         
@@ -63,14 +63,14 @@ public void setGroup() {
     }
     
     @Test
-    public void getServiceName() {
+    public void testGetServiceName() {
         String serviceName = metadataOperation.getServiceName();
         
         Assert.assertNull(serviceName);
     }
     
     @Test
-    public void setServiceName() {
+    public void testSetServiceName() {
         String serviceName = "nacos";
         metadataOperation.setServiceName(serviceName);
         
@@ -78,14 +78,14 @@ public void setServiceName() {
     }
     
     @Test
-    public void getTag() {
+    public void testGetTag() {
         String tag = metadataOperation.getTag();
         
         Assert.assertNull(tag);
     }
     
     @Test
-    public void setTag() {
+    public void testSetTag() {
         String tag = "tag";
         metadataOperation.setTag(tag);
         
@@ -93,14 +93,14 @@ public void setTag() {
     }
     
     @Test
-    public void getMetadata() {
+    public void testGetMetadata() {
         Object metadata = metadataOperation.getMetadata();
         
         Assert.assertNull(metadata);
     }
     
     @Test
-    public void setMetadata() {
+    public void testSetMetadata() {
         String metadata = "metadata";
         metadataOperation.setMetadata(metadata);