dev(trdl-actions): e2e-test action setup-app manually

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

Author: nervgh

.github/workflows/_test_e2e_install.yaml

@@ -1,4 +1,4 @@
-name: Test e2e Install
+name: Test e2e install
 
 on: [push]
 

.github/workflows/_test_e2e_setup-app_manual.yaml

@@ -0,0 +1,25 @@
+name: Test e2e setup-app manually
+
+on: [push]
+
+jobs:
+  test-e2e-setup-app-manually:
+    runs-on: ubuntu-22.04
+    timeout-minutes: 10
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup application
+        uses: ./setup-app
+        with:
+          repo: kubedog
+          url: https://tuf.kubedog.werf.io
+          root-version: 12
+          root-sha512: e1d3c7bcfdf473fe1466c5e9d9030bea0fed857d0563db1407754d2795256e4d063b099156807346cdcdc21d747326cc43f96fa2cacda5f1c67c8349fe09894d
+          group: stable
+
+      - name: Use kubedog binary
+        run: |
+          kubedog --help

install/dist/index.mjs

@@ -30020,7 +30020,7 @@ async function Run() {
 }
 async function Do(trdlCli, gpgCli, inputs) {
     coreExports.startGroup(`Install or self-update ${trdlCli.name}.`);
-    coreExports.info(format(`parsed inputs=%o`, inputs));
+    coreExports.info(format(`Parsed inputs=%o`, inputs));
     const defaults = trdlCli.defaults();
     coreExports.info(format(`${trdlCli.name} repository defaults=%o`, defaults));
     const options = await getOptions(inputs, defaults);

install/dist/index.mjs.map

@@ -111,7 +111,7 @@ export async function Run(): Promise<void> {
 
 export async function Do(trdlCli: TrdlCli, gpgCli: GpgCli, inputs: inputs): Promise<void> {
   startGroup(`Install or self-update ${trdlCli.name}.`)
-  info(format(`parsed inputs=%o`, inputs))
+  info(format(`Parsed inputs=%o`, inputs))
 
   const defaults = trdlCli.defaults()
   info(format(`${trdlCli.name} repository defaults=%o`, defaults))

install/src/action.ts

@@ -163,7 +163,7 @@ describe('trdl-cli.ts', function () {
       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 })
+      expect(libExec.execOutput).toHaveBeenCalledWith(cliName, ['version'])
     })
   })
 })

lib/trdl-cli.test.ts

@@ -11,23 +11,24 @@ inputs:
       arguments. For example: werf'
     required: false
   force:
-    description: 'Perform operation anyway if set to true. For both "trdl add" and "trdl update"'
+    description: 'Perform operation anyway if set to true. For both "trdl add" and "trdl update".'
     required: false
     default: 'true'
   repo:
-    description: 'The certain repository name. For "trdl add", "trdl update" and "trdl bin-path"'
+    description:
+      'The certain repository name. Required if present not given. For "trdl add", "trdl update" and "trdl bin-path".'
     required: true
   url:
-    description: 'The certain repository url. For "trdl add"'
+    description: 'The certain repository url. Required if present not given. For "trdl add"'
     required: true
   root-version:
-    description: 'The certain repository root version. For "trdl add"'
+    description: 'The certain repository root version. Required if present not given. For "trdl add"'
     required: false
   root-sha512:
-    description: 'The certain repository root sha512. For "trdl add"'
+    description: 'The certain repository root sha512. Required if present not given. For "trdl add"'
     required: false
   group:
-    description: 'The certain group. For both "trdl update" and "trdl bin-path"'
+    description: 'The certain group. Required if present not given. For both "trdl update" and "trdl bin-path"'
     required: true
   channel:
     description: 'The one from allowed channels. For both "trdl update" and "trdl bin-path"'

setup-app/action.yml

