Skip to content

Commit 563ac87

Browse files
claudedeviantintegral
authored andcommitted
test: remove redundant assertContains assertions in testProgressAdvanceIsCalledForEachEntry
The assertContains('0') and assertContains('11') assertions are redundant because the assertGreaterThan(2, count($progressStates)) already verifies that progress is being shown. The 11/11 final state is also verified by testProgressFinishShowsCompletion.
1 parent 05a384e commit 563ac87

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

tests/src/Functional/SplitCommandTest.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -341,10 +341,6 @@ public function testProgressAdvanceIsCalledForEachEntry(): void
341341
$this->assertGreaterThan(2, \count($progressStates),
342342
'Progress bar should show multiple intermediate states when progressAdvance() is called. '.
343343
'Found states: '.implode(', ', $progressStates));
344-
345-
// Verify we start at 0 and end at 11
346-
$this->assertContains('0', $progressStates, 'Progress should start at 0');
347-
$this->assertContains('11', $progressStates, 'Progress should end at 11');
348344
}
349345

350346
public function testProgressFinishShowsCompletion(): void

0 commit comments

Comments
 (0)