fix(#5): add getFilePath method to get file path

Author: dukeluo

lib/rules/filename-naming-convention.js

@@ -4,12 +4,7 @@
  */
 'use strict';
 
-const {
-  getFilename,
-  getBasename,
-  getPathFromRepositoryRoot,
-  toPosixPath,
-} = require('../utils/filename');
+const { getFilename, getBasename, getFilePath } = require('../utils/filename');
 const {
   checkSettings,
   namingPatternValidator,
@@ -70,16 +65,13 @@ module.exports = {
           return;
         }
 
-        const pathFromRepositoryRoot = getPathFromRepositoryRoot(
-          toPosixPath(context.getPhysicalFilename()),
-          toPosixPath(context.getCwd())
-        );
-        const filename = getFilename(pathFromRepositoryRoot);
+        const filenameWithPath = getFilePath(context);
+        const filename = getFilename(filenameWithPath);
 
         for (const [fexPattern, namingPattern] of Object.entries(rules)) {
           const matchResult =
             matchRule(
-              pathFromRepositoryRoot,
+              filenameWithPath,
               fexPattern,
               getBasename(filename, ignoreMiddleExtensions),
               namingPattern

lib/rules/folder-match-with-fex.js

@@ -7,7 +7,7 @@
 const {
   getFolderPath,
   getFilename,
-  toPosixPath,
+  getFilePath,
 } = require('../utils/filename');
 const { checkSettings, globPatternValidator } = require('../utils/settings');
 const { getDocUrl } = require('../utils/doc');
@@ -59,7 +59,7 @@ module.exports = {
           return;
         }
 
-        const filenameWithPath = toPosixPath(context.getPhysicalFilename());
+        const filenameWithPath = getFilePath(context);
         const filename = getFilename(filenameWithPath);
         const folderPath = getFolderPath(filenameWithPath);
 

lib/rules/folder-naming-convention.js

@@ -9,7 +9,7 @@ const {
   getFolderPath,
   getSubPaths,
   getAllFolders,
-  toPosixPath,
+  getFilePath,
 } = require('../utils/filename');
 const {
   checkSettings,
@@ -63,7 +63,7 @@ module.exports = {
           return;
         }
 
-        const filenameWithPath = toPosixPath(context.getPhysicalFilename());
+        const filenameWithPath = getFilePath(context);
         const folderPath = getFolderPath(filenameWithPath);
         const subPaths = getSubPaths(folderPath);
 

lib/rules/no-index.js

@@ -5,7 +5,7 @@
 'use strict';
 
 const { getDocUrl } = require('../utils/doc');
-const { getFilename, getBasename, toPosixPath } = require('../utils/filename');
+const { getFilename, getBasename, getFilePath } = require('../utils/filename');
 
 /**
  * @type {import('eslint').Rule.RuleModule}
@@ -26,7 +26,7 @@ module.exports = {
   create(context) {
     return {
       Program: (node) => {
-        const filenameWithPath = toPosixPath(context.getPhysicalFilename());
+        const filenameWithPath = getFilePath(context);
         const filename = getFilename(filenameWithPath);
         const basename = getBasename(filename);
 

lib/utils/filename.js

@@ -65,24 +65,56 @@ const getSubPaths = (p) => {
 
 /**
  * @type {string} path from repository root
- * @param {string} fullPath filename with full path in posix style
- * @param {string} repositoryRoot path of repository root in posix style
+ * @param {string} fullPath filename with full path
+ * @param {string} repositoryRoot path of repository root
  */
 const getPathFromRepositoryRoot = (fullPath, repositoryRoot) =>
-  fullPath.replace(path.join(repositoryRoot, path.posix.sep), '');
+  fullPath.replace(path.join(repositoryRoot, path.sep), '');
 
 /**
  * @type {string} file path in posix style
  * @param {string} p file path based on the operating system
  */
 const toPosixPath = (p) => p.split(path.sep).join(path.posix.sep);
 
+/**
+ * @type {string} file path without drive letter on windows
+ * @param {string} p file path on windows
+ */
+const removeDriveLetter = (p) => p.replace(/^[A-Za-z]:\\/, '');
+
+/**
+ * Callback for file path
+ * @callback callback
+ * @param {string} p file path
+ */
+/**
+ * @type {callback} piped function
+ * @param {callback[]} fns callback functions
+ */
+const pipe =
+  (...fns) =>
+  (x) =>
+    fns.reduce((v, f) => f(v), x);
+
+/**
+ * @type {string} file path in posix style
+ * @param {Rule.RuleContext} context rule eslint context
+ */
+const getFilePath = (context) => {
+  const pathFromRoot = getPathFromRepositoryRoot(
+    context.getPhysicalFilename(),
+    context.getCwd()
+  );
+
+  return pipe(removeDriveLetter, toPosixPath)(pathFromRoot);
+};
+
 module.exports = {
   getFolderPath,
   getFilename,
   getBasename,
   getSubPaths,
   getAllFolders,
-  getPathFromRepositoryRoot,
-  toPosixPath,
+  getFilePath,
 };