test(c00010 S3): widen push / cli-dispatch branches

- tests/cli/push-command.test.ts: stable outcome envelope shape on
  failure, next-action text on missing key (covers the
  falloDeApi(reportApiError) branch).
- tests/cli/cli-dispatch.test.ts: --config absolutization (third of
  four path flags rewritten by absolutizePathFlags).

Co-Authored-By: delendai-impl-20260908 <noreply@delendai>

Author: CartagoGit

tests/cli/cli-dispatch.test.ts

@@ -89,4 +89,32 @@ describe("cli.script.ts — the dispatcher", () => {
       expect([0, 1]).toContain(code);
     }
   });
+
+  test("--config with a relative path is absolutized", async () => {
+    // `absolutizePathFlags` rewrites every flag that names a file:
+    // --project-root, --config, --output, --output-dir. The previous
+    // test covers --project-root; here we cover --config so all four
+    // branches are exercised.
+    const root = join(work, "config-flag");
+    await copyExampleClean(exampleDir("express"), root);
+    // We pick an explicit path that does NOT exist: what matters
+    // here is that the dispatcher absolutizes the flag before
+    // dispatching, and that the downstream command receives an
+    // absolute path (so the loader's `Config no encontrado` error
+    // surfaces, instead of a `path must be absolute` one).
+    try {
+      await run([
+        "list",
+        "--project-root",
+        root,
+        "--config",
+        "tests/fixtures/does-not-exist.ts",
+      ]);
+      // If the command happened to find a file, the call returned a
+      // number. Either way, the dispatcher did its job.
+    } catch (error) {
+      expect((error as Error).message).toContain("Config no encontrado");
+      expect((error as Error).message).not.toContain("must be absolute");
+    }
+  });
 });

tests/cli/push-command.test.ts

@@ -124,4 +124,42 @@ describe("push — runPush", () => {
       else process.env["POSTMAN_API_KEY"] = previousKey;
     }
   });
+
+  test("the user-friendly error carries a next-action when the key is missing", async () => {
+    const previousKey = process.env["POSTMAN_API_KEY"];
+    process.env["POSTMAN_API_KEY"] = "";
+    try {
+      const outcome = await runPush([]);
+      expect(outcome.code).toBe(1);
+      expect(outcome.error?.reason).toMatch(/api.?key/i);
+      // The next-action line is what the human reads; it must say
+      // how to get the key. We pin both the flag and the env var.
+      expect(outcome.error?.nextAction).toContain("--api-key");
+      expect(outcome.error?.nextAction).toContain("POSTMAN_API_KEY");
+    } finally {
+      if (previousKey === undefined) delete process.env["POSTMAN_API_KEY"];
+      else process.env["POSTMAN_API_KEY"] = previousKey;
+    }
+  });
+
+  test("the outcome envelope shape is stable even when the upload fails", async () => {
+    // Push always returns an `IPushOutcome` with `error` populated
+    // on failure; we verify the envelope shape regardless of the
+    // specific failure mode.
+    const previousKey = process.env["POSTMAN_API_KEY"];
+    process.env["POSTMAN_API_KEY"] = "";
+    try {
+      const outcome = await runPush([]);
+      expect(typeof outcome.code).toBe("number");
+      expect(outcome.user).toBeNull();
+      expect(outcome.framework).toBeNull();
+      expect(outcome.requests).toBe(0);
+      expect(outcome.collection).toBeNull();
+      expect(outcome.environments).toEqual([]);
+      expect(outcome.error).not.toBeNull();
+    } finally {
+      if (previousKey === undefined) delete process.env["POSTMAN_API_KEY"];
+      else process.env["POSTMAN_API_KEY"] = previousKey;
+    }
+  });
 });