From 382d4b423dbf2a1a07f9f97274e94fb613ce6c2a Mon Sep 17 00:00:00 2001
From: godotg <jaysunxiao@gmail.com>
Date: Tue, 2 Jul 2024 18:57:23 +0800
Subject: [PATCH] test[protocol]: test

---
 .../protocol/collection/ConcurrentHashMapTest.java    |  3 +--
 .../com/zfoo/protocol/collection/GeneralTreeTest.java |  5 +++--
 .../com/zfoo/protocol/collection/HashMapTest.java     |  3 +--
 .../java/com/zfoo/protocol/util/DomUtilsTest.java     | 11 +++++------
 .../java/com/zfoo/protocol/util/JsonUtilTest.java     |  5 -----
 5 files changed, 10 insertions(+), 17 deletions(-)

diff --git a/protocol/src/test/java/com/zfoo/protocol/collection/ConcurrentHashMapTest.java b/protocol/src/test/java/com/zfoo/protocol/collection/ConcurrentHashMapTest.java
index f3953ced..81856c9d 100644
--- a/protocol/src/test/java/com/zfoo/protocol/collection/ConcurrentHashMapTest.java
+++ b/protocol/src/test/java/com/zfoo/protocol/collection/ConcurrentHashMapTest.java
@@ -46,7 +46,6 @@ public class ConcurrentHashMapTest {
     @Test
     public void primitiveMapTest() {
         var map = new LongObjectHashMap<Integer>();
-        var startTime = System.currentTimeMillis();
 
         var num = 10;
 
@@ -67,7 +66,7 @@ public class ConcurrentHashMapTest {
             }
         }
 
-        System.out.println(System.currentTimeMillis() - startTime);
+        Assert.assertTrue(map.isEmpty());
     }
 
 }
diff --git a/protocol/src/test/java/com/zfoo/protocol/collection/GeneralTreeTest.java b/protocol/src/test/java/com/zfoo/protocol/collection/GeneralTreeTest.java
index 47d65120..ca46fa4b 100644
--- a/protocol/src/test/java/com/zfoo/protocol/collection/GeneralTreeTest.java
+++ b/protocol/src/test/java/com/zfoo/protocol/collection/GeneralTreeTest.java
@@ -13,6 +13,7 @@
 package com.zfoo.protocol.collection;
 
 import com.zfoo.protocol.collection.tree.GeneralTree;
+import org.junit.Assert;
 import org.junit.Test;
 
 /**
@@ -30,8 +31,8 @@ public class GeneralTreeTest {
         generalTree.addNode("a.b.d", "world2");
         generalTree.addNode("a.b.e", "world3");
 
-        System.out.println(generalTree.getNodeByPath("a.b").getData());
-        System.out.println(generalTree.getNodeByPath("a.b").getChildren());
+        Assert.assertEquals("hello", generalTree.getNodeByPath("a.b").getData());
+        Assert.assertEquals("[[a.b.c]:[world1], [a.b.d]:[world2], [a.b.e]:[world3]]", generalTree.getNodeByPath("a.b").getChildren().toString());
     }
 
 }
diff --git a/protocol/src/test/java/com/zfoo/protocol/collection/HashMapTest.java b/protocol/src/test/java/com/zfoo/protocol/collection/HashMapTest.java
index c83fda46..1cc55ab3 100644
--- a/protocol/src/test/java/com/zfoo/protocol/collection/HashMapTest.java
+++ b/protocol/src/test/java/com/zfoo/protocol/collection/HashMapTest.java
@@ -37,8 +37,7 @@ public class HashMapTest {
         map.put(1, 1);
         map.put(2, 2);
         map.put(3, 3);
-        System.out.println(map.size());
-        System.out.println(map);
+        Assert.assertEquals(map.size(), 4);
     }
 
     private void assertKey(HashMapIntInt primitiveMap, HashMap<Integer, Integer> javaMap, int key) {
diff --git a/protocol/src/test/java/com/zfoo/protocol/util/DomUtilsTest.java b/protocol/src/test/java/com/zfoo/protocol/util/DomUtilsTest.java
index e8c208ec..1e5c574f 100644
--- a/protocol/src/test/java/com/zfoo/protocol/util/DomUtilsTest.java
+++ b/protocol/src/test/java/com/zfoo/protocol/util/DomUtilsTest.java
@@ -46,17 +46,16 @@ public class DomUtilsTest {
     @Test
     public void testXmlWithHead() {
         var protos = DomUtils.string2Object(XML_WITH_HEAD, XmlProtocols.class);
-        for (var module : protos.getModules()) {
-            System.out.println(module.getId());
-        }
+        Assert.assertEquals(protos.getModules().get(0).getId(), 1);
+        Assert.assertEquals(protos.getModules().get(1).getId(), 2);
+        Assert.assertEquals(protos.getModules().get(0).getProtocols().get(0).getId(), 1000);
     }
 
     @Test
     public void testXmlOfStandardText() {
         var protos = DomUtils.string2Object(XML_OF_STANDARD_TEXT, XmlProtocols.class);
-        for (var module : protos.getModules()) {
-            System.out.println(module.getId());
-        }
+        Assert.assertEquals(protos.getModules().get(0).getId(), 1);
+        Assert.assertEquals(protos.getModules().get(1).getId(), 2);
     }
 
 }
diff --git a/protocol/src/test/java/com/zfoo/protocol/util/JsonUtilTest.java b/protocol/src/test/java/com/zfoo/protocol/util/JsonUtilTest.java
index cedb9297..70f6e619 100644
--- a/protocol/src/test/java/com/zfoo/protocol/util/JsonUtilTest.java
+++ b/protocol/src/test/java/com/zfoo/protocol/util/JsonUtilTest.java
@@ -111,10 +111,5 @@ public class JsonUtilTest {
         var temp = JsonUtils.string2Object(tripleStr, Triple.class);
     }
 
-    @Test
-    public void prettyPrinterTest() {
-        var user = JsonUtils.string2Object(userJson, User.class);
-        System.out.println(JsonUtils.object2StringPrettyPrinter(user));
-    }
 }