reuse the same code in downloadArtifact

Author: jeremydubreil

src/actions.ts

@@ -3,24 +3,29 @@ import { endGroup, getInput, startGroup } from '@actions/core'
 import { context, getOctokit } from '@actions/github'
 import { retry } from '@octokit/plugin-retry'
 import { Md5 } from 'ts-md5'
-import { DefaultArtifactClient, UploadArtifactResponse } from '@actions/artifact'
+import {
+  Artifact,
+  DefaultArtifactClient,
+  GetArtifactResponse,
+  UploadArtifactResponse,
+} from '@actions/artifact'
 
 const artifact = new DefaultArtifactClient()
 
-export function uploadArtifact(
-  artifactName: string,
-  ...files: string[]
-): Promise<UploadArtifactResponse> {
+export async function uploadArtifact(artifactName: string, ...files: string[]) {
   startGroup('Uploading artifact ' + artifactName)
-  var reponse = artifact.uploadArtifact(artifactName, files, '.')
+  await artifact.uploadArtifact(artifactName, files, '.')
   endGroup()
-  return reponse
 }
 
-export async function downloadArtifact(artifactName: string) {
+export async function downloadArtifact(artifactName: string): Promise<Artifact> {
   startGroup('Downloading artifact ' + artifactName)
-  await create.getArtifact(artifactName)
+  var reponse = await artifact.getArtifact(artifactName)
+  if (reponse === undefined) {
+    throw new Error('Artifact not found')
+  }
   endGroup()
+  return reponse.artifact
 }
 
 export async function postCommentIfInPr(message: string): Promise<string | undefined> {

src/index.ts

@@ -89,12 +89,14 @@ async function runAnalysis() {
 async function displayResults() {
   info('Displaying results')
   const downloadStart = Date.now()
-  await downloadArtifact('results-old')
-  await downloadArtifact('results-new')
+  var artifactOld = await downloadArtifact('results-old')
+  var artifactNew = await downloadArtifact('results-new')
   telemetryCollector.addField(
     'duration.download-artifacts',
     (Date.now() - downloadStart).toString()
   )
+  console.log('Artifact old: ', artifactOld)
+  console.log('Artifact new: ', artifactNew)
   const issuesByTool: { [tool: string]: string } = {}
   if (existsSync(`results-old/${scaReport}`) && existsSync(`results-new/${scaReport}`)) {
     issuesByTool['sca'] = await compareResults(