From b17f41c3e826d4055020cf3d5c19b2957bcaddde Mon Sep 17 00:00:00 2001 From: gram-signal <84339875+gram-signal@users.noreply.github.com> Date: Mon, 29 Aug 2022 15:51:37 -0600 Subject: [PATCH] Check if dashes work in dynamic configuration keys. --- .../configuration/dynamic/DynamicConfigurationTest.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/service/src/test/java/org/whispersystems/textsecuregcm/configuration/dynamic/DynamicConfigurationTest.java b/service/src/test/java/org/whispersystems/textsecuregcm/configuration/dynamic/DynamicConfigurationTest.java index c0a01f198..3cdb8154d 100644 --- a/service/src/test/java/org/whispersystems/textsecuregcm/configuration/dynamic/DynamicConfigurationTest.java +++ b/service/src/test/java/org/whispersystems/textsecuregcm/configuration/dynamic/DynamicConfigurationTest.java @@ -54,6 +54,9 @@ class DynamicConfigurationTest { uuidsOnly: enrolledUuids: - 71618739-114c-4b1f-bb0d-6478a44eb600 + uuids-with-dash: + enrolledUuids: + - 71618739-114c-4b1f-bb0d-6478ffffffff """); final DynamicConfiguration config = @@ -77,6 +80,11 @@ class DynamicConfigurationTest { assertEquals(0, config.getExperimentEnrollmentConfiguration("uuidsOnly").get().getEnrollmentPercentage()); assertEquals(Set.of(UUID.fromString("71618739-114c-4b1f-bb0d-6478a44eb600")), config.getExperimentEnrollmentConfiguration("uuidsOnly").get().getEnrolledUuids()); + + assertTrue(config.getExperimentEnrollmentConfiguration("uuids-with-dash").isPresent()); + assertEquals(0, config.getExperimentEnrollmentConfiguration("uuids-with-dash").get().getEnrollmentPercentage()); + assertEquals(Set.of(UUID.fromString("71618739-114c-4b1f-bb0d-6478ffffffff")), + config.getExperimentEnrollmentConfiguration("uuids-with-dash").get().getEnrolledUuids()); } }