scylladb / argus

Apache License 2.0
4 stars 11 forks source link

fix(stats.py): Correctly collect worst-case status for tests #517

Open k0machi opened 1 day ago

k0machi commented 1 day ago

This commit fixes an issue where worst case collection function would never take the branch to compare statuses, leading to last status always being chosen - for 3 runs of (failed, pass, pass) the outcome would be pass and for (failed, pass, failed) it would be failed. This commit corrects this and now statuses are properly evaluated.

Fixes #488

k0machi commented 1 day ago

Before:

image

After:

image