Skip to content

Commit

Permalink
fixup! fix(test): exit code of lime test -- add proof of current beha…
Browse files Browse the repository at this point in the history
…viour
  • Loading branch information
alquerci committed Apr 10, 2024
1 parent 67168ce commit 77e027d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/unit/vendor/lime/lime_harnessTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ private function makeHarness(): lime_harness
return $harness;
}

private function assertHarnessWithExitStatusAndOutput(lime_harness $harness, $expectedOverallSucceed, string $expectedOutput)
private function assertHarnessWithOverallSucceedAndOutput(lime_harness $harness, $expectedOverallSucceed, string $expectedOutput)
{
ob_start();
$allTestsSucceed = $harness->run();
Expand All @@ -41,7 +41,7 @@ public function testOnlyOneTestFile(): void
__DIR__."/fixtures/$name.php",
]);

$this->assertHarnessWithExitStatusAndOutput($harness, $expectedOverallSucceed, $expectedOutput);
$this->assertHarnessWithOverallSucceedAndOutput($harness, $expectedOverallSucceed, $expectedOutput);
}
}

Expand Down Expand Up @@ -208,7 +208,7 @@ public function test_registerFilesWithGlob_thenRunThemAll(): void

$harness->register_glob(__DIR__."/fixtures/*.php");

$this->assertHarnessWithExitStatusAndOutput($harness, false, <<<'EOF'
$this->assertHarnessWithOverallSucceedAndOutput($harness, false, <<<'EOF'
test/unit/vendor/lime/fixtures/failed................................not ok
Failed tests: 1
test/unit/vendor/lime/fixtures/failed_with_plan_less_than_total......dubious
Expand Down

0 comments on commit 77e027d

Please sign in to comment.