test_runner: make skip/todo/expectFailure use JS truthiness (#61815)

Author: VaishnavIUpadyaya

lib/internal/test_runner/test.js

@@ -679,7 +679,7 @@ class Test extends AsyncResource {
     this.expectedAssertions = plan;
     this.cancelled = false;
     this.expectFailure = parseExpectFailure(expectFailure) || this.parent?.expectFailure;
-    this.skipped = skip !== undefined && skip !== false;
+    this.skipped = Boolean(skip);
     this.isTodo = (todo !== undefined && todo !== false) || this.parent?.isTodo;
     this.startTime = null;
     this.endTime = null;

test/parallel/test-runner-skip-empty-string.js

@@ -0,0 +1,13 @@
+'use strict';
+const { test } = require('node:test');
+const assert = require('assert');
+
+let ran = false;
+
+test('skip option empty string should not skip', { skip: '' }, () => {
+  ran = true;
+});
+
+process.on('exit', () => {
+  assert.ok(ran);
+});