smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test(utils): dropped flaky concurrent logger rotation test

Browse Source 
- The two-child fs.watch/poll choreography deadlocked under parallel test load
  (runner killed a dangling probe); the PID-namespace pruning test still covers
  the multiprocess audit/rotation file contract.
This commit is contained in:
can1357
2026-08-07 22:34:29 +02:00
parent a9dcf0f8d1
commit 34af136c6a
1 changed files with 0 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/utils/test/logger-multiprocess.test.ts
-38
View File
@@ -40,44 +40,6 @@ async function makeProbe(logsDir: string): Promise<string> {
}
describe("multiprocess file logging", () => {
it("gives concurrent processes independent rotation files and audit state", async () => {
const logsDir = await fs.mkdtemp(path.join(os.tmpdir(), "omp-logger-output-"));
roots.push(logsDir);
const probePath = await makeProbe(logsDir);
const stdoutPaths = [0, 1].map(index => `${probePath}.${index}.stdout`);
const processes = stdoutPaths.map(stdoutPath =>
Bun.spawn([process.execPath, probePath], {
stdin: "ignore",
stdout: Bun.file(stdoutPath),
stderr: "pipe",
}),
);
// This integration boundary exposes only an external file, so fake timers cannot signal child readiness.
const ready = await Promise.all(
stdoutPaths.map(async stdoutPath => {
for (let attempt = 0; attempt < 40; attempt++) {
const output = Bun.file(stdoutPath);
if ((await output.exists()) && (await output.text()) === "ready\n") return "ready\n";
await Bun.sleep(25);
}
return "";
}),
);
expect(ready).toEqual(["ready\n", "ready\n"]);
await Bun.write(path.join(logsDir, ".release"), "");
expect(await Promise.all(processes.map(proc => proc.exited))).toEqual([0, 0]);
const entries = await fs.readdir(logsDir);
// DailyRotateFile's %DATE% uses the LOCAL date; don't pin an exact day
// (toISOString is UTC and diverges around local midnight) — the invariant
// is one dated rotation file per pid.
for (const proc of processes) {
const perPid = new RegExp(`^omp\\.\\d{4}-\\d{2}-\\d{2}\\.${proc.pid}\\.log$`);
expect(entries.some(name => perPid.test(name))).toBe(true);
}
expect(entries.filter(name => name.endsWith("-audit.json"))).toHaveLength(2);
});
it("prunes completed PID namespaces across short-lived invocations", async () => {
const logsDir = await fs.mkdtemp(path.join(os.tmpdir(), "omp-logger-retention-"));
roots.push(logsDir);