@@ -30067,7 +30067,7 @@ async function installTrdl(binPath, toolName, toolVersion) {
 }
 async function Do$2(trdlCli, gpgCli, inputs) {
     coreExports.startGroup(`Install or self-update ${trdlCli.name}.`);
-    coreExports.info(format(`parsed inputs=%o`, inputs));
+    coreExports.info(format(`Parsed inputs=%o`, inputs));
     const defaults = trdlCli.defaults();
     coreExports.info(format(`${trdlCli.name} repository defaults=%o`, defaults));
     const options = await getOptions(inputs, defaults);
@@ -30121,32 +30121,32 @@ function mapInputsCmdArgs(inputs) {
     };
 }
 async function Do$1(trdlCli, p) {
-    coreExports.startGroup('Adding application via "trdl add".');
+    coreExports.startGroup(`Adding application via "${trdlCli.name} add".`);
     const noPreset = p === preset.unknown;
-    coreExports.debug(format(`using preset=%s`, !noPreset));
+    coreExports.info(format(`Using preset=%s.`, !noPreset));
     const inputs = parseInputs$1(noPreset);
-    coreExports.debug(format(`parsed inputs=%o`, inputs));
+    coreExports.info(format(`Parsed inputs=%o.`, inputs));
     const args = noPreset ? mapInputsCmdArgs(inputs) : getAddArgs(p);
-    coreExports.debug(format(`merged(preset, inputs) args=%o`, args));
+    coreExports.info(format(`Options for finding or adding application=%o.`, args));
     await trdlCli.mustExist();
     const list = await trdlCli.list();
     const found = list.find((item) => args.repo === item.name);
     if (!found) {
-        coreExports.info(format('Application not found. Adding it via "trdl add" with args=%o.', args));
+        coreExports.info(`Application not found. Adding application via "${trdlCli.name} add".`);
         await trdlCli.add(args);
         coreExports.endGroup();
         return;
     }
     if (!inputs.force) {
         if (found.url !== args.url) {
-            throw new Error(`Already added repo.url=${found.url} is not matched with given input.url=${args.url}. Use the force input to overwrite.`);
+            throw new Error(`Application is already added with repo.url=${found.url} which is not matched with given input.url=${args.url}. Use the force input to overwrite.`);
         }
-        coreExports.info(format('Adding skipped. Application is already added with inputs.url=%s.', args.url));
+        coreExports.info(`Application addition skipped because it is already added with inputs.url=${args.url}.`);
         coreExports.endGroup();
         return;
     }
     // force adding
-    coreExports.info(format('Force adding application using sequence of "trdl remove" and "trdl add" with args=%o.', args));
+    coreExports.info(`Force adding application using "${trdlCli.name} remove" and "${trdlCli.name} add".`);
     await trdlCli.remove(args);
     await trdlCli.add(args);
     coreExports.endGroup();
@@ -30257,28 +30257,28 @@ function formatTrdlUseEnv(args) {
     };
 }
 async function Do(trdlCli, p) {
-    coreExports.startGroup('Using application via "trdl update" and "trdl bin-path"');
+    coreExports.startGroup(`Using application via "${trdlCli.name} update" and "${trdlCli.name} bin-path".`);
     const noPreset = p === preset.unknown;
-    coreExports.debug(format(`using preset=%s`, !noPreset));
+    coreExports.info(format(`Using preset=%s`, !noPreset));
     const inputs = parseInputs(noPreset);
-    coreExports.debug(format(`parsed inputs=%o`, inputs));
+    coreExports.info(format(`Parsed inputs=%o`, inputs));
     const args = noPreset ? mapInputsToCmdArgs(inputs) : getUpdateArgs(p);
-    coreExports.debug(format(`merged(preset, inputs) args=%o`, args));
+    coreExports.info(format(`Options for using application=%o`, args));
     await trdlCli.mustExist();
     let appPath = await trdlCli.binPath(args);
-    coreExports.debug(format(`"trdl bin-path" application path=%s`, appPath));
+    coreExports.info(`Found application path=${appPath}`);
     const hasAppPath = appPath !== '';
     const opts = { inBackground: hasAppPath };
-    coreExports.info(format('Updating application via "trdl update" with args=%o and options=%o.', args, opts));
+    coreExports.info(format(`Updating application via "${trdlCli.name} update" with options=%o`, opts));
     await trdlCli.update(args, opts);
     if (!hasAppPath) {
         appPath = await trdlCli.binPath(args);
-        coreExports.debug(format(`"trdl bin-path" application path=%s`, appPath));
+        coreExports.info(`Found application path=${appPath}`);
     }
     const trdlUseEnv = formatTrdlUseEnv(args);
-    coreExports.info(format('Exporting $%s=%s', trdlUseEnv.key, trdlUseEnv.value));
+    coreExports.info(format('Exporting variable $%s=%s', trdlUseEnv.key, trdlUseEnv.value));
     coreExports.exportVariable(trdlUseEnv.key, trdlUseEnv.value);
-    coreExports.info(format('Extending $PATH variable with app_path=%s', appPath));
+    coreExports.info(`Extending $PATH variable with app_path=${appPath}`);
     coreExports.addPath(appPath);
     coreExports.endGroup();
 }

setup-app/dist/index.mjs

@@ -78,7 +78,7 @@ describe('setup-app/src/add.ts', function () {
       trdlCli.list.mockResolvedValueOnce([listItem])
 
       const expectedErr = new Error(
-        `Already added repo.url=${listItem.url} is not matched with given input.url=${addArgs.url}. Use the force input to overwrite.`
+        `Application is already added with repo.url=${listItem.url} which is not matched with given input.url=${addArgs.url}. Use the force input to overwrite.`
       )
       await expect(Do(trdlCli, preset.werf)).rejects.toThrow(expectedErr)