Use TreeSet for toString()
diff --git a/src/main/java/org/apache/accumulo/access/Authorizations.java b/src/main/java/org/apache/accumulo/access/Authorizations.java
index b13d6bb..2695f25 100644
--- a/src/main/java/org/apache/accumulo/access/Authorizations.java
+++ b/src/main/java/org/apache/accumulo/access/Authorizations.java
@@ -20,7 +20,7 @@
import java.util.Collection;
import java.util.Set;
-import java.util.stream.Collectors;
+import java.util.TreeSet;
/**
* An immutable collection of authorization strings.
@@ -64,7 +64,7 @@
*/
@Override
public String toString() {
- return authorizations.stream().sorted().collect(Collectors.joining(","));
+ return new TreeSet<>(authorizations).toString();
}
public static Authorizations of(String... authorizations) {
diff --git a/src/test/java/org/apache/accumulo/access/AccessExpressionTest.java b/src/test/java/org/apache/accumulo/access/AccessExpressionTest.java
index 2980d43..cd9b671 100644
--- a/src/test/java/org/apache/accumulo/access/AccessExpressionTest.java
+++ b/src/test/java/org/apache/accumulo/access/AccessExpressionTest.java
@@ -47,14 +47,14 @@
// is the expected authorization in the expression
var testData = new ArrayList<List<String>>();
- testData.add(List.of("", ""));
- testData.add(List.of("a", "a"));
- testData.add(List.of("(a)", "a"));
- testData.add(List.of("Z|M|A", "A,M,Z"));
- testData.add(List.of("Z&M&A", "A,M,Z"));
- testData.add(List.of("(Y|B|Y)&(Z|A|Z)", "A,B,Y,Z"));
- testData.add(List.of("(Y&B&Y)|(Z&A&Z)", "A,B,Y,Z"));
- testData.add(List.of("(A1|B1)&((A1|B2)&(B2|C1))", "A1,B1,B2,C1"));
+ testData.add(List.of("", "[]"));
+ testData.add(List.of("a", "[a]"));
+ testData.add(List.of("(a)", "[a]"));
+ testData.add(List.of("Z|M|A", "[A, M, Z]"));
+ testData.add(List.of("Z&M&A", "[A, M, Z]"));
+ testData.add(List.of("(Y|B|Y)&(Z|A|Z)", "[A, B, Y, Z]"));
+ testData.add(List.of("(Y&B&Y)|(Z&A&Z)", "[A, B, Y, Z]"));
+ testData.add(List.of("(A1|B1)&((A1|B2)&(B2|C1))", "[A1, B1, B2, C1]"));
for (var testCase : testData) {
assertEquals(2, testCase.size());
