fix(use-env-prefix): fix around treating filename (#117)

fix #116

Author: tyankatsu0105

lib/rules/use-env-prefix.ts

@@ -5,7 +5,7 @@
  */
 import { AST } from "vue-eslint-parser";
 import minimatch from "minimatch";
-import { createRule, Env } from "../utils";
+import { createRule, Env, getPathFromProjectRoot } from "../utils";
 import * as Fs from "fs";
 
 const PREFIX = "GRIDSOME_";
@@ -49,14 +49,14 @@ export = createRule<[Options], MessageIds>({
   },
   defaultOptions,
   create(context) {
-    const filename = context.getFilename();
+    const path = getPathFromProjectRoot(context.getFilename(), process.cwd());
 
     const pathsForBrowserfileOption = context.options[0]
       ?.pathsForBrowserfile || ["src/**/*"];
     const envPathOption = context.options[0]?.envPath || ".env";
 
     const isClientfile = pathsForBrowserfileOption.some((clientPath) =>
-      minimatch(filename, clientPath)
+      minimatch(path, clientPath)
     );
 
     const envSource = Fs.readFileSync(envPathOption, { encoding: "utf8" });

lib/utils/index.ts

@@ -18,3 +18,8 @@ export * from "./directive";
 // env
 // ====================================================
 export * from "./env";
+
+// ====================================================
+// others
+// ====================================================
+export * from "./path";

lib/utils/path.ts

@@ -0,0 +1,2 @@
+export const getPathFromProjectRoot = (filename: string, cwd: string) =>
+  filename.split("/").splice(cwd.split("/").length).join("/");

tests/lib/rules/use-env-prefix.spec.ts

@@ -1,3 +1,4 @@
+import path from "path";
 import { RuleTester, loadFixtureCreator } from "../../util";
 
 import rule from "../../../lib/rules/use-env-prefix";
@@ -17,7 +18,7 @@ const loadFixture = loadFixtureCreator(
 tester.run("use-env-prefix", rule, {
   valid: [
     {
-      filename: "src/components/code.vue",
+      filename: path.join(process.cwd(), "src/components/code.vue"),
       ...loadFixture({
         fixtureDirectory: "valid/01",
       }),
@@ -29,7 +30,7 @@ tester.run("use-env-prefix", rule, {
       ],
     },
     {
-      filename: "src/client.js",
+      filename: path.join(process.cwd(), "src/client.js"),
       ...loadFixture({
         fixtureDirectory: "valid/02",
         filenames: {
@@ -46,7 +47,7 @@ tester.run("use-env-prefix", rule, {
   ],
   invalid: [
     {
-      filename: "src/components/code.vue",
+      filename: path.join(process.cwd(), "src/components/code.vue"),
       ...loadFixture({
         fixtureDirectory: "invalid/01",
       }),
@@ -63,7 +64,7 @@ tester.run("use-env-prefix", rule, {
       ],
     },
     {
-      filename: "src/client.js",
+      filename: path.join(process.cwd(), "src/client.js"),
       ...loadFixture({
         fixtureDirectory: "invalid/02",
         filenames: {

tests/lib/utils/path.spec.ts

@@ -0,0 +1,16 @@
+import { getPathFromProjectRoot } from "../../../lib/utils/path";
+
+import { expect } from "chai";
+
+describe("path", () => {
+  describe("getPathFromProjectRoot", () => {
+    it("when call getPathFromProjectRoot with context.getFilename() and process.cwd(), return path from project root", () => {
+      const filename = "/User/path/to/project/src/index.js";
+      const cwd = "/User/path/to/project";
+
+      const result = getPathFromProjectRoot(filename, cwd);
+
+      expect(result).to.deep.equal("src/index.js");
+    });
+  });
+});