fix: send additional report options to analyzeFolders

snyk · Apr 4, 2023 · f9eaf43 · f9eaf43
1 parent e7ada14
commit f9eaf43
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 0 deletions.
diff --git a/src/lib/plugins/sast/analysis.ts b/src/lib/plugins/sast/analysis.ts
@@ -111,7 +111,9 @@ async function getCodeAnalysis(
       reportOptions: {
         enabled: options.report ?? false,
         projectName: options['project-name'],
+        targetName: options['target-name'],
         targetRef: options['target-reference'],
+        remoteRepoUrl: options['remote-repo-url'],
       },
     }),
     analysisContext: {

diff --git a/test/jest/unit/snyk-code/snyk-code-test.spec.ts b/test/jest/unit/snyk-code/snyk-code-test.spec.ts
@@ -829,6 +829,13 @@ describe('Test snyk code', () => {
       localCodeEngine: { url: '', allowCloudUpload: true, enabled: false },
     };
 
+    const reportOptions = {
+      projectName: 'test-project-name',
+      targetName: 'test-target-name',
+      targetRef: 'test-target-ref',
+      remoteRepoUrl: 'https://github.com/owner/repo',
+    };
+
     analyzeFoldersMock.mockResolvedValue(
       sampleAnalyzeFoldersWithReportAndIgnoresResponse,
     );
@@ -837,11 +844,25 @@ describe('Test snyk code', () => {
       {
         path: '',
         code: true,
+        report: true,
+        'project-name': reportOptions.projectName,
+        'target-name': reportOptions.targetName,
+        'target-reference': reportOptions.targetRef,
+        'remote-repo-url': reportOptions.remoteRepoUrl,
       },
       sastSettings,
       'test-id',
     );
 
+    expect(analyzeFoldersMock).toHaveBeenCalledWith(
+      expect.objectContaining({
+        reportOptions: {
+          enabled: true,
+          ...reportOptions,
+        },
+      }),
+    );
+
     const expectedReportResults = {
       projectId: 'test-project-id',
       snapshotId: 'test-snapshot-id',