Skip to content

Commit 36cbb8b

Browse files
committed
style(tests): format code for improved readability in test files
Signed-off-by: liuhy <liuhongyu@apache.org>
1 parent 9a775b9 commit 36cbb8b

5 files changed

Lines changed: 30 additions & 17 deletions

File tree

agentscope-core/src/test/java/io/agentscope/core/formatter/gemini/GeminiConversationMergerTest.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,11 @@ void testMergeTextConversation() {
6767
ToolResultBlock.builder()
6868
.id("call-1")
6969
.name("search")
70-
.output(List.of(TextBlock.builder().text("ok").build()))
70+
.output(
71+
List.of(
72+
TextBlock.builder()
73+
.text("ok")
74+
.build()))
7175
.build()))
7276
.build();
7377

@@ -131,8 +135,7 @@ void testMergeTextThenMediaConversation() {
131135
.build();
132136

133137
GeminiContent merged =
134-
merger.mergeToContent(
135-
List.of(msg), m -> "agent", blocks -> "", "# Prompt\n");
138+
merger.mergeToContent(List.of(msg), m -> "agent", blocks -> "", "# Prompt\n");
136139

137140
assertEquals(3, merged.getParts().size());
138141
assertTrue(merged.getParts().get(0).getText().startsWith("# Prompt\n<history>agent: Look"));
@@ -155,4 +158,3 @@ void testMergeEmptyConversation() {
155158
assertTrue(merged.getParts().isEmpty());
156159
}
157160
}
158-

agentscope-core/src/test/java/io/agentscope/core/formatter/gemini/GeminiMessageConverterTest.java

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1001,7 +1001,9 @@ void testToolCallUsesPrecedingThinkingSignature() {
10011001
@DisplayName("Should prefer metadata thought signature over preceding thinking signature")
10021002
void testToolCallMetadataSignatureOverridesThinkingSignature() {
10031003
ThinkingBlock thinking =
1004-
ThinkingBlock.builder().metadata(Map.of("thoughtSignature", "sig_from_thinking")).build();
1004+
ThinkingBlock.builder()
1005+
.metadata(Map.of("thoughtSignature", "sig_from_thinking"))
1006+
.build();
10051007
ToolUseBlock toolUse =
10061008
ToolUseBlock.builder()
10071009
.id("call_meta_sig")
@@ -1045,7 +1047,8 @@ void testToolCallByteArrayThoughtSignature() {
10451047
List<GeminiContent> result = converter.convertMessages(List.of(msg));
10461048
GeminiPart part = result.get(0).getParts().get(0);
10471049

1048-
assertEquals(Base64.getEncoder().encodeToString(signatureBytes), part.getThoughtSignature());
1050+
assertEquals(
1051+
Base64.getEncoder().encodeToString(signatureBytes), part.getThoughtSignature());
10491052
}
10501053

10511054
@Test
@@ -1059,15 +1062,20 @@ void testSameRoleNotMergedWhenFunctionExists() {
10591062
.build();
10601063

10611064
Msg assistantToolMsg =
1062-
Msg.builder().name("assistant").role(MsgRole.ASSISTANT).content(List.of(toolUse)).build();
1065+
Msg.builder()
1066+
.name("assistant")
1067+
.role(MsgRole.ASSISTANT)
1068+
.content(List.of(toolUse))
1069+
.build();
10631070
Msg assistantTextMsg =
10641071
Msg.builder()
10651072
.name("assistant")
10661073
.role(MsgRole.ASSISTANT)
10671074
.content(List.of(TextBlock.builder().text("after tool").build()))
10681075
.build();
10691076

1070-
List<GeminiContent> result = converter.convertMessages(List.of(assistantToolMsg, assistantTextMsg));
1077+
List<GeminiContent> result =
1078+
converter.convertMessages(List.of(assistantToolMsg, assistantTextMsg));
10711079

10721080
assertEquals(2, result.size());
10731081
assertFalse(result.get(0).getParts().isEmpty());

agentscope-core/src/test/java/io/agentscope/core/formatter/gemini/GeminiMultiAgentFormatterTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,8 +151,7 @@ void testCustomConversationHistoryPrompt() {
151151
@Test
152152
void testApplyOptionsDelegatesToChatFormatter() {
153153
GeminiRequest request = new GeminiRequest();
154-
GenerateOptions options =
155-
GenerateOptions.builder().temperature(0.7).maxTokens(256).build();
154+
GenerateOptions options = GenerateOptions.builder().temperature(0.7).maxTokens(256).build();
156155

157156
formatter.applyOptions(request, options, null);
158157

agentscope-core/src/test/java/io/agentscope/core/formatter/gemini/GeminiToolsHelperTest.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -200,8 +200,7 @@ void testGenerateResponseSchemaKeepsResponseWrapper() {
200200
@Test
201201
void testUnwrapResponseSchemaViaReflection() throws Exception {
202202
Method method =
203-
GeminiToolsHelper.class.getDeclaredMethod(
204-
"unwrapResponseSchema", Map.class);
203+
GeminiToolsHelper.class.getDeclaredMethod("unwrapResponseSchema", Map.class);
205204
method.setAccessible(true);
206205

207206
Map<String, Object> wrapped =
@@ -233,7 +232,11 @@ void testUnwrapResponseSchemaViaReflection() throws Exception {
233232
"type",
234233
"object",
235234
"properties",
236-
Map.of("x", Map.of("type", "string"), "y", Map.of("type", "string"))));
235+
Map.of(
236+
"x",
237+
Map.of("type", "string"),
238+
"y",
239+
Map.of("type", "string"))));
237240
@SuppressWarnings("unchecked")
238241
Map<String, Object> unchangedProps = (Map<String, Object>) unchanged.get("properties");
239242
assertNotNull(unchangedProps.get("x"));

agentscope-core/src/test/java/io/agentscope/core/message/ThinkingBlockTest.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,7 @@ void testBuilderWithNullThinkingDefaultsToEmptyString() {
4444

4545
@Test
4646
void testSignatureConvenienceMethodSetsMetadata() {
47-
ThinkingBlock block =
48-
ThinkingBlock.builder().thinking("t").signature("sig-123").build();
47+
ThinkingBlock block = ThinkingBlock.builder().thinking("t").signature("sig-123").build();
4948

5049
assertNotNull(block.getMetadata());
5150
assertEquals("sig-123", block.getSignature());
@@ -92,10 +91,12 @@ void testReasoningDetailsMetadataKeyCanBeStored() {
9291
ThinkingBlock block =
9392
ThinkingBlock.builder()
9493
.thinking("t")
95-
.metadata(Map.of(ThinkingBlock.METADATA_REASONING_DETAILS, "[{\"type\":\"text\"}]"))
94+
.metadata(
95+
Map.of(
96+
ThinkingBlock.METADATA_REASONING_DETAILS,
97+
"[{\"type\":\"text\"}]"))
9698
.build();
9799

98100
assertTrue(block.getMetadata().containsKey(ThinkingBlock.METADATA_REASONING_DETAILS));
99101
}
100102
}
101-

0 commit comments

Comments
 (0)