modify to use clipboard object

Author: s-kai273

src/client/application/importPath/copyImportPathCommand.ts

@@ -2,7 +2,7 @@ import * as path from 'path';
 import * as vscode from 'vscode';
 import { inject, injectable } from 'inversify';
 
-import { ICommandManager, IWorkspaceService } from '../../common/application/types';
+import { IClipboard, ICommandManager, IWorkspaceService } from '../../common/application/types';
 import { IExtensionSingleActivationService } from '../../activation/types';
 import { Commands } from '../../common/constants';
 import { getSysPath } from '../../common/utils/pythonUtils';
@@ -14,6 +14,7 @@ export class CopyImportPathCommand implements IExtensionSingleActivationService
     constructor(
         @inject(ICommandManager) private readonly commands: ICommandManager,
         @inject(IWorkspaceService) private readonly workspace: IWorkspaceService,
+        @inject(IClipboard) private readonly clipboard: IClipboard,
     ) {}
 
     async activate(): Promise<void> {
@@ -28,7 +29,8 @@ export class CopyImportPathCommand implements IExtensionSingleActivationService
         }
 
         const importPath = this.resolveImportPath(uri.fsPath);
-        await vscode.env.clipboard.writeText(importPath);
+        // await vscode.env.clipboard.writeText(importPath);
+        await this.clipboard.writeText(importPath);
         void vscode.window.showInformationMessage(`Copied: ${importPath}`);
     }
 

src/test/application/importPath/copyImportPathCommand.unit.test.ts

@@ -7,28 +7,32 @@ import { anything, instance, mock, verify, when } from 'ts-mockito';
 import { expect } from 'chai';
 import * as vscode from 'vscode';
 import { CopyImportPathCommand } from '../../../client/application/importPath/copyImportPathCommand';
-import { ICommandManager, IWorkspaceService } from '../../../client/common/application/types';
+import { IClipboard, ICommandManager, IWorkspaceService } from '../../../client/common/application/types';
 import * as pythonUtils from '../../../client/common/utils/pythonUtils';
+import { ClipboardService } from '../../../client/common/application/clipboard';
+import { CommandManager } from '../../../client/common/application/commandManager';
+import { WorkspaceService } from '../../../client/common/application/workspace';
 
 suite('Copy Import Path Command', () => {
     let command: CopyImportPathCommand;
     let commandManager: ICommandManager;
     let workspaceService: IWorkspaceService;
+    let clipboard: IClipboard;
     let originalGetSysPath: () => string[];
 
     let clipboardText = '';
 
     setup(() => {
-        commandManager = mock<ICommandManager>();
-        workspaceService = mock<IWorkspaceService>();
-        command = new CopyImportPathCommand(instance(commandManager), instance(workspaceService));
+        commandManager = mock(CommandManager);
+        workspaceService = mock(WorkspaceService);
+        clipboard = mock(ClipboardService);
+        command = new CopyImportPathCommand(instance(commandManager), instance(workspaceService), instance(clipboard));
         originalGetSysPath = pythonUtils.getSysPath;
 
         clipboardText = '';
-
-        (vscode.env.clipboard as typeof vscode.env.clipboard).writeText = async (text: string) => {
+        when(clipboard.writeText(anything())).thenCall(async (text: string) => {
             clipboardText = text;
-        };
+        });
     });
 
     teardown(() => {
@@ -43,8 +47,7 @@ suite('Copy Import Path Command', () => {
     test('execute() – sys.path match takes precedence', async () => {
         const absPath = '/home/user/project/src/pkg/module.py';
         const uri = vscode.Uri.file(absPath);
-        ((pythonUtils as unknown) as { getSysPath: () => string[] }).getSysPath = originalGetSysPath;
-        // ((pythonUtils as unknown) as { getSysPath: () => string[] }).getSysPath = () => ['/home/user/project/src'];
+        ((pythonUtils as unknown) as { getSysPath: () => string[] }).getSysPath = () => ['/home/user/project/src'];
 
         when(workspaceService.getWorkspaceFolder(anything())).thenReturn(undefined);
         ((vscode.window as unknown) as { activeTextEditor: { document: { uri: vscode.Uri } } }).activeTextEditor = {