fix tests

jfrog · Sep 4, 2024 · 654f779 · 654f779
1 parent 2a21e7b
commit 654f779
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/lib/utils.js b/lib/utils.js
@@ -328,14 +328,16 @@ class Utils {
         if (projectKey) {
             Utils.exportVariableIfNotSet('JFROG_CLI_BUILD_PROJECT', projectKey);
         }
-        // Enable command summaries to allow generating job summary and code scanning sarif.
-        Utils.enableCommandSummaries();
+        // Enable job summaries if disable was not requested.
+        if (!core.getBooleanInput(Utils.JOB_SUMMARY_DISABLE)) {
+            Utils.enableJobSummaries();
+        }
     }
     /**
      * Enabling job summary is done by setting the output dir for the summaries.
      * If the output dir is not set, the CLI won't generate the summary Markdown files.
      */
-    static enableCommandSummaries() {
+    static enableJobSummaries() {
         let tempDir = this.getTempDirectory();
         Utils.exportVariableIfNotSet(Utils.JFROG_CLI_COMMAND_SUMMARY_OUTPUT_DIR_ENV, tempDir);
     }

diff --git a/src/utils.ts b/src/utils.ts
@@ -375,15 +375,17 @@ export class Utils {
             Utils.exportVariableIfNotSet('JFROG_CLI_BUILD_PROJECT', projectKey);
         }
 
-        // Enable command summaries to allow generating job summary and code scanning sarif.
-        Utils.enableCommandSummaries();
+        // Enable job summaries if disable was not requested.
+        if (!core.getBooleanInput(Utils.JOB_SUMMARY_DISABLE)) {
+            Utils.enableJobSummaries();
+        }
     }
 
     /**
      * Enabling job summary is done by setting the output dir for the summaries.
      * If the output dir is not set, the CLI won't generate the summary Markdown files.
      */
-    private static enableCommandSummaries() {
+    private static enableJobSummaries() {
         let tempDir: string = this.getTempDirectory();
         Utils.exportVariableIfNotSet(Utils.JFROG_CLI_COMMAND_SUMMARY_OUTPUT_DIR_ENV, tempDir);
     }