dev(trdl-actions): e2e-test install action

Signed-off-by: Alexandr Zaytsev <[email protected]>

Author: nervgh

.github/workflows/_lint.yaml

@@ -3,7 +3,7 @@ name: Lint
 on: [push]
 
 jobs:
-  _:
+  lint:
     runs-on: ubuntu-22.04
     timeout-minutes: 10
 

.github/workflows/_test_e2e_install.yaml

@@ -0,0 +1,21 @@
+name: Test e2e Install
+
+on: [push]
+
+jobs:
+  test-e2e-install:
+    runs-on: ubuntu-22.04
+    timeout-minutes: 10
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Install trdl
+        uses: ./install
+
+      - name: Use trdl binary
+        run: |
+          which trdl
+          ls -l $(which trdl)
+          trdl --help

.github/workflows/_test_unit.yaml

@@ -3,7 +3,7 @@ name: Test Unit
 on: [push]
 
 jobs:
-  _:
+  test-unit:
     runs-on: ubuntu-22.04
     timeout-minutes: 10
 

install/dist/index.mjs

@@ -28,6 +28,7 @@ import require$$0$9 from 'diagnostics_channel';
 import require$$2$2 from 'child_process';
 import require$$6$1 from 'timers';
 import { chmodSync } from 'node:fs';
+import { join } from 'node:path';
 
 var commonjsGlobal = typeof globalThis !== 'undefined' ? globalThis : typeof window !== 'undefined' ? window : typeof global !== 'undefined' ? global : typeof self !== 'undefined' ? self : {};
 
