(servicecomb-java-chassis) branch master updated: change MessageTest encoding tests to deterministic implementation (#4971)

[liubao]

This is an automated email from the ASF dual-hosted git repository.

liubao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git


The following commit(s) were added to refs/heads/master by this push:
     new 3a1fa0759 change MessageTest encoding tests to deterministic 
implementation (#4971)
3a1fa0759 is described below

commit 3a1fa0759477e42ae4f33dea1b721c2b09063abf
Author: Anthony Xu <anthonyyxu.w...@gmail.com>
AuthorDate: Fri Oct 10 09:13:49 2025 -0500

    change MessageTest encoding tests to deterministic implementation (#4971)
---
 .../servicecomb/registry/lightweight/MessageTest.java | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git 
a/service-registry/registry-lightweight/src/test/java/org/apache/servicecomb/registry/lightweight/MessageTest.java
 
b/service-registry/registry-lightweight/src/test/java/org/apache/servicecomb/registry/lightweight/MessageTest.java
index 66181ce87..0e79f3296 100644
--- 
a/service-registry/registry-lightweight/src/test/java/org/apache/servicecomb/registry/lightweight/MessageTest.java
+++ 
b/service-registry/registry-lightweight/src/test/java/org/apache/servicecomb/registry/lightweight/MessageTest.java
@@ -18,10 +18,12 @@
 package org.apache.servicecomb.registry.lightweight;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.fail;
 
 import org.junit.jupiter.api.Test;
 
 import io.vertx.core.json.Json;
+import org.skyscreamer.jsonassert.JSONAssert;
 
 class MessageTest {
   private String toLinuxPrettyJson(Object value) {
@@ -33,7 +35,8 @@ class MessageTest {
   void should_encode_register_type() {
     Message<RegisterRequest> msg = Message.of(MessageType.REGISTER, new 
RegisterRequest());
 
-    assertThat(toLinuxPrettyJson(msg)).isEqualTo(""
+    try {
+      JSONAssert.assertEquals(toLinuxPrettyJson(msg), ""
         + "{\n"
         + "  \"type\" : \"REGISTER\",\n"
         + "  \"body\" : {\n"
@@ -45,7 +48,11 @@ class MessageTest {
         + "    \"status\" : null,\n"
         + "    \"endpoints\" : null\n"
         + "  }\n"
-        + "}");
+        + "}", false);
+    } catch (Exception e) {
+      fail("Failed to compare JSONs: " + e.getMessage(), e);
+    }
+
   }
 
   @Test
@@ -60,14 +67,18 @@ class MessageTest {
   void should_encode_unregister_type() {
     Message<UnregisterRequest> msg = Message.of(MessageType.UNREGISTER, new 
UnregisterRequest());
 
-    assertThat(toLinuxPrettyJson(msg)).isEqualTo(""
+    try {
+      JSONAssert.assertEquals(toLinuxPrettyJson(msg), ""
         + "{\n"
         + "  \"type\" : \"UNREGISTER\",\n"
         + "  \"body\" : {\n"
         + "    \"serviceId\" : null,\n"
         + "    \"instanceId\" : null\n"
         + "  }\n"
-        + "}");
+        + "}", false);
+    } catch (Exception e) {
+      fail("Failed to compare JSONs: " + e.getMessage(), e);
+    }
   }
 
   @Test