internal/dnfjson: update test data for regression test

update test data so that TestMultiDirCacheRead fails before the fix and succeeds after Golang's os.ReadDir() returns entries in alphabetical order and since for our test data, the alphabetical order of the entries matches the mtime-based sorting, so tests pass even with no sorting at all. This change breaks the alphabetical order of data.
osbuild · Nov 20, 2023 · 1838a06 · 1838a06
1 parent ad57cc4
commit 1838a06
Showing 1 changed file with 15 additions and 15 deletions.
diff --git a/internal/dnfjson/cache_test.go b/internal/dnfjson/cache_test.go
@@ -103,24 +103,24 @@ var testCfgs = map[string]testCache{
 		"df2665154150abf76f4d86156228a75c39f3f31a79d4a861d76b1edd89814b62-98177081b9162766/repodata/repomd.xml":                                                                                      fileInfo{12817, 300},
 	},
 	"fake-real": { // fake but resembling real data
-		"1111111111111111111111111111111111111111111111111111111111111111.solv":           fileInfo{100, 0},
-		"1111111111111111111111111111111111111111111111111111111111111111-filenames.solv": fileInfo{200, 0},
-		"1111111111111111111111111111111111111111111111111111111111111111.whatever":       fileInfo{110, 0},
-		"1111111111111111111111111111111111111111111111111111111111111111/repodata/a":     fileInfo{1000, 0},
-		"1111111111111111111111111111111111111111111111111111111111111111/repodata/b":     fileInfo{3829, 0},
-		"1111111111111111111111111111111111111111111111111111111111111111/repodata/c":     fileInfo{831989, 0},
+		"3333333333333333333333333333333333333333333333333333333333333333.solv":           fileInfo{100, 0},
+		"3333333333333333333333333333333333333333333333333333333333333333-filenames.solv": fileInfo{200, 0},
+		"3333333333333333333333333333333333333333333333333333333333333333.whatever":       fileInfo{110, 0},
+		"3333333333333333333333333333333333333333333333333333333333333333/repodata/a":     fileInfo{1000, 0},
+		"3333333333333333333333333333333333333333333333333333333333333333/repodata/b":     fileInfo{3829, 0},
+		"3333333333333333333333333333333333333333333333333333333333333333/repodata/c":     fileInfo{831989, 0},
 		"2222222222222222222222222222222222222222222222222222222222222222.solv":           fileInfo{120, 2},
 		"2222222222222222222222222222222222222222222222222222222222222222-filenames.solv": fileInfo{232, 2},
 		"2222222222222222222222222222222222222222222222222222222222222222.whatever":       fileInfo{110, 2},
 		"2222222222222222222222222222222222222222222222222222222222222222/repodata/a":     fileInfo{1000, 2},
 		"2222222222222222222222222222222222222222222222222222222222222222/repodata/b":     fileInfo{3829, 2},
 		"2222222222222222222222222222222222222222222222222222222222222222/repodata/c":     fileInfo{831989, 2},
-		"3333333333333333333333333333333333333333333333333333333333333333.solv":           fileInfo{105, 4},
-		"3333333333333333333333333333333333333333333333333333333333333333-filenames.solv": fileInfo{200, 4},
-		"3333333333333333333333333333333333333333333333333333333333333333.whatever":       fileInfo{110, 4},
-		"3333333333333333333333333333333333333333333333333333333333333333/repodata/a":     fileInfo{2390, 4},
-		"3333333333333333333333333333333333333333333333333333333333333333/repodata/b":     fileInfo{1234890, 4},
-		"3333333333333333333333333333333333333333333333333333333333333333/repodata/c":     fileInfo{483, 4},
+		"1111111111111111111111111111111111111111111111111111111111111111.solv":           fileInfo{105, 4},
+		"1111111111111111111111111111111111111111111111111111111111111111-filenames.solv": fileInfo{200, 4},
+		"1111111111111111111111111111111111111111111111111111111111111111.whatever":       fileInfo{110, 4},
+		"1111111111111111111111111111111111111111111111111111111111111111/repodata/a":     fileInfo{2390, 4},
+		"1111111111111111111111111111111111111111111111111111111111111111/repodata/b":     fileInfo{1234890, 4},
+		"1111111111111111111111111111111111111111111111111111111111111111/repodata/c":     fileInfo{483, 4},
 	},
 	"completely-fake": { // just a mess of files (including files without a repo ID)
 		"somefile": fileInfo{192, 10291920},
@@ -223,7 +223,7 @@ func TestCacheCleanup(t *testing.T) {
 	rhelTotalRepoSize := sizeSum(testCfgs["rhel84-aarch64"])
 
 	fakeRealSize2 := sizeSum(testCfgs["fake-real"], "2222222222222222222222222222222222222222222222222222222222222222")
-	fakeRealSize3 := sizeSum(testCfgs["fake-real"], "3333333333333333333333333333333333333333333333333333333333333333")
+	fakeRealSize3 := sizeSum(testCfgs["fake-real"], "1111111111111111111111111111111111111111111111111111111111111111")
 
 	testCases := map[string]testCase{
 		// max size 1 byte -> clean will delete everything
@@ -264,13 +264,13 @@ func TestCacheCleanup(t *testing.T) {
 			cache:              testCfgs["fake-real"],
 			maxSize:            fakeRealSize3 + fakeRealSize2 + 102400,
 			minSizeAfterShrink: fakeRealSize3 + fakeRealSize2,
-			repoIDsAfterShrink: []string{"3333333333333333333333333333333333333333333333333333333333333333", "2222222222222222222222222222222222222222222222222222222222222222"},
+			repoIDsAfterShrink: []string{"1111111111111111111111111111111111111111111111111111111111111111", "2222222222222222222222222222222222222222222222222222222222222222"},
 		},
 		"fake-real-delete-2": {
 			cache:              testCfgs["fake-real"],
 			maxSize:            fakeRealSize3 + 102400,
 			minSizeAfterShrink: fakeRealSize3,
-			repoIDsAfterShrink: []string{"3333333333333333333333333333333333333333333333333333333333333333"},
+			repoIDsAfterShrink: []string{"1111111111111111111111111111111111111111111111111111111111111111"},
 		},
 		// max size is huge -> clean wont delete anything
 		"rhel-no-delete": {