Previously this was limping on because calling t.Fatal in a goroutine doesn't kill the whole test. This change propagates the error so that it poisons the error group, causing the test to exit properly. This means that nil checkpoints can't live on inside the slice of collected checkpoints.
Previously this was limping on because calling t.Fatal in a goroutine doesn't kill the whole test. This change propagates the error so that it poisons the error group, causing the test to exit properly. This means that nil checkpoints can't live on inside the slice of collected checkpoints.