Files
OmniRoute/tests/unit/cli/setup-qwen.test.ts
Diego Rodrigues de Sa e Souza 93265eede3 test(infra): retry recursive temp-dir removal on main (main twin of #11968) (#12246)
* test(infra): retry recursive temp-dir removal on main (main twin of #11968)

`main` has been red since b342c1a361 on the vitest and integration gates:

  ✖ tests/unit/autoCombo/provider-family-combos.test.ts > auto/<family>
  ✖ chat pipeline applies Codex OAuth fingerprint and priority tier inside combos

Both call resetStorage() from beforeEach, which does an fs.rmSync(TEST_DATA_DIR,
{recursive: true, force: true}) with no retry, and intermittently loses the race
with a not-yet-released SQLite handle (ENOTEMPTY).

release/v3.8.51 fixed this in #11968 with a mechanical codemod adding
maxRetries/retryDelay to every recursive rm/rmSync/rmdirSync under tests/, but
that PR landed only on the release branch. Because main only receives work at
the release squash, it stayed broken for the whole cycle — and repo-wide gates
then turn every open PR into main red on checks unrelated to their diff.

This is the --base main twin: re-runs the same codemod that already shipped on
the release branch (scripts/ad-hoc/codemod-rm-maxretries.mjs), so the two
branches converge on identical test-teardown semantics. Test-only; no product
logic is touched.

The remaining three failures reported on #12133 (unit full suite exceeding its
4800s ceiling, package-artifact exceeding 1200s, and the boot-smoke that is
skipped as a consequence) are runner-contention timeouts, not code defects —
validate-release-green.mjs runs those heavy gates concurrently on one shared
hosted runner. There is no fix to port for those.

* chore(scripts): carry the rm-maxretries codemod onto main alongside its output

The codemod that generated the previous commit lives in the repo on
release/v3.8.51 (added by #11968) but was never on main. Bringing it over keeps
the tool next to the change it produced, so the transformation stays
reproducible and auditable from either branch.
2026-09-01 01:48:00 -03:00

91 lines
3.2 KiB
TypeScript

import test from "node:test";
import assert from "node:assert/strict";
import fs from "node:fs/promises";
import os from "node:os";
import path from "node:path";
import { resolveQwenTarget, runSetupQwenCommand } from "../../../bin/cli/commands/setup-qwen.mjs";
test("resolveQwenTarget normalizes a remote endpoint to the OpenAI /v1 base", () => {
assert.deepEqual(resolveQwenTarget({ remote: "https://omni.example/", apiKey: "sk-explicit" }), {
baseUrl: "https://omni.example/v1",
apiKey: "sk-explicit",
});
});
test("setup-qwen writes current V4 settings and only its dedicated env key", async () => {
const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), "omniroute-setup-qwen-"));
const settingsPath = path.join(tempDir, "settings.json");
const envPath = path.join(tempDir, ".env");
await fs.writeFile(
settingsPath,
JSON.stringify({
ui: { theme: "dark" },
modelProviders: {
openai: [
{
id: "personal",
envKey: "OPENAI_API_KEY",
baseUrl: "https://api.openai.com/v1",
},
],
},
})
);
await fs.writeFile(envPath, "OPENAI_API_KEY=keep-me\n");
try {
const code = await runSetupQwenCommand({
remote: "http://router:20128",
apiKey: "sk-qwen-dedicated",
model: "qwen/qwen3.8-max-preview",
configPath: settingsPath,
envPath,
yes: true,
// These tests exercise the merge/write logic, not the container guard
// (#10057) — keep them hermetic on container devboxes/CI.
allowContainerWrite: true,
});
assert.equal(code, 0);
const settings = JSON.parse(await fs.readFile(settingsPath, "utf8"));
assert.equal(settings.ui.theme, "dark");
assert.equal(Array.isArray(settings.modelProviders.openai), true);
assert.equal(settings.modelProviders.openai.length, 2);
assert.deepEqual(settings.modelProviders.openai[1], {
id: "qwen/qwen3.8-max-preview",
name: "qwen/qwen3.8-max-preview (OmniRoute)",
envKey: "OMNIROUTE_API_KEY",
baseUrl: "http://router:20128/v1",
});
assert.equal(JSON.stringify(settings).includes("sk-qwen-dedicated"), false);
const env = await fs.readFile(envPath, "utf8");
assert.match(env, /^OPENAI_API_KEY=keep-me$/m);
assert.match(env, /^OMNIROUTE_API_KEY="sk-qwen-dedicated"$/m);
} finally {
await fs.rm(tempDir, { recursive: true, force: true, maxRetries: 5, retryDelay: 100 });
}
});
test("setup-qwen does not overwrite an invalid settings file", async () => {
const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), "omniroute-setup-qwen-bad-"));
const settingsPath = path.join(tempDir, "settings.json");
await fs.writeFile(settingsPath, "{ invalid JSON");
try {
const code = await runSetupQwenCommand({
remote: "http://router:20128",
model: "model-id",
configPath: settingsPath,
yes: true,
// See above — hermetic regardless of container detection (#10057).
allowContainerWrite: true,
});
assert.equal(code, 1);
assert.equal(await fs.readFile(settingsPath, "utf8"), "{ invalid JSON");
} finally {
await fs.rm(tempDir, { recursive: true, force: true, maxRetries: 5, retryDelay: 100 });
}
});