Skip to content

Commit 8fd5f09

Browse files
committed
Update comments
1 parent beb3ec5 commit 8fd5f09

File tree

2 files changed

+0
-5
lines changed

2 files changed

+0
-5
lines changed

tools/flakeguard/reports/data_test.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -386,8 +386,6 @@ func TestAggregateOutputs(t *testing.T) {
386386

387387
result := aggregatedReport.Results[0]
388388

389-
// Expected outputs after aggregation:
390-
// The aggregator should have merged both test runs' outputs under the same run key ("run1").
391389
expectedOutputs := map[string][]string{
392390
"run1": {
393391
"Output from report1 test run",

tools/flakeguard/runner/runner_test.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -590,9 +590,6 @@ func TestOmitOutputsOnSuccess(t *testing.T) {
590590
}
591591
}
592592
require.NotNil(t, testPassResult, "expected 'TestPass' result not found in report")
593-
594-
// Since TestPass always succeeds and OmitOutputsOnSuccess is true,
595-
// we expect no PassedOutputs or general outputs.
596593
require.Empty(t, testPassResult.PassedOutputs, "expected no passed outputs due to OmitOutputsOnSuccess")
597594
require.Empty(t, testPassResult.Outputs, "expected no captured outputs due to OmitOutputsOnSuccess and a successful test")
598595
}

0 commit comments

Comments
 (0)