test: add cli test to validate multiple manifest generation paths result in comparable output

.eslintrc.json

@@ -6,7 +6,7 @@
   "extends": ["eslint:recommended", "plugin:@typescript-eslint/recommended"],
   "parser": "@typescript-eslint/parser",
   "parserOptions": {
-    "project": ["tsconfig.json"],
+    "project": ["tsconfig.eslint.json"],
     "ecmaVersion": 2022
   },
   "rules": {
@@ -21,6 +21,14 @@
     "max-statements": ["warn", { "max": 20 }, { "ignoreTopLevelFunctions": true }],
     "no-invalid-this": "warn"
   },
+  "overrides": [
+    {
+      "files": ["*.test.ts"],
+      "rules": {
+        "max-nested-callbacks": ["warn", { "max": 8 }]
+      }
+    }
+  ],
   "plugins": ["@typescript-eslint"],
   "ignorePatterns": [
     "src/templates",
@@ -30,7 +38,8 @@
     "pepr-test-module",
     "build.mjs",
     "journey",
-    "__mocks__"
+    "__mocks__",
+    "integration/testroot"
   ],
   "root": true
 }

.github/workflows/node.js.yml

@@ -44,6 +44,25 @@ jobs:
         uses: codecov/codecov-action@1e68e06f1dbfde0e4cefc87efeba9e4643565303 # v5.1.2
         env:
           CODECOV_TOKEN: ${{ secrets.CODECOV_ORG_TOKEN }}
+
+  integration:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+
+      - name: Use Node.js 22
+        uses: actions/setup-node@39370e3970a6d050c480ffad4ff0ed4d3fdee5af # v4.1.0
+        with:
+          node-version: 22
+          cache: "npm"
+      - name: Setup Helm
+        uses: azure/setup-helm@fe7b79cd5ee1e45176fcad797de68ecaf3ca4814 # v4.2.0
+        with:
+          version: v3.3.4
+
+      - run: npm ci
+      - run: npm run test:integration
+
   journey:
     runs-on: ubuntu-latest
     steps:

integration/.gitignore

@@ -0,0 +1,2 @@
+testroot/
+.npm/

integration/cli/build.defaults.test.ts

@@ -0,0 +1,52 @@
+// SPDX-License-Identifier: Apache-2.0
+// SPDX-FileCopyrightText: 2023-Present The Pepr Authors
+
+import { beforeAll, describe, expect, it } from "@jest/globals";
+import * as path from "node:path";
+import * as fs from "node:fs/promises";
+import { Workdir } from "../helpers/workdir";
+import * as time from "../helpers/time";
+import * as pepr from "../helpers/pepr";
+
+const FILE = path.basename(__filename);
+const HERE = __dirname;
+
+describe("build", () => {
+  const workdir = new Workdir(`${FILE}`, `${HERE}/../testroot/cli`);
+
+  beforeAll(async () => {
+    await workdir.recreate();
+  });
+
+  describe("builds a module", () => {
+    const id = FILE.split(".").at(1);
+    const testModule = `${workdir.path()}/${id}`;
+
+    beforeAll(async () => {
+      await fs.rm(testModule, { recursive: true, force: true });
+      const argz = [
+        `--name ${id}`,
+        `--description ${id}`,
+        `--errorBehavior reject`,
+        "--confirm",
+        "--skip-post-init",
+      ].join(" ");
+      await pepr.cli(workdir.path(), { cmd: `pepr init ${argz}` });
+      await pepr.tgzifyModule(testModule);
+      await pepr.cli(testModule, { cmd: `npm install` });
+    }, time.toMs("2m"));
+
+    describe("using default build options", () => {
+      it(
+        "builds",
+        async () => {
+          const build = await pepr.cli(testModule, { cmd: `pepr build` });
+          expect(build.exitcode).toBe(0);
+          expect(build.stderr.join("").trim()).toBe("");
+          expect(build.stdout.join("").trim()).toContain("K8s resource for the module saved");
+        },
+        time.toMs("1m"),
+      );
+    });
+  });
+});

integration/cli/build.help.test.ts

