chore: exclude fixtures from eslint (#682)

* chore: ignore fixtures for eslint * chore: run prettier over fixtures Co-authored-by: Netlify Team Account 1 <netlify-team-account-1@users.noreply.github.com> Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
netlify · Sep 29, 2021 · 3c4f32a · 3c4f32a · github-actions · Sep 29, 2021
1 parent df6c093
commit 3c4f32a
Show file tree

Hide file tree

Showing 73 changed files with 58 additions and 180 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -13,4 +13,5 @@ module.exports = {
       },
     },
   ],
+  ignorePatterns: ['tests/fixtures/**/*'],
 }
diff --git a/benchmarks/fixtures/next_api_hello/nextPage.js b/benchmarks/fixtures/next_api_hello/nextPage.js
@@ -1989,13 +1989,11 @@ function wrapfunction (fn, message) {
   }
 
   var args = createArgumentsString(fn.length)
-  var deprecate = this // eslint-disable-line no-unused-vars
+  var deprecate = this
   var stack = getStack()
   var site = callSiteLocation(stack[1])
 
   site.name = fn.name
-
-   // eslint-disable-next-line no-eval
   var deprecatedfn = eval('(function (' + args + ') {\n' +
     '"use strict"\n' +
     'log.call(deprecate, message, site)\n' +

diff --git a/benchmarks/fixtures/next_index/nextPage.js b/benchmarks/fixtures/next_index/nextPage.js
@@ -301,7 +301,7 @@ renderOpts.inAmpMode=inAmpMode;renderOpts.hybridAmp=hybridAmp;const docComponent
 unstable_runtimeJS: true?pageConfig.unstable_runtimeJS:undefined,unstable_JsPreload:pageConfig.unstable_JsPreload,dangerousAsPath:router.asPath,ampState,props,headTags:await headTags(documentCtx),isFallback,docProps,pathname,ampPath,query,inAmpMode,hybridAmp,dynamicImportsIds,dynamicImports,gsp:!!getStaticProps?true:undefined,gssp:!!getServerSideProps?true:undefined,gip:hasPageGetInitialProps?true:undefined,appGip:!defaultAppGetInitialProps?true:undefined,devOnlyCacheBusterQueryString,scriptLoader});if(false){}if(inAmpMode&&html){// inject HTML to AMP_RENDER_TARGET to allow rendering
 // directly to body in AMP mode
 const ampRenderIndex=html.indexOf(_constants2.AMP_RENDER_TARGET);html=html.substring(0,ampRenderIndex)+`<!-- __NEXT_DATA__ -->${docProps.html}`+html.substring(ampRenderIndex+_constants2.AMP_RENDER_TARGET.length);html=await(0,_optimizeAmp.default)(html,renderOpts.ampOptimizerConfig);if(!renderOpts.ampSkipValidation&&renderOpts.ampValidator){await renderOpts.ampValidator(html,pathname);}}// Avoid postProcess if both flags are false
-if(false){}if(renderOpts.optimizeCss){// eslint-disable-next-line import/no-extraneous-dependencies
+if(false){}if(renderOpts.optimizeCss){
 const Critters=__webpack_require__("WADY");const cssOptimizer=new Critters({ssrMode:true,reduceInlineStyles:false,path:renderOpts.distDir,publicPath:'/_next/',preload:'media',fonts:false,...renderOpts.optimizeCss});html=await cssOptimizer.process(html);}if(inAmpMode||hybridAmp){// fix &amp being escaped for amphtml rel link
 html=html.replace(/&amp;amp=1/g,'&amp=1');}return html;}function errorToJSON(err){const{name,message,stack}=err;return{name,message,stack};}function serializeError(dev,err){if(dev){return errorToJSON(err);}return{name:'Internal Server Error.',message:'500 - Internal Server Error.',statusCode:500};}
 //# sourceMappingURL=render.js.map
