test_runner: avoid unnecessary error creation

Author: pmarchini

lib/internal/test_runner/test.js

@@ -773,13 +773,16 @@ class Test extends AsyncResource {
       return;
     }
 
-    const cancelledError = this.root.bailed ? new ERR_TEST_FAILURE(
-      'test bailed out',
-      kTestBailedOut,
-    ) : new ERR_TEST_FAILURE(
-      'test did not finish before its parent and was cancelled',
-      kCancelledByParent,
-    );
+    let cancelledError;
+    if (!error) {
+      cancelledError = this.root.bailed ? new ERR_TEST_FAILURE(
+        'test bailed out',
+        kTestBailedOut,
+      ) : new ERR_TEST_FAILURE(
+        'test did not finish before its parent and was cancelled',
+        kCancelledByParent,
+      );
+    }
 
     this.fail(error || cancelledError);
     this.cancelled = true;
@@ -837,8 +840,6 @@ class Test extends AsyncResource {
 
     if (this.bail && !this.root.bailed) {
       this.root.bailed = true;
-      this.bailed = true;
-      this.endTime ??= hrtime();
       this.root.postRun();
     }
   }