@@ -0,0 +1,31 @@
+// SPDX-License-Identifier: Apache-2.0
+// SPDX-FileCopyrightText: 2023-Present The Pepr Authors
+
+import { beforeAll, describe, expect, it } from "@jest/globals";
+import * as path from "node:path";
+import { Workdir } from "../helpers/workdir";
+import * as pepr from "../helpers/pepr";
+import * as time from "../helpers/time";
+
+const FILE = path.basename(__filename);
+const HERE = __dirname;
+
+describe("build", () => {
+  const workdir = new Workdir(`${FILE}`, `${HERE}/../testroot/cli`);
+
+  beforeAll(async () => {
+    await workdir.recreate();
+  });
+
+  it(
+    "gives command line help",
+    async () => {
+      const argz = "--help";
+      const result = await pepr.cli(workdir.path(), { cmd: `pepr build ${argz}` });
+      expect(result.exitcode).toBe(0);
+      expect(result.stderr.join("").trim()).toBe("");
+      expect(result.stdout.at(0)).toMatch("Usage: pepr build");
+    },
+    time.toMs("30s"),
+  );
+});

integration/cli/build.noembed.test.ts

@@ -0,0 +1,93 @@
+// SPDX-License-Identifier: Apache-2.0
+// SPDX-FileCopyrightText: 2023-Present The Pepr Authors
+
+import { beforeAll, describe, expect, it } from "@jest/globals";
+import * as path from "node:path";
+import * as fs from "node:fs/promises";
+import { Workdir } from "../helpers/workdir";
+import * as time from "../helpers/time";
+import * as pepr from "../helpers/pepr";
+import * as resource from "../helpers/resource";
+import * as file from "../helpers/file";
+
+const FILE = path.basename(__filename);
+const HERE = __dirname;
+
+describe("build", () => {
+  const workdir = new Workdir(`${FILE}`, `${HERE}/../testroot/cli`);
+
+  beforeAll(async () => {
+    await workdir.recreate();
+  });
+
+  describe("builds a module", () => {
+    const id = FILE.split(".").at(1);
+    const testModule = `${workdir.path()}/${id}`;
+
+    beforeAll(async () => {
+      await fs.rm(testModule, { recursive: true, force: true });
+      const argz = [
+        `--name ${id}`,
+        `--description ${id}`,
+        `--errorBehavior reject`,
+        "--confirm",
+        "--skip-post-init",
+      ].join(" ");
+      await pepr.cli(workdir.path(), { cmd: `pepr init ${argz}` });
+      await pepr.tgzifyModule(testModule);
+      await pepr.cli(testModule, { cmd: `npm install` });
+    }, time.toMs("2m"));
+
+    describe("for use as a library", () => {
+      let packageJson;
+      let uuid: string;
+
+      it(
+        "builds",
+        async () => {
+          const argz = [`--no-embed`].join(" ");
+          const build = await pepr.cli(testModule, { cmd: `pepr build ${argz}` });
+          expect(build.exitcode).toBe(0);
+
+          // TODO: team talk
+          // Should this be writing to stderr? Even with a 0 exit code..?
+          expect(build.stderr.join("").trim()).toContain("Error: Cannot find module");
+          // TODO: end
+
+          expect(build.stdout.join("").trim()).toContain("");
+
+          packageJson = await resource.oneFromFile(`${testModule}/package.json`);
+          uuid = packageJson.pepr.uuid;
+        },
+        time.toMs("1m"),
+      );
+
+      it(
+        "outputs appropriate configuration",
+        async () => {
+          const missing = [
+            `${testModule}/dist/pepr-${uuid}.js`,
+            `${testModule}/dist/pepr-${uuid}.js.map`,
+            `${testModule}/dist/pepr-${uuid}.js.LEGAL.txt`,
+            `${testModule}/dist/pepr-module-${uuid}.yaml`,
+            `${testModule}/dist/zarf.yaml`,
+            `${testModule}/dist/${uuid}-chart/`,
+          ];
+          for (const path of missing) {
+            expect(await file.exists(path)).toBe(false);
+          }
+
+          const found = [
+            `${testModule}/dist/pepr.js`,
+            `${testModule}/dist/pepr.js.map`,
+            `${testModule}/dist/pepr.js.LEGAL.txt`,
+          ];
+          for (const path of found) {
+            expect(await file.exists(path)).toBe(true);
+          }
+        },
+        time.toMs("1m"),
+      );
+    });
+  });
+});