Skip to content

Commit aeba66d

Browse files
committed
Remove invalid unittest
Signed-off-by: Stuart Loxton <[email protected]>
1 parent 16e8cae commit aeba66d

File tree

4 files changed

+9
-7
lines changed

4 files changed

+9
-7
lines changed

models/spring-ai-bedrock-converse/src/main/java/org/springframework/ai/bedrock/converse/BedrockChatOptions.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,9 @@ public void setMaxTokens(Integer maxTokens) {
130130
this.maxTokens = maxTokens;
131131
}
132132

133-
public Map<String, String> getRequestParameters() { return this.requestParameters; }
133+
public Map<String, String> getRequestParameters() {
134+
return this.requestParameters;
135+
}
134136

135137
public void setRequestParameters(Map<String, String> requestParameters) {
136138
this.requestParameters = requestParameters;
@@ -262,9 +264,8 @@ public boolean equals(Object o) {
262264
@Override
263265
public int hashCode() {
264266
return Objects.hash(this.model, this.frequencyPenalty, this.maxTokens, this.presencePenalty,
265-
this.requestParameters, this.stopSequences, this.temperature, this.topK, this.topP,
266-
this.toolCallbacks, this.toolNames, this.toolContext,
267-
this.internalToolExecutionEnabled);
267+
this.requestParameters, this.stopSequences, this.temperature, this.topK, this.topP, this.toolCallbacks,
268+
this.toolNames, this.toolContext, this.internalToolExecutionEnabled);
268269
}
269270

270271
public static class Builder {

models/spring-ai-bedrock-converse/src/main/java/org/springframework/ai/bedrock/converse/BedrockProxyChatModel.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -425,7 +425,8 @@ else if (message.getMessageType() == MessageType.TOOL) {
425425
Document additionalModelRequestFields = ConverseApiUtils
426426
.getChatOptionsAdditionalModelRequestFields(this.defaultOptions, prompt.getOptions());
427427

428-
Map<String, String> requestMetadata = ConverseApiUtils.getRequestMetadata(prompt.getUserMessage().getMetadata());
428+
Map<String, String> requestMetadata = ConverseApiUtils
429+
.getRequestMetadata(prompt.getUserMessage().getMetadata());
429430

430431
return ConverseRequest.builder()
431432
.modelId(updatedRuntimeOptions.getModel())

models/spring-ai-bedrock-converse/src/test/java/org/springframework/ai/bedrock/converse/BedrockChatOptionsTests.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ void testBuilderWithAllFields() {
4747
assertThat(options)
4848
.extracting("model", "frequencyPenalty", "maxTokens", "presencePenalty", "requestParameters",
4949
"stopSequences", "temperature", "topP", "topK")
50-
.containsExactly("test-model", 0.0, 100, 0.0, Map.of("requestId", "1234"), List.of("stop1", "stop2"), 0.7, 0.8, 50);
50+
.containsExactly("test-model", 0.0, 100, 0.0, Map.of("requestId", "1234"), List.of("stop1", "stop2"), 0.7,
51+
0.8, 50);
5152
}
5253

5354
@Test

models/spring-ai-bedrock-converse/src/test/java/org/springframework/ai/bedrock/converse/BedrockConverseChatClientIT.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,6 @@ void call() {
8080
logger.info("" + response);
8181
assertThat(response.getResults()).hasSize(1);
8282
assertThat(response.getResults().get(0).getOutput().getText()).contains("Blackbeard");
83-
assertThat(response.getResults().get(0).getMetadata().containsKey("requestId"));
8483
}
8584

8685
@Test

0 commit comments

Comments
 (0)