@@ -29857,10 +29858,10 @@ class GpgCli {
         }
     }
     async import(ascPath) {
-        await execOutput(this.name, ['--import', ascPath]);
+        await execOutput(this.name, ['--import', ascPath], { silent: false, failOnStdErr: false });
     }
     async verify(sigPath, binPath) {
-        await execOutput(this.name, ['--verify', sigPath, binPath]);
+        await execOutput(this.name, ['--verify', sigPath, binPath], { silent: false, failOnStdErr: false });
     }
     async help() {
         const { stdout } = await execOutput(this.name, ['--help']);
@@ -29914,6 +29915,10 @@ class TrdlCli {
         const { stdout } = await execOutput(this.name, ['list']);
         return stdout.slice(1).map(parseLineToItem);
     }
+    async version() {
+        const { stdout } = await execOutput(this.name, ['version'], { silent: false });
+        return stdout.join('');
+    }
 }
 function parseLineToItem(line) {
     const [name, url, default_, channel] = line.trim().split(/ +/);
@@ -29998,27 +30003,30 @@ async function downloadParallel(binUrl, sigUrl, ascUrl) {
 function findTrdlCache(toolName, toolVersion) {
     return toolCacheExports.find(toolName, toolVersion);
 }
-async function installTrdl(toolName, toolVersion, binPath) {
+async function installTrdl(binPath, toolName, toolVersion) {
     // install tool
-    const installedPath = await toolCacheExports.cacheFile(binPath, toolName, toolName, toolVersion);
+    const cachedPath = await toolCacheExports.cacheFile(binPath, toolName, toolName, toolVersion);
+    const cachedFile = join(cachedPath, toolName);
+    coreExports.info(format('cached file=%s', cachedFile));
     // set permissions
-    chmodSync(installedPath, 0o755);
+    chmodSync(cachedFile, 0o755);
     // add tool to $PATH
-    coreExports.addPath(installedPath);
+    coreExports.addPath(cachedFile);
 }
 async function Run() {
+    coreExports.setCommandEcho(coreExports.isDebug());
     const trdlCli = new TrdlCli();
     const gpgCli = new GpgCli();
     const inputs = parseInputs();
     await Do(trdlCli, gpgCli, inputs);
 }
 async function Do(trdlCli, gpgCli, inputs) {
     coreExports.startGroup('Install or self-update trdl.');
-    coreExports.debug(format(`parsed inputs=%o`, inputs));
+    coreExports.info(format(`parsed inputs=%o`, inputs));
     const defaults = trdlCli.defaults();
-    coreExports.debug(format(`trdl defaults=%o`, defaults));
+    coreExports.info(format(`trdl defaults=%o`, defaults));
     const options = await getOptions(inputs, defaults);
-    coreExports.debug(format(`installation options=%o`, options));
+    coreExports.info(format(`installation options=%o`, options));
     const toolCache = findTrdlCache(defaults.repo, options.version);
     if (toolCache) {
         coreExports.info(`Installation skipped. trdl@v${options.version} is found at path ${toolCache}.`);
@@ -30030,16 +30038,19 @@ async function Do(trdlCli, gpgCli, inputs) {
     }
     await gpgCli.mustGnuGP();
     const [binUrl, sigUrl, ascUrl] = formatDownloadUrls(options.version);
-    coreExports.debug(format('%s bin_url=%s', defaults.repo, binUrl));
-    coreExports.debug(format('%s sig_url=%s', defaults.repo, sigUrl));
-    coreExports.debug(format('%s asc_url=%s', defaults.repo, ascUrl));
-    coreExports.info('Downloading signatures.');
+    coreExports.info(format('%s binUrl=%s', defaults.repo, binUrl));
+    coreExports.info(format('%s sigUrl=%s', defaults.repo, sigUrl));
+    coreExports.info(format('%s ascUrl=%s', defaults.repo, ascUrl));
+    coreExports.info('Downloading binary and signatures.');
     const [binPath, sigPath, ascPath] = await downloadParallel(binUrl, sigUrl, ascUrl);
     coreExports.info('Importing and verifying gpg keys.');
     await gpgCli.import(ascPath);
     await gpgCli.verify(sigPath, binPath);
     coreExports.info('Installing trdl and adding it to the $PATH.');
-    await installTrdl(defaults.repo, options.version, binPath);
+    await installTrdl(binPath, defaults.repo, options.version);
+    coreExports.info('Showing installed version.');
+    const version = await trdlCli.version();
+    coreExports.info(format('Installed version: trdl@%s', version));
     coreExports.endGroup();
 }
 

install/dist/index.mjs.map

@@ -4,6 +4,7 @@ import * as toolCache from '../../test/mocks/tool-cache'
 import * as fs from '../../test/mocks/fs'
 import { trdlCli } from '../../test/mocks/trdl-cli'
 import { gpgCli } from '../../test/mocks/gpg-cli'
+import { join } from 'node:path'
 
 // Mocks should be declared before the module being tested is imported.
 jest.unstable_mockModule('@actions/core', () => core)
@@ -86,16 +87,16 @@ describe('install/action.ts', function () {
       expect(trdlCli.update).toHaveBeenCalledWith(defaults)
     })
     it('should install trdl if tool cache is not found', async function () {
-      const binPath = 'bin path'
+      const binPath = '/tmp/cache/path'
       const sigPath = 'sig path'
       const ascPath = 'asc path'
 
       toolCache.downloadTool.mockResolvedValueOnce(binPath)
       toolCache.downloadTool.mockResolvedValueOnce(sigPath)
       toolCache.downloadTool.mockResolvedValueOnce(ascPath)
 
-      const installedPath = '/tmp/installed/path'
-      toolCache.cacheFile.mockResolvedValueOnce(installedPath)
+      const cachedPath = '/tmp/installed/path'
+      toolCache.cacheFile.mockResolvedValueOnce(cachedPath)
 
       await Do(trdlCli, gpgCli, inputs)
 
@@ -104,8 +105,9 @@ describe('install/action.ts', function () {
       expect(gpgCli.import).toHaveBeenCalledWith(ascPath)
       expect(gpgCli.verify).toHaveBeenCalledWith(sigPath, binPath)
       expect(toolCache.cacheFile).toHaveBeenCalledWith(binPath, defaults.repo, defaults.repo, inputs.version)
-      expect(fs.chmodSync).toHaveBeenCalledWith(installedPath, 0o755)
-      expect(core.addPath).toHaveBeenCalledWith(installedPath)
+      expect(fs.chmodSync).toHaveBeenCalledWith(join(cachedPath, defaults.repo), 0o755)
+      expect(core.addPath).toHaveBeenCalledWith(join(cachedPath, defaults.repo))
+      expect(trdlCli.version).toHaveBeenCalled()
     })
   })
 })

install/src/action.test.ts

@@ -1,7 +1,8 @@
-import { getInput, platform, addPath, info, startGroup, endGroup, debug } from '@actions/core'
+import { getInput, platform, addPath, info, startGroup, endGroup, setCommandEcho, isDebug } from '@actions/core'
 import { HttpClient } from '@actions/http-client'
 import { downloadTool, find, cacheFile } from '@actions/tool-cache'
 import { chmodSync } from 'node:fs'
+import { join } from 'node:path'
 import { GpgCli } from '../../lib/gpg-cli'
 import { Defaults, TrdlCli } from '../../lib/trdl-cli'
 import { format } from 'util'
@@ -90,16 +91,20 @@ function findTrdlCache(toolName: string, toolVersion: string): string {
   return find(toolName, toolVersion)
 }
 
-async function installTrdl(toolName: string, toolVersion: string, binPath: string): Promise<void> {
+async function installTrdl(binPath: string, toolName: string, toolVersion: string): Promise<void> {
   // install tool
-  const installedPath = await cacheFile(binPath, toolName, toolName, toolVersion)
+  const cachedPath = await cacheFile(binPath, toolName, toolName, toolVersion)
+  const cachedFile = join(cachedPath, toolName)
+  info(format('cached file=%s', cachedFile))
   // set permissions
-  chmodSync(installedPath, 0o755)
+  chmodSync(cachedFile, 0o755)
   // add tool to $PATH
-  addPath(installedPath)
+  addPath(cachedFile)
 }
 
 export async function Run(): Promise<void> {
+  setCommandEcho(isDebug())
+
   const trdlCli = new TrdlCli()
   const gpgCli = new GpgCli()
   const inputs = parseInputs()
@@ -109,13 +114,13 @@ export async function Run(): Promise<void> {
 
 export async function Do(trdlCli: TrdlCli, gpgCli: GpgCli, inputs: inputs): Promise<void> {
   startGroup('Install or self-update trdl.')
-  debug(format(`parsed inputs=%o`, inputs))
+  info(format(`parsed inputs=%o`, inputs))
 
   const defaults = trdlCli.defaults()
-  debug(format(`trdl defaults=%o`, defaults))
+  info(format(`trdl defaults=%o`, defaults))
 
   const options = await getOptions(inputs, defaults)
-  debug(format(`installation options=%o`, options))
+  info(format(`installation options=%o`, options))
 
   const toolCache = findTrdlCache(defaults.repo, options.version)
 
@@ -133,18 +138,22 @@ export async function Do(trdlCli: TrdlCli, gpgCli: GpgCli, inputs: inputs): Prom
   await gpgCli.mustGnuGP()
 
   const [binUrl, sigUrl, ascUrl] = formatDownloadUrls(options.version)
-  debug(format('%s bin_url=%s', defaults.repo, binUrl))
-  debug(format('%s sig_url=%s', defaults.repo, sigUrl))
-  debug(format('%s asc_url=%s', defaults.repo, ascUrl))
+  info(format('%s binUrl=%s', defaults.repo, binUrl))
+  info(format('%s sigUrl=%s', defaults.repo, sigUrl))
+  info(format('%s ascUrl=%s', defaults.repo, ascUrl))
 
-  info('Downloading signatures.')
+  info('Downloading binary and signatures.')
   const [binPath, sigPath, ascPath] = await downloadParallel(binUrl, sigUrl, ascUrl)
 
   info('Importing and verifying gpg keys.')
   await gpgCli.import(ascPath)
   await gpgCli.verify(sigPath, binPath)
 
   info('Installing trdl and adding it to the $PATH.')
-  await installTrdl(defaults.repo, options.version, binPath)
+  await installTrdl(binPath, defaults.repo, options.version)
+
+  info('Showing installed version.')
+  const version = await trdlCli.version()
+  info(format('Installed version: trdl@%s', version))
   endGroup()
 }

install/src/action.ts

@@ -23,7 +23,10 @@ describe('gpg-cli.ts', function () {
       const ascPath = 'asc path'
       const result = await cli.import(ascPath)
       expect(result).toBeUndefined()
-      expect(libExec.execOutput).toHaveBeenCalledWith(cliName, ['--import', ascPath])
+      expect(libExec.execOutput).toHaveBeenCalledWith(cliName, ['--import', ascPath], {
+        silent: false,
+        failOnStdErr: false
+      })
     })
   })
 
@@ -33,7 +36,10 @@ describe('gpg-cli.ts', function () {
       const binPath = 'bin path'
       const result = await cli.verify(sigPath, binPath)
       expect(result).toBeUndefined()
-      expect(libExec.execOutput).toHaveBeenCalledWith(cliName, ['--verify', sigPath, binPath])
+      expect(libExec.execOutput).toHaveBeenCalledWith(cliName, ['--verify', sigPath, binPath], {
+        silent: false,
+        failOnStdErr: false
+      })
     })
   })
 

lib/gpg-cli.test.ts

@@ -15,11 +15,11 @@ export class GpgCli {
   }
 
   async import(ascPath: string): Promise<void> {
-    await execOutput(this.name, ['--import', ascPath])
+    await execOutput(this.name, ['--import', ascPath], { silent: false, failOnStdErr: false })
   }
 
   async verify(sigPath: string, binPath: string): Promise<void> {
-    await execOutput(this.name, ['--verify', sigPath, binPath])
+    await execOutput(this.name, ['--verify', sigPath, binPath], { silent: false, failOnStdErr: false })
   }
 
   async help(): Promise<string> {

lib/gpg-cli.ts

@@ -157,4 +157,13 @@ describe('trdl-cli.ts', function () {
       expect(libExec.execOutput).toHaveBeenCalledWith(cliName, ['list'])
     })
   })
+  describe('version', function () {
+    it('should work', async function () {
+      const stdout: string[] = ['1.2.3']
+      libExec.execOutput.mockResolvedValueOnce({ stdout, stderr: [], exitCode: 0 })
+      const result = await cli.version()
+      expect(result).toEqual(stdout.join(''))
+      expect(libExec.execOutput).toHaveBeenCalledWith(cliName, ['version'], { silent: false })
+    })
+  })
 })