Adds integration test and fixes linting

github · cw-alexcroteau · Feb 1, 2022 · Jan 24, 2022 · Jan 25, 2022 · Jan 25, 2022
commit 1bfa9ace605d639a8b63a49326a46e5c31d5cf2a
diff --git a/.github/workflows/__ref-commit-input.yml b/.github/workflows/__ref-commit-input.yml
diff --git a/pr-checks/checks/ref-commit-input.yml b/pr-checks/checks/ref-commit-input.yml
@@ -0,0 +1,23 @@
+name: "'ref' and 'sha' from inputs"
+description: "Checks that specifying 'ref' and 'sha' as inputs works"
+steps:
+  - uses: ./../action/init
+    with:
+      tools: ${{ steps.prepare-test.outputs.tools-url }}
+      languages: cpp,csharp,java,javascript,python
+      config-file: github/codeql-action/tests/multi-language-repo/.github/codeql/custom-queries.yml@${{ github.sha }}
+  - name: Build code
+    shell: bash
+    run: ./build.sh
+  - uses: ./../action/analyze
+    with:
+      ref: 'refs/heads/main'
+      sha: 'dc1e2655b666a6d3624deba0983c9e6105d92dd4'
+    env:
+      TEST_MODE: true
+  - uses: ./../action/upload-sarif
+    with:
+      ref: 'refs/heads/main'
+      sha: 'dc1e2655b666a6d3624deba0983c9e6105d92dd4'
+    env:
+      TEST_MODE: true
diff --git a/src/analyze-action.ts b/src/analyze-action.ts
@@ -187,7 +187,7 @@ async function run() {
         apiDetails,
         logger
       );
-      core.setOutput('sarif-id', uploadResult.sarifID);
+      core.setOutput("sarif-id", uploadResult.sarifID);
     } else {
       logger.info("Not uploading results");
     }

diff --git a/src/upload-sarif-action.ts b/src/upload-sarif-action.ts
@@ -63,7 +63,7 @@ async function run() {
       apiDetails,
       getActionsLogger()
     );
-    core.setOutput('sarif-id', uploadResult.sarifID);
+    core.setOutput("sarif-id", uploadResult.sarifID);
     if (actionsUtil.getRequiredInput("wait-for-processing") === "true") {
       await upload_lib.waitForProcessing(
         parseRepositoryNwo(getRequiredEnvParam("GITHUB_REPOSITORY")),