filename-case: Improve report position (#1227)

rules/filename-case.js

@@ -158,7 +158,7 @@ const create = context => {
 	}
 
 	return {
-		Program: node => {
+		Program() {
 			const extension = path.extname(filenameWithExtension);
 			const filename = path.basename(filenameWithExtension, extension);
 			const base = filename + extension;
@@ -180,7 +180,9 @@ const create = context => {
 			});
 
 			context.report({
-				node,
+				// Report on first character like `unicode-bom` rule
+				// https://github.com/eslint/eslint/blob/8a77b661bc921c3408bae01b3aa41579edfc6e58/lib/rules/unicode-bom.js#L46
+				loc: {column: 0, line: 1},
 				messageId: MESSAGE_ID,
 				data: {
 					chosenCases: englishishJoinWords(chosenCases.map(x => cases[x].name)),

test/filename-case.mjs

@@ -538,3 +538,13 @@ test({
 		)
 	]
 });
+
+test.snapshot({
+	valid: [],
+	invalid: [
+		{
+			code: 'foo();\n'.repeat(10),
+			filename: 'src/foo/foo_bar.js'
+		}
+	]
+});

test/snapshots/filename-case.mjs.md

@@ -0,0 +1,35 @@
+# Snapshot report for `test/filename-case.mjs`
+
+The actual snapshot is saved in `filename-case.mjs.snap`.
+
+Generated by [AVA](https://avajs.dev).
+
+## Invalid #1
+       1 | foo();
+       2 | foo();
+       3 | foo();
+       4 | foo();
+       5 | foo();
+       6 | foo();
+       7 | foo();
+       8 | foo();
+       9 | foo();
+      10 | foo();
+      11 |
+
+> Error 1/1
+
+    `␊
+    >  1 | foo();␊
+         | ^ Filename is not in kebab case. Rename it to \`foo-bar.js\`.␊
+       2 | foo();␊
+       3 | foo();␊
+       4 | foo();␊
+       5 | foo();␊
+       6 | foo();␊
+       7 | foo();␊
+       8 | foo();␊
+       9 | foo();␊
+      10 | foo();␊
+      11 |␊
+    `