From dd1f1f4e24b31dd61dca57b6caffdd5367908f76 Mon Sep 17 00:00:00 2001 From: Raz Luvaton <16746759+rluvaton@users.noreply.github.com> Date: Thu, 3 Aug 2023 22:41:20 +0300 Subject: [PATCH] test: remove --no-warnings flag in test_runner fixtures no longer needed after #48915 fix PR-URL: https://github.com/nodejs/node/pull/48989 Reviewed-By: Moshe Atlow Reviewed-By: Chemi Atlow --- test/fixtures/test-runner/output/abort.js | 1 - test/fixtures/test-runner/output/abort_hooks.js | 1 - test/fixtures/test-runner/output/abort_suite.js | 1 - test/fixtures/test-runner/output/describe_it.js | 1 - test/fixtures/test-runner/output/describe_nested.js | 1 - test/fixtures/test-runner/output/dot_reporter.js | 1 - test/fixtures/test-runner/output/hooks.js | 1 - test/fixtures/test-runner/output/name_pattern.js | 2 +- test/fixtures/test-runner/output/name_pattern_with_only.js | 2 +- test/fixtures/test-runner/output/no_refs.js | 1 - test/fixtures/test-runner/output/no_tests.js | 1 - test/fixtures/test-runner/output/only_tests.js | 2 +- test/fixtures/test-runner/output/output_cli.js | 1 - test/fixtures/test-runner/output/single.js | 1 - test/fixtures/test-runner/output/spec_reporter.js | 1 - test/fixtures/test-runner/output/spec_reporter_cli.js | 1 - test/fixtures/test-runner/output/spec_reporter_successful.js | 2 +- test/fixtures/test-runner/output/unresolved_promise.js | 1 - 18 files changed, 4 insertions(+), 18 deletions(-) diff --git a/test/fixtures/test-runner/output/abort.js b/test/fixtures/test-runner/output/abort.js index 0cd9c9b9273503..eba48d9ec58718 100644 --- a/test/fixtures/test-runner/output/abort.js +++ b/test/fixtures/test-runner/output/abort.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const test = require('node:test'); diff --git a/test/fixtures/test-runner/output/abort_hooks.js b/test/fixtures/test-runner/output/abort_hooks.js index b0f1da80d62719..8395f70e86185e 100644 --- a/test/fixtures/test-runner/output/abort_hooks.js +++ b/test/fixtures/test-runner/output/abort_hooks.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; const { before, beforeEach, describe, it, after, afterEach } = require('node:test'); diff --git a/test/fixtures/test-runner/output/abort_suite.js b/test/fixtures/test-runner/output/abort_suite.js index 419698320a5f7d..8a2a2c05458fd4 100644 --- a/test/fixtures/test-runner/output/abort_suite.js +++ b/test/fixtures/test-runner/output/abort_suite.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const { describe, it } = require('node:test'); diff --git a/test/fixtures/test-runner/output/describe_it.js b/test/fixtures/test-runner/output/describe_it.js index 6625747d026969..942b1c5317abab 100644 --- a/test/fixtures/test-runner/output/describe_it.js +++ b/test/fixtures/test-runner/output/describe_it.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const assert = require('node:assert'); diff --git a/test/fixtures/test-runner/output/describe_nested.js b/test/fixtures/test-runner/output/describe_nested.js index 40ea150a018f3d..3cd4dcbb06c849 100644 --- a/test/fixtures/test-runner/output/describe_nested.js +++ b/test/fixtures/test-runner/output/describe_nested.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const { describe, it } = require('node:test'); diff --git a/test/fixtures/test-runner/output/dot_reporter.js b/test/fixtures/test-runner/output/dot_reporter.js index 72a8aaa10e491b..e9b8f5cead88f0 100644 --- a/test/fixtures/test-runner/output/dot_reporter.js +++ b/test/fixtures/test-runner/output/dot_reporter.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const fixtures = require('../../../common/fixtures'); diff --git a/test/fixtures/test-runner/output/hooks.js b/test/fixtures/test-runner/output/hooks.js index 827da5d5646262..00a6c23499f52e 100644 --- a/test/fixtures/test-runner/output/hooks.js +++ b/test/fixtures/test-runner/output/hooks.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; const common = require('../../../common'); const assert = require('assert'); diff --git a/test/fixtures/test-runner/output/name_pattern.js b/test/fixtures/test-runner/output/name_pattern.js index f183c09057fa33..10e7619b9cfcb9 100644 --- a/test/fixtures/test-runner/output/name_pattern.js +++ b/test/fixtures/test-runner/output/name_pattern.js @@ -1,4 +1,4 @@ -// Flags: --no-warnings --test-name-pattern=enabled --test-name-pattern=yes --test-name-pattern=/pattern/i +// Flags: --test-name-pattern=enabled --test-name-pattern=yes --test-name-pattern=/pattern/i 'use strict'; const common = require('../../../common'); const { diff --git a/test/fixtures/test-runner/output/name_pattern_with_only.js b/test/fixtures/test-runner/output/name_pattern_with_only.js index a3e2f1be2ad42d..bc68b7a5fdf7e7 100644 --- a/test/fixtures/test-runner/output/name_pattern_with_only.js +++ b/test/fixtures/test-runner/output/name_pattern_with_only.js @@ -1,4 +1,4 @@ -// Flags: --no-warnings --test-only --test-name-pattern=enabled +// Flags: --test-only --test-name-pattern=enabled 'use strict'; const common = require('../../../common'); const { test } = require('node:test'); diff --git a/test/fixtures/test-runner/output/no_refs.js b/test/fixtures/test-runner/output/no_refs.js index 7b36e01133da6e..0d7cc6e01791d0 100644 --- a/test/fixtures/test-runner/output/no_refs.js +++ b/test/fixtures/test-runner/output/no_refs.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const test = require('node:test'); diff --git a/test/fixtures/test-runner/output/no_tests.js b/test/fixtures/test-runner/output/no_tests.js index f9a9506706180e..2644e29fe26f85 100644 --- a/test/fixtures/test-runner/output/no_tests.js +++ b/test/fixtures/test-runner/output/no_tests.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const test = require('node:test'); diff --git a/test/fixtures/test-runner/output/only_tests.js b/test/fixtures/test-runner/output/only_tests.js index 5ac4a90c2cf264..26266b524454b7 100644 --- a/test/fixtures/test-runner/output/only_tests.js +++ b/test/fixtures/test-runner/output/only_tests.js @@ -1,4 +1,4 @@ -// Flags: --no-warnings --test-only +// Flags: --test-only 'use strict'; require('../../../common'); const { test, describe, it } = require('node:test'); diff --git a/test/fixtures/test-runner/output/output_cli.js b/test/fixtures/test-runner/output/output_cli.js index 50ef07233314b1..4c6b029c6580c0 100644 --- a/test/fixtures/test-runner/output/output_cli.js +++ b/test/fixtures/test-runner/output/output_cli.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const fixtures = require('../../../common/fixtures'); diff --git a/test/fixtures/test-runner/output/single.js b/test/fixtures/test-runner/output/single.js index e099ec3c375bb7..568e5ba986ad49 100644 --- a/test/fixtures/test-runner/output/single.js +++ b/test/fixtures/test-runner/output/single.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; const test = require('node:test'); test('last test', () => {}); diff --git a/test/fixtures/test-runner/output/spec_reporter.js b/test/fixtures/test-runner/output/spec_reporter.js index 6a7c2d655f93b3..46e18b1ca8630d 100644 --- a/test/fixtures/test-runner/output/spec_reporter.js +++ b/test/fixtures/test-runner/output/spec_reporter.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const fixtures = require('../../../common/fixtures'); diff --git a/test/fixtures/test-runner/output/spec_reporter_cli.js b/test/fixtures/test-runner/output/spec_reporter_cli.js index e88a7221fb4e71..b0c72e51ab66b8 100644 --- a/test/fixtures/test-runner/output/spec_reporter_cli.js +++ b/test/fixtures/test-runner/output/spec_reporter_cli.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const fixtures = require('../../../common/fixtures'); diff --git a/test/fixtures/test-runner/output/spec_reporter_successful.js b/test/fixtures/test-runner/output/spec_reporter_successful.js index fb9800c0491722..a7ffeb59cc9d1e 100644 --- a/test/fixtures/test-runner/output/spec_reporter_successful.js +++ b/test/fixtures/test-runner/output/spec_reporter_successful.js @@ -1,4 +1,4 @@ -// Flags: --no-warnings --test-reporter=spec +// Flags: --test-reporter=spec 'use strict'; require('../../../common'); const { it } = require('node:test'); diff --git a/test/fixtures/test-runner/output/unresolved_promise.js b/test/fixtures/test-runner/output/unresolved_promise.js index a43f064d44de8e..daf2cee24b6cf9 100644 --- a/test/fixtures/test-runner/output/unresolved_promise.js +++ b/test/fixtures/test-runner/output/unresolved_promise.js @@ -1,4 +1,3 @@ -// Flags: --no-warnings 'use strict'; require('../../../common'); const test = require('node:test');