@@ -1338,8 +1338,6 @@ module.exports = (chalk, tmp) => {
 	const styles = [];
 	const chunks = [];
 	let chunk = [];
-
-	// eslint-disable-next-line max-params
 	tmp.replace(TEMPLATE_REGEX, (m, escapeChar, inverse, style, close, chr) => {
 		if (escapeChar) {
 			chunk.push(unescape(escapeChar));
@@ -1827,7 +1825,7 @@ function build(_styles, _empty, key) {
 
 	// `__proto__` is used because we must return a function, but there is
 	// no way to create a function with a different prototype
-	builder.__proto__ = proto; // eslint-disable-line no-proto
+	builder.__proto__ = proto;
 
 	return builder;
 }
@@ -1899,7 +1897,7 @@ function chalkTag(chalk, strings) {
 
 Object.defineProperties(Chalk.prototype, styles);
 
-module.exports = Chalk(); // eslint-disable-line new-cap
+module.exports = Chalk();
 module.exports.supportsColor = stdoutColor;
 module.exports.default = module.exports; // For TypeScript
 
@@ -2847,7 +2845,7 @@ function unique() {
       } else {
         keys.add(key);
       }
-    } // eslint-disable-next-line default-case
+    }
 
 
     switch (h.type) {
@@ -3563,13 +3561,11 @@ function wrapfunction (fn, message) {
   }
 
   var args = createArgumentsString(fn.length)
-  var deprecate = this // eslint-disable-line no-unused-vars
+  var deprecate = this
   var stack = getStack()
   var site = callSiteLocation(stack[1])
 
   site.name = fn.name
-
-   // eslint-disable-next-line no-eval
   var deprecatedfn = eval('(function (' + args + ') {\n' +
     '"use strict"\n' +
     'log.call(deprecate, message, site)\n' +
@@ -5138,7 +5134,7 @@ function shouldUseNative() {
 		// Detect buggy property enumeration order in older V8 versions.
 
 		// https://bugs.chromium.org/p/v8/issues/detail?id=4118
-		var test1 = new String('abc');  // eslint-disable-line no-new-wrappers
+		var test1 = new String('abc');
 		test1[5] = 'de';
 		if (Object.getOwnPropertyNames(test1)[0] === '5') {
 			return false;
@@ -9140,7 +9136,6 @@ function mitt() {
     },
 
     emit(type, ...evts) {
-      // eslint-disable-next-line array-callback-return
       ;
       (all[type] || []).slice().map(handler => {
         handler(...evts);

diff --git a/benchmarks/fixtures/next_index_2/nextPage.js b/benchmarks/fixtures/next_index_2/nextPage.js
@@ -301,7 +301,7 @@ renderOpts.inAmpMode=inAmpMode;renderOpts.hybridAmp=hybridAmp;const docComponent
 unstable_runtimeJS: true?pageConfig.unstable_runtimeJS:undefined,unstable_JsPreload:pageConfig.unstable_JsPreload,dangerousAsPath:router.asPath,ampState,props,headTags:await headTags(documentCtx),isFallback,docProps,pathname,ampPath,query,inAmpMode,hybridAmp,dynamicImportsIds,dynamicImports,gsp:!!getStaticProps?true:undefined,gssp:!!getServerSideProps?true:undefined,gip:hasPageGetInitialProps?true:undefined,appGip:!defaultAppGetInitialProps?true:undefined,devOnlyCacheBusterQueryString,scriptLoader});if(false){}if(inAmpMode&&html){// inject HTML to AMP_RENDER_TARGET to allow rendering
 // directly to body in AMP mode
 const ampRenderIndex=html.indexOf(_constants2.AMP_RENDER_TARGET);html=html.substring(0,ampRenderIndex)+`<!-- __NEXT_DATA__ -->${docProps.html}`+html.substring(ampRenderIndex+_constants2.AMP_RENDER_TARGET.length);html=await(0,_optimizeAmp.default)(html,renderOpts.ampOptimizerConfig);if(!renderOpts.ampSkipValidation&&renderOpts.ampValidator){await renderOpts.ampValidator(html,pathname);}}// Avoid postProcess if both flags are false
-if(false){}if(renderOpts.optimizeCss){// eslint-disable-next-line import/no-extraneous-dependencies
+if(false){}if(renderOpts.optimizeCss){
 const Critters=__webpack_require__("WADY");const cssOptimizer=new Critters({ssrMode:true,reduceInlineStyles:false,path:renderOpts.distDir,publicPath:'/_next/',preload:'media',fonts:false,...renderOpts.optimizeCss});html=await cssOptimizer.process(html);}if(inAmpMode||hybridAmp){// fix &amp being escaped for amphtml rel link
 html=html.replace(/&amp;amp=1/g,'&amp=1');}return html;}function errorToJSON(err){const{name,message,stack}=err;return{name,message,stack};}function serializeError(dev,err){if(dev){return errorToJSON(err);}return{name:'Internal Server Error.',message:'500 - Internal Server Error.',statusCode:500};}
 //# sourceMappingURL=render.js.map
@@ -1338,8 +1338,6 @@ module.exports = (chalk, tmp) => {
 	const styles = [];
 	const chunks = [];
 	let chunk = [];
-
-	// eslint-disable-next-line max-params
 	tmp.replace(TEMPLATE_REGEX, (m, escapeChar, inverse, style, close, chr) => {
 		if (escapeChar) {
 			chunk.push(unescape(escapeChar));
@@ -1827,7 +1825,7 @@ function build(_styles, _empty, key) {
 
 	// `__proto__` is used because we must return a function, but there is
 	// no way to create a function with a different prototype
-	builder.__proto__ = proto; // eslint-disable-line no-proto
+	builder.__proto__ = proto;
 
 	return builder;
 }
@@ -1899,7 +1897,7 @@ function chalkTag(chalk, strings) {
 
 Object.defineProperties(Chalk.prototype, styles);
 
-module.exports = Chalk(); // eslint-disable-line new-cap
+module.exports = Chalk();
 module.exports.supportsColor = stdoutColor;
 module.exports.default = module.exports; // For TypeScript
 
@@ -2847,7 +2845,7 @@ function unique() {
       } else {
         keys.add(key);
       }
-    } // eslint-disable-next-line default-case
+    }
 
 
     switch (h.type) {
@@ -3563,13 +3561,11 @@ function wrapfunction (fn, message) {
   }
 
   var args = createArgumentsString(fn.length)
-  var deprecate = this // eslint-disable-line no-unused-vars
+  var deprecate = this
   var stack = getStack()
   var site = callSiteLocation(stack[1])
 
   site.name = fn.name
-
-   // eslint-disable-next-line no-eval
   var deprecatedfn = eval('(function (' + args + ') {\n' +
     '"use strict"\n' +
     'log.call(deprecate, message, site)\n' +
@@ -5138,7 +5134,7 @@ function shouldUseNative() {
 		// Detect buggy property enumeration order in older V8 versions.
 
 		// https://bugs.chromium.org/p/v8/issues/detail?id=4118
-		var test1 = new String('abc');  // eslint-disable-line no-new-wrappers
+		var test1 = new String('abc');
 		test1[5] = 'de';
 		if (Object.getOwnPropertyNames(test1)[0] === '5') {
 			return false;
@@ -9140,7 +9136,6 @@ function mitt() {
     },
 
     emit(type, ...evts) {
-      // eslint-disable-next-line array-callback-return
       ;
       (all[type] || []).slice().map(handler => {
         handler(...evts);

diff --git a/benchmarks/fixtures/next_index_3/nextPage.js b/benchmarks/fixtures/next_index_3/nextPage.js
@@ -301,7 +301,7 @@ renderOpts.inAmpMode=inAmpMode;renderOpts.hybridAmp=hybridAmp;const docComponent
 unstable_runtimeJS: true?pageConfig.unstable_runtimeJS:undefined,unstable_JsPreload:pageConfig.unstable_JsPreload,dangerousAsPath:router.asPath,ampState,props,headTags:await headTags(documentCtx),isFallback,docProps,pathname,ampPath,query,inAmpMode,hybridAmp,dynamicImportsIds,dynamicImports,gsp:!!getStaticProps?true:undefined,gssp:!!getServerSideProps?true:undefined,gip:hasPageGetInitialProps?true:undefined,appGip:!defaultAppGetInitialProps?true:undefined,devOnlyCacheBusterQueryString,scriptLoader});if(false){}if(inAmpMode&&html){// inject HTML to AMP_RENDER_TARGET to allow rendering
 // directly to body in AMP mode
 const ampRenderIndex=html.indexOf(_constants2.AMP_RENDER_TARGET);html=html.substring(0,ampRenderIndex)+`<!-- __NEXT_DATA__ -->${docProps.html}`+html.substring(ampRenderIndex+_constants2.AMP_RENDER_TARGET.length);html=await(0,_optimizeAmp.default)(html,renderOpts.ampOptimizerConfig);if(!renderOpts.ampSkipValidation&&renderOpts.ampValidator){await renderOpts.ampValidator(html,pathname);}}// Avoid postProcess if both flags are false
-if(false){}if(renderOpts.optimizeCss){// eslint-disable-next-line import/no-extraneous-dependencies
+if(false){}if(renderOpts.optimizeCss){
 const Critters=__webpack_require__("WADY");const cssOptimizer=new Critters({ssrMode:true,reduceInlineStyles:false,path:renderOpts.distDir,publicPath:'/_next/',preload:'media',fonts:false,...renderOpts.optimizeCss});html=await cssOptimizer.process(html);}if(inAmpMode||hybridAmp){// fix &amp being escaped for amphtml rel link
 html=html.replace(/&amp;amp=1/g,'&amp=1');}return html;}function errorToJSON(err){const{name,message,stack}=err;return{name,message,stack};}function serializeError(dev,err){if(dev){return errorToJSON(err);}return{name:'Internal Server Error.',message:'500 - Internal Server Error.',statusCode:500};}
 //# sourceMappingURL=render.js.map
@@ -1338,8 +1338,6 @@ module.exports = (chalk, tmp) => {
 	const styles = [];
 	const chunks = [];
 	let chunk = [];
-
-	// eslint-disable-next-line max-params
 	tmp.replace(TEMPLATE_REGEX, (m, escapeChar, inverse, style, close, chr) => {
 		if (escapeChar) {
 			chunk.push(unescape(escapeChar));
@@ -1827,7 +1825,7 @@ function build(_styles, _empty, key) {
 
 	// `__proto__` is used because we must return a function, but there is
 	// no way to create a function with a different prototype
-	builder.__proto__ = proto; // eslint-disable-line no-proto
+	builder.__proto__ = proto;
 
 	return builder;
 }
@@ -1899,7 +1897,7 @@ function chalkTag(chalk, strings) {
 
 Object.defineProperties(Chalk.prototype, styles);
 
-module.exports = Chalk(); // eslint-disable-line new-cap
+module.exports = Chalk();
 module.exports.supportsColor = stdoutColor;
 module.exports.default = module.exports; // For TypeScript
 
@@ -2847,7 +2845,7 @@ function unique() {
       } else {
         keys.add(key);
       }
-    } // eslint-disable-next-line default-case
+    }
 
 
     switch (h.type) {
@@ -3563,13 +3561,11 @@ function wrapfunction (fn, message) {
   }
 
   var args = createArgumentsString(fn.length)
-  var deprecate = this // eslint-disable-line no-unused-vars
+  var deprecate = this
   var stack = getStack()
   var site = callSiteLocation(stack[1])
 
   site.name = fn.name
-
-   // eslint-disable-next-line no-eval
   var deprecatedfn = eval('(function (' + args + ') {\n' +
     '"use strict"\n' +
     'log.call(deprecate, message, site)\n' +
@@ -5138,7 +5134,7 @@ function shouldUseNative() {
 		// Detect buggy property enumeration order in older V8 versions.
 
 		// https://bugs.chromium.org/p/v8/issues/detail?id=4118
-		var test1 = new String('abc');  // eslint-disable-line no-new-wrappers
+		var test1 = new String('abc');
 		test1[5] = 'de';
 		if (Object.getOwnPropertyNames(test1)[0] === '5') {
 			return false;
@@ -9140,7 +9136,6 @@ function mitt() {
     },
 
     emit(type, ...evts) {
-      // eslint-disable-next-line array-callback-return
       ;
       (all[type] || []).slice().map(handler => {
         handler(...evts);

diff --git a/benchmarks/fixtures/next_index_4/nextPage.js b/benchmarks/fixtures/next_index_4/nextPage.js
@@ -301,7 +301,7 @@ renderOpts.inAmpMode=inAmpMode;renderOpts.hybridAmp=hybridAmp;const docComponent
 unstable_runtimeJS: true?pageConfig.unstable_runtimeJS:undefined,unstable_JsPreload:pageConfig.unstable_JsPreload,dangerousAsPath:router.asPath,ampState,props,headTags:await headTags(documentCtx),isFallback,docProps,pathname,ampPath,query,inAmpMode,hybridAmp,dynamicImportsIds,dynamicImports,gsp:!!getStaticProps?true:undefined,gssp:!!getServerSideProps?true:undefined,gip:hasPageGetInitialProps?true:undefined,appGip:!defaultAppGetInitialProps?true:undefined,devOnlyCacheBusterQueryString,scriptLoader});if(false){}if(inAmpMode&&html){// inject HTML to AMP_RENDER_TARGET to allow rendering
 // directly to body in AMP mode
 const ampRenderIndex=html.indexOf(_constants2.AMP_RENDER_TARGET);html=html.substring(0,ampRenderIndex)+`<!-- __NEXT_DATA__ -->${docProps.html}`+html.substring(ampRenderIndex+_constants2.AMP_RENDER_TARGET.length);html=await(0,_optimizeAmp.default)(html,renderOpts.ampOptimizerConfig);if(!renderOpts.ampSkipValidation&&renderOpts.ampValidator){await renderOpts.ampValidator(html,pathname);}}// Avoid postProcess if both flags are false
-if(false){}if(renderOpts.optimizeCss){// eslint-disable-next-line import/no-extraneous-dependencies
+if(false){}if(renderOpts.optimizeCss){
 const Critters=__webpack_require__("WADY");const cssOptimizer=new Critters({ssrMode:true,reduceInlineStyles:false,path:renderOpts.distDir,publicPath:'/_next/',preload:'media',fonts:false,...renderOpts.optimizeCss});html=await cssOptimizer.process(html);}if(inAmpMode||hybridAmp){// fix &amp being escaped for amphtml rel link
 html=html.replace(/&amp;amp=1/g,'&amp=1');}return html;}function errorToJSON(err){const{name,message,stack}=err;return{name,message,stack};}function serializeError(dev,err){if(dev){return errorToJSON(err);}return{name:'Internal Server Error.',message:'500 - Internal Server Error.',statusCode:500};}
 //# sourceMappingURL=render.js.map
@@ -1338,8 +1338,6 @@ module.exports = (chalk, tmp) => {
 	const styles = [];
 	const chunks = [];
 	let chunk = [];
-
-	// eslint-disable-next-line max-params
 	tmp.replace(TEMPLATE_REGEX, (m, escapeChar, inverse, style, close, chr) => {
 		if (escapeChar) {
 			chunk.push(unescape(escapeChar));
@@ -1827,7 +1825,7 @@ function build(_styles, _empty, key) {
 
 	// `__proto__` is used because we must return a function, but there is
 	// no way to create a function with a different prototype
-	builder.__proto__ = proto; // eslint-disable-line no-proto
+	builder.__proto__ = proto;
 
 	return builder;
 }
@@ -1899,7 +1897,7 @@ function chalkTag(chalk, strings) {
 
 Object.defineProperties(Chalk.prototype, styles);
 
-module.exports = Chalk(); // eslint-disable-line new-cap
+module.exports = Chalk();
 module.exports.supportsColor = stdoutColor;
 module.exports.default = module.exports; // For TypeScript
 
@@ -2847,7 +2845,7 @@ function unique() {
       } else {
         keys.add(key);
       }
-    } // eslint-disable-next-line default-case
+    }
 
 
     switch (h.type) {
@@ -3563,13 +3561,11 @@ function wrapfunction (fn, message) {
   }
 
   var args = createArgumentsString(fn.length)
-  var deprecate = this // eslint-disable-line no-unused-vars
+  var deprecate = this
   var stack = getStack()
   var site = callSiteLocation(stack[1])
 
   site.name = fn.name
-
-   // eslint-disable-next-line no-eval
   var deprecatedfn = eval('(function (' + args + ') {\n' +
     '"use strict"\n' +
     'log.call(deprecate, message, site)\n' +
@@ -5138,7 +5134,7 @@ function shouldUseNative() {
 		// Detect buggy property enumeration order in older V8 versions.
 
 		// https://bugs.chromium.org/p/v8/issues/detail?id=4118
-		var test1 = new String('abc');  // eslint-disable-line no-new-wrappers
+		var test1 = new String('abc');
 		test1[5] = 'de';
 		if (Object.getOwnPropertyNames(test1)[0] === '5') {
 			return false;
@@ -9140,7 +9136,6 @@ function mitt() {
     },
 
     emit(type, ...evts) {
-      // eslint-disable-next-line array-callback-return
       ;
       (all[type] || []).slice().map(handler => {
         handler(...evts);

diff --git a/tests/fixtures/invalid-require/function.js b/tests/fixtures/invalid-require/function.js
@@ -1,6 +1,4 @@
-// eslint-disable-next-line no-constant-condition
 if (false) {
-  // eslint-disable-next-line import/no-unresolved, import/no-unassigned-import, node/global-require
   require('@scope')
 }
 

diff --git a/tests/fixtures/local-node-module/function.js b/tests/fixtures/local-node-module/function.js
@@ -1,4 +1,3 @@
-// eslint-disable-next-line import/no-unassigned-import
 require('test')
 
 module.exports = true
diff --git a/tests/fixtures/local-require-esm/function/file.js b/tests/fixtures/local-require-esm/function/file.js
@@ -1,4 +1,3 @@
-// eslint-disable-next-line node/no-unsupported-features/es-syntax
 export default function getZero() {
   return 0
 }
diff --git a/tests/fixtures/local-require-esm/function/function.js b/tests/fixtures/local-require-esm/function/function.js
@@ -1,5 +1,3 @@
-// eslint-disable-next-line node/no-unsupported-features/es-syntax, import/extensions
 import getZero from './file.js'
 
-// eslint-disable-next-line node/no-unsupported-features/es-syntax
 export const ZERO = getZero()
diff --git a/tests/fixtures/local-require-esm/function_export_only/function_export_only.js b/tests/fixtures/local-require-esm/function_export_only/function_export_only.js
@@ -1,2 +1 @@
-// eslint-disable-next-line node/no-unsupported-features/es-syntax
 export const howdy = 'Yee haw!'
diff --git a/tests/fixtures/local-require-esm/function_import_only/function_import_only.js b/tests/fixtures/local-require-esm/function_import_only/function_import_only.js
@@ -1,4 +1,3 @@
-// eslint-disable-next-line node/no-unsupported-features/es-syntax
 import getZero from '../function/file'
 
 getZero()
diff --git a/tests/fixtures/many-dependencies/function.js b/tests/fixtures/many-dependencies/function.js
@@ -1,2 +1 @@
-// eslint-disable-next-line node/global-require
 module.exports = [require('execa'), require('del')].every(Boolean)
diff --git a/tests/fixtures/no-package-json/function.js b/tests/fixtures/no-package-json/function.js
@@ -1,2 +1 @@
-// eslint-disable-next-line node/no-extraneous-require
 module.exports = require('test')
diff --git a/tests/fixtures/node-module-deep-conditional/function.js b/tests/fixtures/node-module-deep-conditional/function.js
@@ -1,4 +1,3 @@
-// eslint-disable-next-line import/no-unassigned-import
 require('test')
 
 module.exports = true
diff --git a/tests/fixtures/node-module-dynamic-import-2/function.js b/tests/fixtures/node-module-dynamic-import-2/function.js
@@ -1,4 +1,2 @@
 const id = (arg) => arg
-
-// eslint-disable-next-line import/no-dynamic-require, node/global-require, prefer-template
-module.exports = (lang) => [require('./lang/' + lang + '.json'), require('./lang/' + id(lang) + '.json')]
+module.exports = (lang) => [require(`./lang/${lang}.json`), require(`./lang/${id(lang)}.json`)]
diff --git a/tests/fixtures/node-module-dynamic-import-3/function1/index.js b/tests/fixtures/node-module-dynamic-import-3/function1/index.js
@@ -1,4 +1,2 @@
 const id = (arg) => arg
-
-// eslint-disable-next-line import/no-dynamic-require, node/global-require, prefer-template
-module.exports = (lang) => [require('./lang/' + lang + '.json'), require('./lang/' + id(lang) + '.json')]
+module.exports = (lang) => [require(`./lang/${lang}.json`), require(`./lang/${id(lang)}.json`)]
diff --git a/tests/fixtures/node-module-dynamic-import-3/function10/index.js b/tests/fixtures/node-module-dynamic-import-3/function10/index.js
@@ -1,4 +1,2 @@
 const id = (arg) => arg
-
-// eslint-disable-next-line import/no-dynamic-require, node/global-require, prefer-template
-module.exports = (lang) => [require('./lang/' + lang + '.json'), require('./lang/' + id(lang) + '.json')]
+module.exports = (lang) => [require(`./lang/${lang}.json`), require(`./lang/${id(lang)}.json`)]
diff --git a/tests/fixtures/node-module-dynamic-import-3/function11/index.js b/tests/fixtures/node-module-dynamic-import-3/function11/index.js
@@ -1,4 +1,2 @@
 const id = (arg) => arg
-
-// eslint-disable-next-line import/no-dynamic-require, node/global-require, prefer-template
-module.exports = (lang) => [require('./lang/' + lang + '.json'), require('./lang/' + id(lang) + '.json')]
+module.exports = (lang) => [require(`./lang/${lang}.json`), require(`./lang/${id(lang)}.json`)]