dont use NaN

Author: rluvaton

lib/internal/test_runner/test.js

@@ -656,7 +656,7 @@ class Test extends AsyncResource {
       this.parent.processPendingSubtests();
     } else if (!this.reported) {
       if (!this.passed && failed === 0 && this.error) {
-        this.reporter.fail(0, kFilename, NaN, kFilename, {
+        this.reporter.fail(0, kFilename, this.subtests.length + 1, kFilename, {
           __proto__: null,
           duration_ms: this.#duration(),
           error: this.error,

test/fixtures/test-runner/output/global_after_should_fail_the_test.snapshot

@@ -5,7 +5,7 @@ ok 1 - this is a test
   ---
   duration_ms: *
   ...
-not ok NaN - /Users/rluvaton/dev/open-source/node/node-fork/test/fixtures/test-runner/output/global_after_should_fail_the_test.js
+not ok 2 - /Users/rluvaton/dev/open-source/node/node-fork/test/fixtures/test-runner/output/global_after_should_fail_the_test.js
   ---
   duration_ms: *
   failureType: 'hookFailed'