tools: enable linter in test/fixtures/errors

aduh95 · aduh95 · commit 80d58c372dda · 2025-05-19T12:13:50.000+02:00
PR-URL: #57701 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -60,6 +60,7 @@ export default [
     // We want to lint only a few specific fixtures folders
     'test/fixtures/*',
     '!test/fixtures/console',
+    '!test/fixtures/errors',
     '!test/fixtures/eval',
     '!test/fixtures/test-runner',
     'test/fixtures/test-runner/*',
diff --git a/test/fixtures/errors/force_colors.js b/test/fixtures/errors/force_colors.js
@@ -1 +1,2 @@
-throw new Error('Should include grayed stack trace')
+'use strict';
+throw new Error('Should include grayed stack trace');
diff --git a/test/fixtures/errors/force_colors.snapshot b/test/fixtures/errors/force_colors.snapshot
@@ -1,9 +1,9 @@
-*force_colors.js:1
-throw new Error('Should include grayed stack trace')
+*force_colors.js:2
+throw new Error('Should include grayed stack trace');
 ^
 
 Error: Should include grayed stack trace
-    at Object.<anonymous> [90m(/[39mtest*force_colors.js:1:7[90m)[39m
+    at Object.<anonymous> [90m(/[39mtest*force_colors.js:2:7[90m)[39m
 [90m    at *[39m
 [90m    at *[39m
 [90m    at *[39m
diff --git a/test/fixtures/errors/throw_in_eval_anonymous.js b/test/fixtures/errors/throw_in_eval_anonymous.js
@@ -6,4 +6,4 @@ eval(`
 
   throw new Error('error in anonymous script');
 
-`)
+`);
diff --git a/test/fixtures/errors/throw_in_eval_named.js b/test/fixtures/errors/throw_in_eval_named.js
diff --git a/test/fixtures/errors/throw_in_line_with_tabs.js b/test/fixtures/errors/throw_in_line_with_tabs.js
@@ -19,7 +19,7 @@
 // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
 // USE OR OTHER DEALINGS IN THE SOFTWARE.
 
-/* eslint-disable indent, no-tabs */
+/* eslint-disable @stylistic/js/indent, @stylistic/js/no-tabs */
 'use strict';
 require('../../common');
 

Original file line number	Diff line number	Diff line change
`@@ -1 +1,2 @@`
`1`		`-throw new Error('Should include grayed stack trace')`
	`1`	`+'use strict';`
	`2`	`+throw new Error('Should include grayed stack trace');`
-Original file line number
+Diff line change
@@ @@ -1,9 +1,9 @@ @@
 -*force_colors.js:1
 -throw new Error('Should include grayed stack trace')
 +*force_colors.js:2
 +throw new Error('Should include grayed stack trace');
+^
 Error: Should include grayed stack trace
 -    at Object.<anonymous> [90m(/[39mtest*force_colors.js:1:7[90m)[39m
 +    at Object.<anonymous> [90m(/[39mtest*force_colors.js:2:7[90m)[39m
 [90m    at *[39m
 [90m    at *[39m
 [90m    at *[39m
Original file line number	Diff line number	Diff line change
@@ -6,4 +6,4 @@ eval(`
`6`	`6`
`7`	`7`	`throw new Error('error in anonymous script');`
`8`	`8`
`9`		-`)
	`9`	+`);