Fix echo output in check-action.yml and update logging messages in ut…

…ils.ts

.github/workflows/check-action.yml

@@ -26,7 +26,7 @@ jobs:
 
       - name: Echo output of installer
         if: always()
-        run: echo "${{ steps.install_kit.outputs }}"
+        run: echo "${{ steps.install_kit.outputs.kit-path }}"
 
       - name: Run kit tools
         shell: bash
@@ -52,7 +52,7 @@ jobs:
 
       - name: Echo output of installer
         if: always()
-        run: echo "${{ steps.install_kit.outputs }}"
+        run: echo "${{ steps.install_kit.outputs.kit-path }}"
 
       - name: Run installed tools
         shell: bash

__tests__/kit-release.test.ts

@@ -0,0 +1,38 @@
+import { findMatchingRelease, getReleases } from '../src/releases/kit-release'
+
+describe('getReleases', () => {
+  beforeEach(() => {
+    jest.clearAllMocks()
+  })
+
+  it('should retrieve releases successfully', async () => {
+    const release = await getReleases(getToken(), false)
+    expect(release).toBeDefined()
+    expect(release.length).toBeGreaterThan(0)
+  })
+
+  it('should retrieve latest release successfully', async () => {
+    const release = await getReleases(getToken(), true)
+    expect(release).toBeDefined()
+    expect(release.length).toBe(1)
+  })
+
+  it('should find matching release', async () => {
+    const releases = await getReleases(getToken(), false)
+    const matchingRelease = findMatchingRelease(releases, 'v0.1.0')
+    expect(matchingRelease).toBeDefined()
+    expect(matchingRelease?.tag).toBe('v0.1.0')
+  })
+})
+
+function getToken(): string {
+  const token = process.env['GITHUB_TOKEN'] || ''
+  if (!token) {
+    /* eslint-disable-next-line no-console */
+    console.warn(
+      'Skipping GitHub tests. Set $GITHUB_TOKEN to run GitHub tests.'
+    )
+  }
+
+  return token
+}

__tests__/main.test.ts

@@ -12,78 +12,29 @@ import * as main from '../src/main'
 // Mock the action's main function
 const runMock = jest.spyOn(main, 'run')
 
-// Other utilities
-const timeRegex = /^\d{2}:\d{2}:\d{2}/
-
 // Mock the GitHub Actions core library
-let debugMock: jest.SpiedFunction<typeof core.debug>
-let errorMock: jest.SpiedFunction<typeof core.error>
 let getInputMock: jest.SpiedFunction<typeof core.getInput>
-let setFailedMock: jest.SpiedFunction<typeof core.setFailed>
-let setOutputMock: jest.SpiedFunction<typeof core.setOutput>
 
 describe('action', () => {
   beforeEach(() => {
     jest.clearAllMocks()
 
-    debugMock = jest.spyOn(core, 'debug').mockImplementation()
-    errorMock = jest.spyOn(core, 'error').mockImplementation()
     getInputMock = jest.spyOn(core, 'getInput').mockImplementation()
-    setFailedMock = jest.spyOn(core, 'setFailed').mockImplementation()
-    setOutputMock = jest.spyOn(core, 'setOutput').mockImplementation()
-  })
-
-  it('sets the time output', async () => {
-    // Set the action's inputs as return values from core.getInput()
-    getInputMock.mockImplementation(name => {
-      switch (name) {
-        case 'milliseconds':
-          return '500'
-        default:
-          return ''
-      }
-    })
-
-    await main.run()
-    expect(runMock).toHaveReturned()
-
-    // Verify that all of the core library functions were called correctly
-    expect(debugMock).toHaveBeenNthCalledWith(1, 'Waiting 500 milliseconds ...')
-    expect(debugMock).toHaveBeenNthCalledWith(
-      2,
-      expect.stringMatching(timeRegex)
-    )
-    expect(debugMock).toHaveBeenNthCalledWith(
-      3,
-      expect.stringMatching(timeRegex)
-    )
-    expect(setOutputMock).toHaveBeenNthCalledWith(
-      1,
-      'time',
-      expect.stringMatching(timeRegex)
-    )
-    expect(errorMock).not.toHaveBeenCalled()
   })
 
-  it('sets a failed status', async () => {
-    // Set the action's inputs as return values from core.getInput()
+  it('runs main', async () => {
     getInputMock.mockImplementation(name => {
       switch (name) {
-        case 'milliseconds':
-          return 'this is not a number'
+        case 'token':
+          return process.env['GITHUB_TOKEN'] || ''
+        case 'version':
+          return 'latest'
         default:
           return ''
       }
     })
 
     await main.run()
     expect(runMock).toHaveReturned()
-
-    // Verify that all of the core library functions were called correctly
-    expect(setFailedMock).toHaveBeenNthCalledWith(
-      1,
-      'milliseconds not a number'
-    )
-    expect(errorMock).not.toHaveBeenCalled()
   })
 })

src/installer/download.ts

@@ -10,9 +10,6 @@ import { getTmpDir } from '../utils/utils'
  * @returns The path the file was downloaded to.
  */
 export async function downloadFile(file: KitArchiveFile): Promise<string> {
-  // tool-cache download downloads to /tmp/<guid> to prevent collisions.
-  // we mimic that behaviour here but keep the file's name so it has the correct extension
-  // a GUID is 128 bits = 16 bytes - this one has no hyphens but it serves the same purpose.
   const guid = crypto.randomBytes(16).toString('hex')
   const filename = `${guid}-${file.archiveFilename}`
   const dlStartTime = Date.now()
@@ -23,7 +20,7 @@ export async function downloadFile(file: KitArchiveFile): Promise<string> {
   ghCore.debug(`Downloaded to ${downloadPath}`)
   const elapsed = Date.now() - dlStartTime
   ghCore.info(
-    `Downloaded ${file.archiveFilename} in ${(elapsed / 1000).toFixed(1)}s`
+    `📦 Downloaded ${file.archiveFilename} in ${(elapsed / 1000).toFixed(1)}s`
   )
   ghToolCache.isExplicitVersion
   return downloadPath

src/installer/hash.ts

@@ -39,7 +39,7 @@ export async function verifyHash(
     )
   }
 
-  ghCore.info(`sha256 verification of ${downloadedArchivePath} succeeded.`)
+  ghCore.info(`✅ sha256 verification of ${downloadedArchivePath} succeeded.`)
 }
 
 /**

src/utils/utils.ts

@@ -31,7 +31,7 @@ export function getOS(): OS {
       default:
         currentOS = 'linux'
     }
-    ghCore.info(`Current operating system is ${currentOS}`)
+    ghCore.debug(`Current operating system is ${currentOS}`)
   }
   return currentOS
 }
@@ -56,7 +56,7 @@ export function getArch(): string {
       default:
         currentArch = 'x86_64'
     }
-    ghCore.info(`Current architecture is ${currentArch}`)
+    ghCore.debug(`Current architecture is ${currentArch}`)
   }
   return currentArch
 }