Upstream/SongodaCore
1
0
Fork 0
mirror of https://github.com/songoda/SongodaCore.git synced 2025-02-03 05:01:27 +01:00
Code Issues Projects Releases Wiki Activity

Make unit tests in LocaleFileManagerTest deterministic

Browse Source 
The order in which files inside of a given directory are listed is not guaranteed in any way.
This causes tests to work on my machine but fail on out GitHub Actions CI/CD pipeline.
This commit is contained in:
Christian Koop 2022-11-26 15:31:43 +01:00
parent 6c6c3e950c
commit 67a69e34e8
No known key found for this signature in database
GPG Key ID: 89A8181384E010A3
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Core/src/test/java/com/songoda/core/locale/LocaleFileManagerTest.java
View File

@ -14,6 +14,8 @@ import java.io.IOException;
import java.nio.charset.StandardCharsets; import java.nio.charset.StandardCharsets;
import java.nio.file.Files; import java.nio.file.Files;
import java.nio.file.Path; import java.nio.file.Path;
import java.util.Arrays;
import java.util.Collections;
import java.util.List; import java.util.List;
import java.util.stream.Stream; import java.util.stream.Stream;
@ -55,7 +57,8 @@ class LocaleFileManagerTest {
String[] localeFiles = plugin.getDataFolder().list(); String[] localeFiles = plugin.getDataFolder().list();
Assertions.assertNotNull(localeFiles); Assertions.assertNotNull(localeFiles);
Assertions.assertArrayEquals(new String[] {"en.yml", "en_US.lang", "de.txt"}, localeFiles); Arrays.sort(localeFiles);
Assertions.assertArrayEquals(new String[] {"de.txt", "en.yml", "en_US.lang"}, localeFiles);
Assertions.assertSame(4, httpClient.callsOnGet.size()); Assertions.assertSame(4, httpClient.callsOnGet.size());
Assertions.assertTrue(httpClient.callsOnGet.get(0).contains("/test/_index.txt")); Assertions.assertTrue(httpClient.callsOnGet.get(0).contains("/test/_index.txt"));
@ -85,7 +88,8 @@ class LocaleFileManagerTest {
String[] localeFiles = plugin.getDataFolder().list(); String[] localeFiles = plugin.getDataFolder().list();
Assertions.assertNotNull(localeFiles); Assertions.assertNotNull(localeFiles);
Assertions.assertArrayEquals(new String[] {"en.yml", "en_US.lang", "fr.lang", "de.txt"}, localeFiles); Arrays.sort(localeFiles);
Assertions.assertArrayEquals(new String[] {"de.txt", "en.yml", "en_US.lang", "fr.lang"}, localeFiles);
Assertions.assertSame(3, httpClient.callsOnGet.size()); Assertions.assertSame(3, httpClient.callsOnGet.size());
Assertions.assertTrue(httpClient.callsOnGet.get(0).contains("/test/_index.txt")); Assertions.assertTrue(httpClient.callsOnGet.get(0).contains("/test/_index.txt"));