chore: add test for OsscodeActionsProvider

src/snyk/snykOss/ossService.ts

@@ -42,7 +42,7 @@ export class OssService extends ProductService<OssIssueData> {
     );
 
     this.registerCodeActionsProvider(
-      new OssCodeActionsProvider(this.result, codeActionAdapter, codeActionKindAdapter, languages, analytics),
+      new OssCodeActionsProvider(languages, codeActionAdapter, codeActionKindAdapter, this.result, analytics),
     );
   }
 

src/snyk/snykOss/providers/ossCodeActionsProvider.ts

@@ -13,10 +13,10 @@ import { DIAGNOSTICS_OSS_COLLECTION_NAME_LS } from '../../snykCode/constants/ana
 
 export class OssCodeActionsProvider extends CodeActionsProvider<OssIssueData> {
   constructor(
-    issues: Readonly<ProductResult<OssIssueData>>,
+    private readonly languages: IVSCodeLanguages,
     private readonly codeActionAdapter: ICodeActionAdapter,
     codeActionKindAdapter: ICodeActionKindAdapter,
-    private readonly languages: IVSCodeLanguages,
+    issues: Readonly<ProductResult<OssIssueData>>,
     analytics: IAnalytics,
   ) {
     super(issues, codeActionKindAdapter, analytics);

src/test/unit/snykOss/providers/ossCodeActionsProvider.test.ts

@@ -0,0 +1,118 @@
+import assert from 'assert';
+import sinon from 'sinon';
+import { CodeAction, CodeActionContext, CodeActionKind, Range, TextDocument } from 'vscode';
+import { IAnalytics } from '../../../../snyk/common/analytics/itly';
+import { OpenCommandIssueType, OpenIssueCommandArg } from '../../../../snyk/common/commands/types';
+import { SNYK_OPEN_ISSUE_COMMAND } from '../../../../snyk/common/constants/commands';
+import { Issue, IssueSeverity, OssIssueData } from '../../../../snyk/common/languageServer/types';
+import { WorkspaceFolderResult } from '../../../../snyk/common/services/productService';
+import { ICodeActionAdapter, ICodeActionKindAdapter } from '../../../../snyk/common/vscode/codeAction';
+import { IVSCodeLanguages } from '../../../../snyk/common/vscode/languages';
+import { OssCodeActionsProvider } from '../../../../snyk/snykOss/providers/ossCodeActionsProvider';
+
+suite('OSS code actions provider', () => {
+  let ossActionsProvider: OssCodeActionsProvider;
+  let logQuickFixIsDisplayed: sinon.SinonSpy;
+  let rangeMock: Range;
+
+  setup(() => {
+    const ossResults = new Map<string, WorkspaceFolderResult<OssIssueData>>();
+    ossResults.set('folderName', [
+      {
+        filePath: '//folderName//package.json',
+        additionalData: {},
+      } as unknown as Issue<OssIssueData>,
+    ]);
+
+    logQuickFixIsDisplayed = sinon.fake();
+    const analytics = {
+      logQuickFixIsDisplayed,
+    } as unknown as IAnalytics;
+
+    const codeActionAdapter = {
+      create: (_: string, _kind?: CodeActionKind) => ({
+        command: {},
+      }),
+    } as ICodeActionAdapter;
+
+    const codeActionKindAdapter = {
+      getQuickFix: sinon.fake(),
+    } as ICodeActionKindAdapter;
+
+    rangeMock = {
+      contains: () => true,
+    } as unknown as Range;
+
+    ossActionsProvider = new OssCodeActionsProvider(
+      {} as IVSCodeLanguages,
+      codeActionAdapter,
+      codeActionKindAdapter,
+      ossResults,
+      analytics,
+    );
+  });
+
+  teardown(() => {
+    sinon.restore();
+  });
+
+  test('Provides the most severe vulnerability CodeAction', () => {
+    // arrange
+    const document = {
+      uri: {
+        fsPath: '//folderName//package.json',
+      },
+    } as unknown as TextDocument;
+
+    const clickedRange = {} as Range;
+    const context = {} as CodeActionContext;
+
+    const vulnerabilities = [
+      {
+        id: 'vulnerability1',
+        severity: IssueSeverity.High,
+      },
+      {
+        id: 'vulnerability2',
+        severity: IssueSeverity.Medium,
+      },
+      {
+        id: 'vulnerability3',
+        severity: IssueSeverity.Critical,
+      },
+    ] as Issue<OssIssueData>[];
+
+    const mostSevereVulnerability = {
+      id: 'vulnerability3',
+      severity: IssueSeverity.Critical,
+    } as Issue<OssIssueData>;
+
+    const codeActions = [
+      {
+        command: {
+          command: SNYK_OPEN_ISSUE_COMMAND,
+          title: SNYK_OPEN_ISSUE_COMMAND,
+          arguments: [
+            {
+              issueType: OpenCommandIssueType.OssVulnerability,
+              issue: mostSevereVulnerability,
+            } as OpenIssueCommandArg,
+          ],
+        },
+      },
+    ] as CodeAction[];
+
+    sinon.stub(ossActionsProvider, 'getIssueRange').returns(rangeMock);
+    // stubbing private methods workaround is to case to any
+    sinon.stub(ossActionsProvider, <any>'getVulnerabilities').returns(vulnerabilities);
+    sinon.stub(ossActionsProvider, <any>'getMostSevereVulnerability').returns(mostSevereVulnerability);
+    sinon.stub(ossActionsProvider, <any>'getActions').returns(codeActions);
+
+    // act
+    const result = ossActionsProvider.provideCodeActions(document, clickedRange, context);
+
+    // assert
+    sinon.assert.calledOnce(logQuickFixIsDisplayed);
+    assert.deepStrictEqual(result, codeActions);
+  });
+});