From 4248fbbb898297410046bbbd855b122043c56492 Mon Sep 17 00:00:00 2001 From: Florian Schmaus Date: Wed, 29 Jun 2016 08:08:13 +0200 Subject: [PATCH] Use lowercase chars in integration testRunId --- .../smack/inttest/SmackIntegrationTestFramework.java | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/SmackIntegrationTestFramework.java b/smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/SmackIntegrationTestFramework.java index b63da871a..b112c5108 100644 --- a/smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/SmackIntegrationTestFramework.java +++ b/smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/SmackIntegrationTestFramework.java @@ -35,6 +35,7 @@ import java.util.HashSet; import java.util.Iterator; import java.util.LinkedList; import java.util.List; +import java.util.Locale; import java.util.Map; import java.util.Map.Entry; import java.util.Set; @@ -584,7 +585,14 @@ public class SmackIntegrationTestFramework { } public static final class TestRunResult { - public final String testRunId = StringUtils.insecureRandomString(5); + + /** + * A short String of lowercase characters and numbers used to identify a integration test + * run. We use lowercase characters because this string will eventually be part of the + * localpart of the used JIDs (and the localpart is case insensitive). + */ + public final String testRunId = StringUtils.insecureRandomString(5).toLowerCase(Locale.US); + private final List successfulTests = Collections.synchronizedList(new LinkedList()); private final List failedIntegrationTests = Collections.synchronizedList(new LinkedList()); private final List impossibleTestMethods = Collections.synchronizedList(new LinkedList());