Move log.Fatalf call to end so that we log all unused goldens before failing the program

This commit is contained in:
David Dworken 2023-12-29 01:16:33 -08:00
parent db19d8ebf1
commit 20fce0ef02
No known key found for this signature in database

View File

@ -61,6 +61,7 @@ func checkGoldensUsed() {
}
// And check for mismatches
var unusedGoldenErr error = nil
for _, f := range files {
goldenName := path.Base(f.Name())
if !slices.Contains(usedGoldens, goldenName) {
@ -73,11 +74,13 @@ func checkGoldensUsed() {
// It is for another OS
continue
}
err = fmt.Errorf("golden file %v was never used", goldenName)
fmt.Println(err)
log.Fatalf("%v", err)
unusedGoldenErr = fmt.Errorf("golden file %v was never used", goldenName)
fmt.Println(unusedGoldenErr)
}
}
if unusedGoldenErr != nil {
log.Fatalf("%v", unusedGoldenErr)
}
// And print out anything that is in UNUSED_GOLDENS that was actually used, so we
// can manually trim UNUSED_GOLDENS