From 971dadda46b5e3b2545cc32cc410f88786762043 Mon Sep 17 00:00:00 2001 From: Eric <5089238+emizzle@users.noreply.github.com> Date: Tue, 28 Jan 2025 20:49:12 +1100 Subject: [PATCH] fix exit status --- tests/integration/testmanager.nim | 6 +++--- tests/testIntegration.nim | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/integration/testmanager.nim b/tests/integration/testmanager.nim index 08822523..bf02e81c 100644 --- a/tests/integration/testmanager.nim +++ b/tests/integration/testmanager.nim @@ -157,15 +157,15 @@ proc duration(manager: TestManager): Duration = let now = Moment.now() (manager.timeEnd |? now) - (manager.timeStart |? now) -proc testsStatus*(manager: TestManager): ?!bool = +proc allTestsPassed*(manager: TestManager): ?!bool = for test in manager.tests: if test.status in {IntegrationTestStatus.New, IntegrationTestStatus.Running}: return failure "Integration tests not complete" if test.status != IntegrationTestStatus.Ok: - return success true + return success false - return success false + return success true proc duration(test: IntegrationTest): Duration = let now = Moment.now() diff --git a/tests/testIntegration.nim b/tests/testIntegration.nim index bc076f3a..9ece9542 100644 --- a/tests/testIntegration.nim +++ b/tests/testIntegration.nim @@ -72,7 +72,7 @@ proc run() {.async.} = trace "stopping test manager" await manager.stop() - without wasSuccessful =? manager.testsStatus, error: + without wasSuccessful =? manager.allTestsPassed, error: raiseAssert "Failed to get test status: " & error.msg if not wasSuccessful: