Skip to content

Commit 4fd74bc

Browse files
committed
add metric assertions to secrets manager tests
1 parent b0b2efe commit 4fd74bc

File tree

1 file changed

+66
-2
lines changed
  • appsignals-tests/contract-tests/src/test/java/software/amazon/opentelemetry/appsignals/test/awssdk/base

1 file changed

+66
-2
lines changed

appsignals-tests/contract-tests/src/test/java/software/amazon/opentelemetry/appsignals/test/awssdk/base/AwsSdkBaseTest.java

Lines changed: 66 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2327,7 +2327,6 @@ protected void doTestSecretsManagerError() throws Exception {
23272327

23282328
var localService = getApplicationOtelServiceName();
23292329
var localOperation = "GET /secretsmanager/error";
2330-
var type = "AWS::SecretsManager::Secret";
23312330

23322331
assertSpanClientAttributes(
23332332
traces,
@@ -2345,6 +2344,39 @@ protected void doTestSecretsManagerError() throws Exception {
23452344
"http://error.test:8080",
23462345
400,
23472346
List.of());
2347+
assertMetricClientAttributes(
2348+
metrics,
2349+
AppSignalsConstants.LATENCY_METRIC,
2350+
localService,
2351+
localOperation,
2352+
getSecretsManagerServiceName(),
2353+
"DescribeSecret",
2354+
null,
2355+
null,
2356+
null,
2357+
5000.0);
2358+
assertMetricClientAttributes(
2359+
metrics,
2360+
AppSignalsConstants.FAULT_METRIC,
2361+
localService,
2362+
localOperation,
2363+
getSecretsManagerServiceName(),
2364+
"DescribeSecret",
2365+
null,
2366+
null,
2367+
null,
2368+
0.0);
2369+
assertMetricClientAttributes(
2370+
metrics,
2371+
AppSignalsConstants.ERROR_METRIC,
2372+
localService,
2373+
localOperation,
2374+
getSecretsManagerServiceName(),
2375+
"DescribeSecret",
2376+
null,
2377+
null,
2378+
null,
2379+
1.0);
23482380
}
23492381

23502382
protected void doTestSecretsManagerFault() throws Exception {
@@ -2359,7 +2391,6 @@ protected void doTestSecretsManagerFault() throws Exception {
23592391

23602392
var localService = getApplicationOtelServiceName();
23612393
var localOperation = "GET /secretsmanager/fault";
2362-
var type = "AWS::SecretsManager::Secret";
23632394

23642395
assertSpanClientAttributes(
23652396
traces,
@@ -2377,6 +2408,39 @@ protected void doTestSecretsManagerFault() throws Exception {
23772408
"http://fault.test:8080",
23782409
500,
23792410
List.of());
2411+
assertMetricClientAttributes(
2412+
metrics,
2413+
AppSignalsConstants.LATENCY_METRIC,
2414+
localService,
2415+
localOperation,
2416+
getSecretsManagerServiceName(),
2417+
"DescribeSecret",
2418+
null,
2419+
null,
2420+
null,
2421+
5000.0);
2422+
assertMetricClientAttributes(
2423+
metrics,
2424+
AppSignalsConstants.FAULT_METRIC,
2425+
localService,
2426+
localOperation,
2427+
getSecretsManagerServiceName(),
2428+
"DescribeSecret",
2429+
null,
2430+
null,
2431+
null,
2432+
1.0);
2433+
assertMetricClientAttributes(
2434+
metrics,
2435+
AppSignalsConstants.ERROR_METRIC,
2436+
localService,
2437+
localOperation,
2438+
getSecretsManagerServiceName(),
2439+
"DescribeSecret",
2440+
null,
2441+
null,
2442+
null,
2443+
0.0);
23802444
}
23812445

23822446
protected void doTestStepFunctionsDescribeStateMachine() throws Exception {

0 commit comments

Comments
 (0)