From c9a02b964de2bb52b75a1e015d7e1847b43bc398 Mon Sep 17 00:00:00 2001 From: Nuno Maduro Date: Thu, 29 Dec 2022 09:34:53 +0000 Subject: [PATCH] fix: `--retry` not running all tests after passing --- src/Factories/TestCaseMethodFactory.php | 4 +++- src/Repositories/TempRepository.php | 8 ++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/src/Factories/TestCaseMethodFactory.php b/src/Factories/TestCaseMethodFactory.php index 59b6c6c7..3620b710 100644 --- a/src/Factories/TestCaseMethodFactory.php +++ b/src/Factories/TestCaseMethodFactory.php @@ -123,7 +123,9 @@ final class TestCaseMethodFactory $methodName = Str::evaluable($this->description); - if (Retry::$retrying && ! TestSuite::getInstance()->retryTempRepository->exists(sprintf('%s::%s', $classFQN, $methodName))) { + $retryRepository = TestSuite::getInstance()->retryTempRepository; + + if (Retry::$retrying && ! $retryRepository->isEmpty() && ! $retryRepository->exists(sprintf('%s::%s', $classFQN, $methodName))) { return ''; } diff --git a/src/Repositories/TempRepository.php b/src/Repositories/TempRepository.php index 9ad1b601..24118551 100644 --- a/src/Repositories/TempRepository.php +++ b/src/Repositories/TempRepository.php @@ -37,6 +37,14 @@ final class TempRepository $this->save([]); } + /** + * Checks if there is any element. + */ + public function isEmpty(): bool + { + return $this->all() === []; + } + /** * Checks if the given element exists. */