Cleanup (#9)

.github/workflows/pr.yml

@@ -12,6 +12,20 @@ jobs:
           npm install
       - run: |
           npm run all
+      - name: Compare the expected and actual dist/ directories
+        run: |
+          if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
+            echo "Detected uncommitted changes after build.  See status below:"
+            git diff
+            exit 1
+          fi
+        id: diff
+      # If index.js was different than expected, upload the expected version as an artifact
+      - uses: actions/upload-artifact@v2
+        if: ${{ failure() && steps.diff.conclusion == 'failure' }}
+        with:
+          name: dist
+          path: dist/
   test-action:
     needs: test-code
     environment: foo

dist/index.js

@@ -58,7 +58,7 @@ function getDeploymentsWaitingFor(octokit, context, inputs, jobId) {
                 .filter(status => !status.log_url.includes(jobId.toString()))
                 .filter(status => new Date(status.created_at) < new Date(currentDeploymentStatus.created_at));
         }
-        core.info(`found ${waitingFor.length} statuses to wait for`);
+        core.info(`found ${waitingFor.length} deployments to wait for`);
         core.debug(util.inspect(waitingFor.map(status => ({ id: status.id, name: status.state }))));
         return waitingFor;
     });
@@ -205,6 +205,8 @@ function queueWorkflow(inputs) {
     });
 }
 exports.queueWorkflow = queueWorkflow;
+function someName(params) {
+}
 
 
 /***/ }),

lib/getDeploymentsWaitingFor.js

@@ -51,7 +51,7 @@ function getDeploymentsWaitingFor(octokit, context, inputs, jobId) {
                 .filter(status => !status.log_url.includes(jobId.toString()))
                 .filter(status => new Date(status.created_at) < new Date(currentDeploymentStatus.created_at));
         }
-        core.info(`found ${waitingFor.length} statuses to wait for`);
+        core.info(`found ${waitingFor.length} deployments to wait for`);
         core.debug(util.inspect(waitingFor.map(status => ({ id: status.id, name: status.state }))));
         return waitingFor;
     });

lib/queueWorkflow.js

@@ -79,3 +79,5 @@ function queueWorkflow(inputs) {
     });
 }
 exports.queueWorkflow = queueWorkflow;
+function someName(params) {
+}

src/getDeploymentsWaitingFor.ts

@@ -36,7 +36,7 @@ export async function getDeploymentsWaitingFor(octokit: Octokit, context: Contex
             .filter(status => new Date(status.created_at) < new Date(currentDeploymentStatus.created_at))
     }
 
-    core.info(`found ${waitingFor.length} statuses to wait for`)
+    core.info(`found ${waitingFor.length} deployments to wait for`)
     core.debug(util.inspect(waitingFor.map(status => ({ id: status.id, name: status.state }))))
 
     return waitingFor

src/queueWorkflow.ts

@@ -61,4 +61,6 @@ export async function queueWorkflow(inputs: Inputs) {
     core.info(latestQueueStatus);
 }
 
-
+function someName(params:string) {
+
+}

tests/index.test.ts

@@ -70,7 +70,7 @@ test('queueWorkflow: nothing to queue', async () => {
 
     let msg = ''
     jest.spyOn(core, 'info').mockImplementation((messsage: string) => {
-        // eat the cals so they don't show up in the test results on the runner
+        // spy on the calls so they don't show up in the test results on the runner
     })
 
     let mockExit = mockProcess.mockProcessExit()
@@ -127,7 +127,7 @@ test('queueWorkflow: deployments to queue', async () => {
     }
 
     jest.spyOn(core, 'info').mockImplementation(() => {
-        // eat the cals so they don't show up in the test results on the runner
+        // spy on the calls so they don't show up in the test results on the runner
     })
 
     // @ts-ignore
@@ -202,7 +202,7 @@ test('queueWorkflow: timed out', async () => {
     })
     let mockExit = mockProcess.mockProcessExit()
     jest.spyOn(core, 'setFailed').mockImplementation((messsage: any) => {
-        // eat the cals so they don't show up in the test results on the runner
+        // spy on the calls so they don't show up in the test results on the runner
     })
     await queueWorkflow(inputs)
     expect(mockExit).toHaveBeenCalledWith(1)