filename-case: Ignore more index files (#826)

sindresorhus · Sep 15, 2020 · 64d03a5 · 64d03a5
1 parent b35c261
commit 64d03a5
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/rules/filename-case.js b/rules/filename-case.js
@@ -9,6 +9,7 @@ const numberRegex = /\d+/;
 const PLACEHOLDER = '\uFFFF\uFFFF\uFFFF';
 const PLACEHOLDER_REGEX = new RegExp(PLACEHOLDER, 'i');
 const isIgnoredChar = char => !/^[a-z\d-_$]$/i.test(char);
+const ignoredByDefault = new Set(['index.js', 'index.mjs', 'index.cjs', 'index.ts', 'index.tsx', 'index.vue']);
 
 function ignoreNumbers(fn) {
 	return string => {
@@ -159,7 +160,7 @@ const create = context => {
 			const filename = path.basename(filenameWithExtension, extension);
 			const base = filename + extension;
 
-			if (base === 'index.js' || ignore.some(regexp => regexp.test(base))) {
+			if (ignoredByDefault.has(base) || ignore.some(regexp => regexp.test(base))) {
 				return;
 			}
 

diff --git a/test/filename-case.js b/test/filename-case.js
@@ -1,5 +1,6 @@
 import test from 'ava';
 import avaRuleTester from 'eslint-ava-rule-tester';
+import {flatten} from 'lodash';
 import rule from '../rules/filename-case';
 
 const ruleTester = avaRuleTester(test, {
@@ -73,7 +74,6 @@ ruleTester.run('filename-case', rule, {
 		testCase('spec/Iss47Spec.js', 'pascalCase'),
 		testCase('spec/Iss47.100Spec.js', 'pascalCase'),
 		testCase('spec/I18n.js', 'pascalCase'),
-		testCase('spec/index.js', 'pascalCase'),
 		testCase(undefined, 'camelCase'),
 		testCase(undefined, 'snakeCase'),
 		testCase(undefined, 'kebabCase'),
@@ -239,7 +239,13 @@ ruleTester.run('filename-case', rule, {
 				},
 				ignore: [/FOOBAR\.js/u, /BaRbAz\.js/u]
 			}
-		])
+		]),
+		// Ignored
+		...flatten(
+			['index.js', 'index.mjs', 'index.cjs', 'index.ts', 'index.tsx', 'index.vue'].map(
+				filename => ['camelCase', 'snakeCase', 'kebabCase', 'pascalCase'].map(chosenCase => testCase(filename, chosenCase))
+			)
+		)
 	],
 	invalid: [
 		testCase(