Skip to content

Commit 7864b18

Browse files
committed
Some comments
1 parent c104d6a commit 7864b18

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

instrumentation/aws-sdk/aws-sdk-2.2/library/src/testBedrockRuntime/java/io/opentelemetry/instrumentation/awssdk/v2_2/internal/Aws2BedrockRuntimeTest.java

+2
Original file line numberDiff line numberDiff line change
@@ -252,6 +252,7 @@ void testConverseToolCallNoMessageContent() {
252252
KeyValue.of("id", Value.of(sanFranciscoToolUseId)),
253253
KeyValue.of("type", Value.of("function"))))))));
254254

255+
// Clear to allow asserting telemetry of user request and tool result processing separately.
255256
getTesting().clearData();
256257

257258
messages.add(response0.output().message());
@@ -635,6 +636,7 @@ void testConverseToolCallStreamNoMessageContent()
635636
KeyValue.of("id", Value.of(sanFranciscoToolUseId)),
636637
KeyValue.of("type", Value.of("function"))))))));
637638

639+
// Clear to allow asserting telemetry of user request and tool result processing separately.
638640
getTesting().clearData();
639641

640642
List<ContentBlock> contentBlocks = new ArrayList<>();

instrumentation/aws-sdk/aws-sdk-2.2/testing/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/AbstractAws2BedrockRuntimeTest.java

+2
Original file line numberDiff line numberDiff line change
@@ -518,6 +518,7 @@ void testConverseToolCall() {
518518
Value.of(
519519
"<thinking> The User has asked for the current weather in two locations: Seattle and San Francisco. To provide the requested information, I will use the \"get_current_weather\" tool for each location separately. </thinking>\n")))));
520520

521+
// Clear to allow asserting telemetry of user request and tool result processing separately.
521522
getTesting().clearData();
522523

523524
messages.add(response0.output().message());
@@ -932,6 +933,7 @@ void testConverseToolCallStream() throws InterruptedException, ExecutionExceptio
932933
Value.of(
933934
"<thinking> The User has asked for the current weather in two locations: Seattle and San Francisco. To provide the requested information, I will use the \"get_current_weather\" tool for each location separately. </thinking>\n")))));
934935

936+
// Clear to allow asserting telemetry of user request and tool result processing separately.
935937
getTesting().clearData();
936938

937939
List<ContentBlock> contentBlocks = new ArrayList<>();

0 commit comments

Comments
 (0)