fix: default node path should be using path.join for cross platform…

… path construction
MatrixAI · Aug 11, 2022 · 27df70f · 27df70f
1 parent 7b1edbd
commit 27df70f
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 8 deletions.
diff --git a/src/utils/utils.ts b/src/utils/utils.ts
@@ -12,20 +12,20 @@ function getDefaultNodePath(): string | undefined {
     const homeDir = os.homedir();
     const dataDir = process.env.XDG_DATA_HOME;
     if (dataDir != null) {
-      p = `${dataDir}/${prefix}`;
+      p = path.join(dataDir, prefix);
     } else {
-      p = `${homeDir}/.local/share/${prefix}`;
+      p = path.join(homeDir, '.local', 'share', prefix);
     }
   } else if (platform === 'darwin') {
     const homeDir = os.homedir();
-    p = `${homeDir}/Library/Application Support/${prefix}`;
+    p = path.join(homeDir, 'Library', 'Application Support', prefix);
   } else if (platform === 'win32') {
     const homeDir = os.homedir();
     const appDataDir = process.env.LOCALAPPDATA;
     if (appDataDir != null) {
-      p = `${appDataDir}/${prefix}`;
+      p = path.join(appDataDir, prefix);
     } else {
-      p = `${homeDir}/AppData/Local/${prefix}`;
+      p = path.join(homeDir, 'AppData', 'Local', prefix);
     }
   } else {
     return;

diff --git a/tests/utils.test.ts b/tests/utils.test.ts
@@ -1,16 +1,32 @@
 import os from 'os';
+import path from 'path';
+import process from 'process';
 import * as utils from '@/utils';
 
 describe('utils', () => {
   test('getting default node path', () => {
     const homeDir = os.homedir();
+    const prefix = 'polykey';
     const p = utils.getDefaultNodePath();
+    expect(p).toBeDefined();
     if (process.platform === 'linux') {
-      expect(p).toBe(`${homeDir}/.local/share/polykey`);
+      const dataDir = process.env.XDG_DATA_HOME;
+      if (dataDir != null) {
+        expect(p).toBe(path.join(dataDir, prefix));
+      } else {
+        expect(p).toBe(path.join(homeDir, '.local', 'share', prefix));
+      }
     } else if (process.platform === 'darwin') {
-      expect(p).toBe(`${homeDir}/Library/Application Support/polykey`);
+      expect(p).toBe(
+        path.join(homeDir, 'Library', 'Application Support', 'polykey'),
+      );
     } else if (process.platform === 'win32') {
-      expect(p).toBe(`${homeDir}/AppData/Local/polykey`);
+      const appDataDir = process.env.LOCALAPPDATA;
+      if (appDataDir != null) {
+        expect(p).toBe(path.join(appDataDir, prefix));
+      } else {
+        expect(p).toBe(path.join(homeDir, 'AppData', 'Local', prefix));
+      }
     }
   });
 });