fix: ignore case of disk symbol when check workspace on windows (#124)

Author: laggage

__mocks__/vscode.ts

@@ -98,6 +98,7 @@ const ThemeColor = jest.fn();
 const EventEmitter = jest.fn().mockImplementation(() => {
     return {
         fire: jest.fn(),
+        event: jest.fn(),
     };
 });
 
@@ -106,8 +107,15 @@ const QuickPickItemKind = {
     Default: 0,
 };
 
+const TreeItem = jest.fn();
+const Disposable = jest.fn();
+(Disposable as any).from = jest.fn();
+const authentication = { registerAuthenticationProvider: jest.fn() };
+
 export = {
     ThemeColor,
+    Disposable,
+    authentication,
     CodeLens,
     languages,
     StatusBarAlignment,
@@ -132,4 +140,5 @@ export = {
     TestRunRequest,
     ViewColumn,
     QuickPickItemKind,
+    TreeItem,
 };

src/services/check-workspace.test.ts

@@ -0,0 +1,49 @@
+import { workspace } from 'vscode';
+
+// eslint-disable-next-line @typescript-eslint/no-misused-promises
+describe('isTargetWorkspace', () => {
+    const testItems: [current: string, target: string, result: boolean][] = [
+        ['c:/test', 'C:/test', true],
+        ['C:/test', 'c:/test', true],
+        ['c:/test', 'c:/test', true],
+        ['D:/test', 'D:/test', true],
+        ['c:/test', 'd:/test', false],
+        ['d:/test', 'd:/test/', false],
+    ];
+    let mockSettings: jest.Mock<any, any, any>;
+
+    beforeAll(() => {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+        const actualGlobalContext = jest.requireActual('./global-state.ts').globalContext;
+        Object.defineProperty(actualGlobalContext, 'extensionName', {
+            get: jest.fn().mockReturnValue(''),
+        });
+        jest.mock('./global-state', () => ({
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+            globalContext: actualGlobalContext,
+        }));
+
+        jest.mock('os', () => ({ platform: jest.fn().mockReturnValue('win32') }));
+        mockSettings = jest.fn();
+        jest.mock('./settings.service', () => ({
+            // eslint-disable-next-line @typescript-eslint/naming-convention
+            Settings: mockSettings,
+        }));
+    });
+
+    for (const item of testItems) {
+        const [current, target, isInTargetWorkspace] = item;
+
+        it(`should${isInTargetWorkspace ? '' : ' not'} in target workspace, ${current}, ${target}`, async () => {
+            const mockedWorkspace = jest.mocked(workspace);
+            // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+            // @ts-ignore
+            mockedWorkspace.workspaceFolders = [{ name: '', uri: { path: current } }];
+
+            (mockSettings as any).workspaceUri = { path: target };
+
+            const { isTargetWorkspace } = await import('@/services/check-workspace');
+            expect(isTargetWorkspace()).toStrictEqual(isInTargetWorkspace);
+        });
+    }
+});

src/services/check-workspace.ts

@@ -1,13 +1,25 @@
+import os from 'os';
 import { commands, workspace } from 'vscode';
 import { refreshPostCategoriesList } from '../commands/post-category/refresh-post-categories-list';
 import { refreshPostsList } from '../commands/posts-list/refresh-posts-list';
 import { globalContext } from './global-state';
 import { PostFileMapManager } from './post-file-map';
 import { Settings } from './settings.service';
 
+const diskSymbolRegex = /^(\S{1,5}:)(.*)/;
+
 export const isTargetWorkspace = (): boolean => {
     const folders = workspace.workspaceFolders;
-    const isTarget = !!folders && folders.length === 1 && folders[0].uri.path === Settings.workspaceUri.path;
+    let currentFolder = folders?.length === 1 ? folders[0].uri.path : undefined;
+    let targetFolder = Settings.workspaceUri.path;
+    const platform = os.platform();
+    if (platform === 'win32' && targetFolder && currentFolder) {
+        const replacer = (sub: string, m0: string | null | undefined, m2: string | null | undefined) =>
+            m0 && m2 ? m0.toLowerCase() + m2 : sub;
+        currentFolder = currentFolder.replace(diskSymbolRegex, replacer);
+        targetFolder = targetFolder.replace(diskSymbolRegex, replacer);
+    }
+    const isTarget = !!currentFolder && currentFolder === targetFolder;
     void commands.executeCommand('setContext', `${globalContext.extensionName}.isTargetWorkspace`, isTarget);
     return isTarget;
 };

test/jest.config.mjs

@@ -14,16 +14,26 @@ const config = {
     testPathIgnorePatterns: ['<rootDir>/src/test/suite'],
     testEnvironment: 'node',
     detectOpenHandles: true,
-    moduleFileExtensions: ['js', 'ts', 'json'],
+    moduleFileExtensions: ['js', 'ts', 'json', 'mjs'],
     collectCoverage: false,
-    transform: { '^.+\\.[tj]sx?$': ['ts-jest', { tsconfig: '<rootDir>/tsconfig.spec.json' }] },
+    transform: {
+        '^.+\\.(mjs|ts|js|cjs|jsx)$': ['ts-jest', { tsconfig: '<rootDir>/tsconfig.spec.json', useESM: true }],
+    },
     rootDir: '../',
+    globals: {
+        // eslint-disable-next-line @typescript-eslint/naming-convention
+        CNBLOGS_CLIENTID: '',
+        // eslint-disable-next-line @typescript-eslint/naming-convention
+        CNBLOGS_CLIENTSECRET: '',
+    },
     forceExit: true,
     roots: ['<rootDir>'],
     verbose: true,
     modulePaths: ['<rootDir>', '<rootDir>/node_modules'],
     moduleDirectories: ['<rootDir>', '<rootDir>/node_modules'],
-    transformIgnorePatterns: ['/node_modules/(?!(got|got-fetch)/)'],
+    transformIgnorePatterns: [
+        'node_modules/(?!(got-fetch|got|@sindresorhus|p-cancelable|@szmarczak|cacheable-request|cacheable-lookup|normalize-url|responselike|lowercase-keys|mimic-response|form-data-encoder))',
+    ],
     projects: ['<rootDir>'],
     automock: false,
     moduleNameMapper: {

tsconfig.spec.json

@@ -5,10 +5,10 @@
     "compilerOptions": {
         "rootDir": "",
         "resolveJsonModule": true,
-        "moduleResolution": "node16",
         "skipLibCheck": true,
         "esModuleInterop": true,
         "allowJs": true,
-        "outDir": "out"
+        "outDir": "out",
+        "module": "CommonJS"
     }
 }