diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-12-03 17:22:00 +0100 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-12-03 17:22:00 +0100 |
commit | 0769d4e04be5f9032f8699cdfa8b6c208645d3a3 (patch) | |
tree | 62b648297483cf7c27e3a5940cd0e2b1be94bf21 /configserver/src | |
parent | f5682f71b81c142acbb409ce4dbd3eeaeec30ffd (diff) |
Avoid sorting an unmodifiable list
Diffstat (limited to 'configserver/src')
-rw-r--r-- | configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java index 74ec5e81da3..2e60a6c6cd0 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java @@ -295,10 +295,8 @@ public class ApplicationRepositoryTest { // deployApp(new File("src/test/apps/app"), prepareParams); List<String> deleted = applicationRepository.deleteUnusedFileDistributionReferences(fileDirectory, keepFileReferencesDuration); - Collections.sort(deleted); - List<String> expected = new ArrayList<>(List.of("bar", "baz0", "baz1")); - Collections.sort(expected); - assertEquals(expected, deleted); + List<String> expected = List.of("bar", "baz0", "baz1"); + assertEquals(expected.stream().sorted().toList(), deleted.stream().sorted().toList()); // bar, baz0 and baz1 will be deleted and foo is not old enough to be considered assertFalse(filereferenceDirOldest.exists()); assertFalse(new File(fileReferencesDir, "baz0").exists()); |