From 389ce830d9f8cb28f07ca620b88cf336998fa652 Mon Sep 17 00:00:00 2001
From: Eric <5089238+emizzle@users.noreply.github.com>
Date: Wed, 19 Mar 2025 15:15:18 +1100
Subject: [PATCH] Remove noCancel from captureProcessOutput call

---
 tests/integration/testmanager.nim | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/integration/testmanager.nim b/tests/integration/testmanager.nim
index f2711b62..156dcc05 100644
--- a/tests/integration/testmanager.nim
+++ b/tests/integration/testmanager.nim
@@ -558,7 +558,7 @@ proc start(test: IntegrationTest) {.async: (raises: []).} =
       test.status = IntegrationTestStatus.Error
       return
 
-    var futCaptureOutput: Future[(seq[string], seq[string])].Raising([])
+    var futCaptureOutput: Future[(seq[string], seq[string])].Raising([CancelledError])
     defer:
       # called at the end of successful runs but also when `start` is cancelled
       # (from `untilTimeout`) due to a timeout. This defer runs first before
@@ -568,7 +568,7 @@ proc start(test: IntegrationTest) {.async: (raises: []).} =
 
     var output = TestOutput.new()
     test.output = success(output)
-    futCaptureOutput = noCancel test.captureProcessOutput()
+    futCaptureOutput = test.captureProcessOutput()
 
     output.exitCode =
       try: