Skip to content

Commit 25e8cc7

Browse files
committed
add assertions for all dii_types in unit tests
1 parent 879ab08 commit 25e8cc7

1 file changed

Lines changed: 21 additions & 30 deletions

File tree

src/test/java/com/uid2/operator/UIDOperatorVerticleTest.java

Lines changed: 21 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -2905,6 +2905,15 @@ void identityMapBatchRequestTooLargeForPhone(Vertx vertx, VertxTestContext testC
29052905
send(vertx, "v2/identity/map", req, 413, json -> testContext.completeNow());
29062906
}
29072907

2908+
private double getIdentityMapInputAmount(String path, String diiType) {
2909+
DistributionSummary ds = registry
2910+
.find("uid2_operator_identity_map_inputs")
2911+
.tag("path", path)
2912+
.tag("dii_type", diiType)
2913+
.summary();
2914+
return ds == null ? 0.0 : ds.totalAmount();
2915+
}
2916+
29082917
@Test
29092918
void identityMapV2InputMetricTaggedWithEmailDiiType(Vertx vertx, VertxTestContext testContext) {
29102919
fakeAuth(201, Role.MAPPER);
@@ -2915,13 +2924,10 @@ void identityMapV2InputMetricTaggedWithEmailDiiType(Vertx vertx, VertxTestContex
29152924
.put("email", new JsonArray().add("test1@uid2.com").add("test2@uid2.com"));
29162925

29172926
send(vertx, "v2/identity/map", req, 200, json -> {
2918-
DistributionSummary summary = Metrics.globalRegistry
2919-
.get("uid2_operator_identity_map_inputs")
2920-
.tag("path", "/v2/identity/map")
2921-
.tag("dii_type", "email")
2922-
.summary();
2923-
assertNotNull(summary);
2924-
assertEquals(2.0, summary.totalAmount());
2927+
assertEquals(2.0, getIdentityMapInputAmount("/v2/identity/map", "email"));
2928+
assertEquals(0.0, getIdentityMapInputAmount("/v2/identity/map", "email_hash"));
2929+
assertEquals(0.0, getIdentityMapInputAmount("/v2/identity/map", "phone"));
2930+
assertEquals(0.0, getIdentityMapInputAmount("/v2/identity/map", "phone_hash"));
29252931
testContext.completeNow();
29262932
});
29272933
}
@@ -2937,13 +2943,10 @@ void identityMapV2InputMetricTaggedWithEmailHashDiiType(Vertx vertx, VertxTestCo
29372943
.add(TokenUtils.getIdentityHashString("test1@uid2.com")));
29382944

29392945
send(vertx, "v2/identity/map", req, 200, json -> {
2940-
DistributionSummary summary = Metrics.globalRegistry
2941-
.get("uid2_operator_identity_map_inputs")
2942-
.tag("path", "/v2/identity/map")
2943-
.tag("dii_type", "email_hash")
2944-
.summary();
2945-
assertNotNull(summary);
2946-
assertEquals(1.0, summary.totalAmount());
2946+
assertEquals(0.0, getIdentityMapInputAmount("/v2/identity/map", "email"));
2947+
assertEquals(1.0, getIdentityMapInputAmount("/v2/identity/map", "email_hash"));
2948+
assertEquals(0.0, getIdentityMapInputAmount("/v2/identity/map", "phone"));
2949+
assertEquals(0.0, getIdentityMapInputAmount("/v2/identity/map", "phone_hash"));
29472950
testContext.completeNow();
29482951
});
29492952
}
@@ -2960,22 +2963,10 @@ void identityMapV3InputMetricTaggedWithPathAndDiiType(Vertx vertx, VertxTestCont
29602963
.add(TokenUtils.getIdentityHashString("+15555555555")));
29612964

29622965
send(vertx, "v3/identity/map", req, 200, json -> {
2963-
DistributionSummary emailSummary = Metrics.globalRegistry
2964-
.get("uid2_operator_identity_map_inputs")
2965-
.tag("path", "/v3/identity/map")
2966-
.tag("dii_type", "email")
2967-
.summary();
2968-
assertNotNull(emailSummary);
2969-
assertEquals(2.0, emailSummary.totalAmount());
2970-
2971-
DistributionSummary phoneHashSummary = Metrics.globalRegistry
2972-
.get("uid2_operator_identity_map_inputs")
2973-
.tag("path", "/v3/identity/map")
2974-
.tag("dii_type", "phone_hash")
2975-
.summary();
2976-
assertNotNull(phoneHashSummary);
2977-
assertEquals(1.0, phoneHashSummary.totalAmount());
2978-
2966+
assertEquals(2.0, getIdentityMapInputAmount("/v3/identity/map", "email"));
2967+
assertEquals(0.0, getIdentityMapInputAmount("/v3/identity/map", "email_hash"));
2968+
assertEquals(0.0, getIdentityMapInputAmount("/v3/identity/map", "phone"));
2969+
assertEquals(1.0, getIdentityMapInputAmount("/v3/identity/map", "phone_hash"));
29792970
testContext.completeNow();
29802971
});
29812972
}

0 commit comments

Comments
 (0)