Skip to content

Commit 0cd5999

Browse files
authored
Merge pull request #17786 from serathius/robustness-watch-validation-tests
Add tests to watch validation
2 parents a7f5d4b + a95a307 commit 0cd5999

File tree

3 files changed

+1325
-161
lines changed

3 files changed

+1325
-161
lines changed

tests/robustness/validate/validate.go

+8-2
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,16 @@ func ValidateAndReturnVisualize(t *testing.T, lg *zap.Logger, cfg Config, report
4444
eventHistory, err := mergeWatchEventHistory(reports)
4545
if err != nil {
4646
t.Errorf("Failed merging watch history to create event history, err: %s", err)
47-
validateWatch(t, lg, cfg, reports, nil)
47+
err = validateWatch(lg, cfg, reports, nil)
48+
if err != nil {
49+
t.Errorf("Failed validating watch history, err: %s", err)
50+
}
4851
return visualize
4952
}
50-
validateWatch(t, lg, cfg, reports, eventHistory)
53+
err = validateWatch(lg, cfg, reports, eventHistory)
54+
if err != nil {
55+
t.Errorf("Failed validating watch history, err: %s", err)
56+
}
5157
validateSerializableOperations(t, lg, patchedOperations, eventHistory)
5258
return visualize
5359
}

0 commit comments

Comments
 (0)