Fix

fisker · May 10, 2024 · cdc9d63 · cdc9d63
1 parent 437a335
commit cdc9d63
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 9 deletions.
diff --git a/rules/fix/switch-call-expression-to-new-expression.js b/rules/fix/switch-call-expression-to-new-expression.js
@@ -1,8 +1,10 @@
 'use strict';
 const {isParenthesized} = require('../utils/parentheses.js');
 const shouldAddParenthesesToNewExpressionCallee = require('../utils/should-add-parentheses-to-new-expression-callee.js');
+const fixSpaceAroundKeyword = require('./fix-space-around-keywords.js');
 
 function * switchCallExpressionToNewExpression(node, sourceCode, fixer) {
+	yield * fixSpaceAroundKeyword(fixer, node, sourceCode);
 	yield fixer.insertTextBefore(node, 'new ');
 
 	const {callee} = node;

diff --git a/rules/throw-new-error.js b/rules/throw-new-error.js
@@ -1,8 +1,5 @@
 'use strict';
-const {
-	switchCallExpressionToNewExpression,
-	fixSpaceAroundKeyword,
-} = require('./fix/index.js');
+const {switchCallExpressionToNewExpression} = require('./fix/index.js');
 
 const messageId = 'throw-new-error';
 const messages = {
@@ -30,10 +27,7 @@ const create = context => ({
 		return {
 			node,
 			messageId,
-			* fix(fixer) {
-				yield * fixSpaceAroundKeyword(fixer, node, context.sourceCode);
-				yield * switchCallExpressionToNewExpression(node, context.sourceCode, fixer);
-			}
+			fix: fixer => switchCallExpressionToNewExpression(node, context.sourceCode, fixer),
 		};
 	},
 });

diff --git a/test/snapshots/new-for-builtins.mjs.md b/test/snapshots/new-for-builtins.mjs.md
@@ -1376,7 +1376,7 @@ Generated by [AVA](https://avajs.dev).
 
     `␊
       1 | function foo() {␊
-      2 | 	returnnew (globalThis).Map()␊
+      2 | 	return new (globalThis).Map()␊
       3 | }␊
     `
 

diff --git a/test/snapshots/new-for-builtins.mjs.snap b/test/snapshots/new-for-builtins.mjs.snap