test(core): add test

Author: Weilong-Qin

agentscope-core/src/main/java/io/agentscope/core/formatter/openai/dto/OpenAIReasoningDetail.java

@@ -18,6 +18,7 @@
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonProperty;
+import java.util.Objects;
 
 /**
  * OpenAI Reasoning Detail DTO (OpenRouter specific for Gemini).
@@ -90,4 +91,26 @@ public Integer getIndex() {
     public void setIndex(Integer index) {
         this.index = index;
     }
+
+    @Override
+    public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
+        if (!(o instanceof OpenAIReasoningDetail)) {
+            return false;
+        }
+        OpenAIReasoningDetail that = (OpenAIReasoningDetail) o;
+        return Objects.equals(this.id, that.id)
+                && Objects.equals(this.type, that.type)
+                && Objects.equals(this.data, that.data)
+                && Objects.equals(this.text, that.text)
+                && Objects.equals(this.format, that.format)
+                && Objects.equals(this.index, that.index);
+    }
+
+    @Override
+    public int hashCode() {
+        return Objects.hash(this.id, this.type, this.data, this.text, this.format, this.index);
+    }
 }

agentscope-core/src/test/java/io/agentscope/core/formatter/openai/dto/OpenAIMessageReasoningFieldTest.java

@@ -16,6 +16,7 @@
 package io.agentscope.core.formatter.openai.dto;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
 import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
@@ -168,4 +169,31 @@ void testDeserializeVllmStreamingChunk() {
 
         assertEquals("Step 1: parse the input...", delta.getReasoningContent());
     }
+
+    @Test
+    @DisplayName("Should compare reasoning details by value")
+    void testReasoningDetailEqualsAndHashCodeUseValues() {
+        OpenAIReasoningDetail first = reasoningDetail("reasoning.text");
+        OpenAIReasoningDetail second = reasoningDetail("reasoning.text");
+        OpenAIReasoningDetail different = reasoningDetail("reasoning.summary");
+
+        assertEquals(first, first);
+        assertNotEquals(first, null);
+        assertNotEquals(first, "other");
+        assertEquals(first, second);
+        assertEquals(second, first);
+        assertEquals(first.hashCode(), second.hashCode());
+        assertNotEquals(first, different);
+    }
+
+    private OpenAIReasoningDetail reasoningDetail(String type) {
+        OpenAIReasoningDetail detail = new OpenAIReasoningDetail();
+        detail.setId("reasoning-1");
+        detail.setType(type);
+        detail.setData("encrypted-data");
+        detail.setText("visible reasoning");
+        detail.setFormat("openai-responses-v1");
+        detail.setIndex(0);
+        return detail;
+    }
 }

agentscope-core/src/test/java/io/agentscope/core/model/OpenAIChatModelTest.java

@@ -17,6 +17,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
@@ -248,6 +249,21 @@ void testGetModelName() {
         assertEquals("gpt-4", model.getModelName());
     }
 
+    @Test
+    @DisplayName("Should compare chat usage by value")
+    void testChatUsageEqualsAndHashCodeUseValues() {
+        ChatUsage first = ChatUsage.builder().inputTokens(10).outputTokens(20).time(1.5).build();
+        ChatUsage second = ChatUsage.builder().inputTokens(10).outputTokens(20).time(1.5).build();
+        ChatUsage different =
+                ChatUsage.builder().inputTokens(10).outputTokens(21).time(1.5).build();
+
+        assertEquals(first, first);
+        assertNotEquals(first, null);
+        assertEquals(first, second);
+        assertEquals(first.hashCode(), second.hashCode());
+        assertNotEquals(first, different);
+    }
+
     @Test
     @DisplayName("Should build model with custom endpoint path")
     void testBuildModelWithEndpointPath() throws Exception {

agentscope-core/src/test/java/io/agentscope/core/session/ListHashUtilTest.java

@@ -150,13 +150,6 @@ void testComputeHashEquivalentMessagesWithChatUsage() {
 
     @Test
     void testComputeHashEquivalentThinkingBlocksWithReasoningDetails() {
-        OpenAIReasoningDetail detail = new OpenAIReasoningDetail();
-        detail.setId("reasoning-1");
-        detail.setType("reasoning.text");
-        detail.setData("encrypted-data");
-        detail.setText("visible reasoning");
-        detail.setFormat("openai-responses-v1");
-        detail.setIndex(0);
         List<Msg> first =
                 List.of(
                         Msg.builder()
@@ -170,7 +163,7 @@ void testComputeHashEquivalentThinkingBlocksWithReasoningDetails() {
                                                         Map.of(
                                                                 ThinkingBlock
                                                                         .METADATA_REASONING_DETAILS,
-                                                                List.of(detail)))
+                                                                List.of(createReasoningDetail())))
                                                 .build())
                                 .build());
 
@@ -187,7 +180,7 @@ void testComputeHashEquivalentThinkingBlocksWithReasoningDetails() {
                                                         Map.of(
                                                                 ThinkingBlock
                                                                         .METADATA_REASONING_DETAILS,
-                                                                List.of(detail)))
+                                                                List.of(createReasoningDetail())))
                                                 .build())
                                 .build());
 
@@ -344,4 +337,15 @@ private List<Msg> createMsgList(int size) {
         }
         return list;
     }
+
+    private OpenAIReasoningDetail createReasoningDetail() {
+        OpenAIReasoningDetail detail = new OpenAIReasoningDetail();
+        detail.setId("reasoning-1");
+        detail.setType("reasoning.text");
+        detail.setData("encrypted-data");
+        detail.setText("visible reasoning");
+        detail.setFormat("openai-responses-v1");
+        detail.setIndex(0);
+        return detail;
+    }
 }