Files
compiler-explorer/test/temp-tests.ts
Matt Godbolt (bot acct) 05f59881f7 Make --tmp-dir authoritative over inherited environment (#8819)
Fixes #8816.

- `setupTempDir()` exports the configured dir as `TMPDIR`, `TMP` *and*
`TEMP`. POSIX `os.tmpdir()` consults `TMPDIR` first, so the old TMP-only
export meant an inherited `TMPDIR` silently defeated `--tmp-dir`
(verified empirically; prod was protected only by `sudo env_reset` in
start.sh). Setting all three also covers native Windows (`TEMP` > `TMP`
there) and WSL, and means spawned tools reading any of the variables
agree.
- Restores the `os.tmpdir() !== tmpDir → throw` sanity check from
613d7f688 (#6052), lost in the #7681 split refactor.
- The startup log now prints `os.tmpdir()` — the value that actually
matters — instead of `TEMP || TMP`, which printed `undefined` on a
default Linux run.
- The `lib/temp.ts` exit hook was `process.on('exit', async ...)`: exit
handlers can't await, so it never removed anything. Replaced with a
synchronous `cleanupSync()` (tested).
- Test hygiene fix that the work surfaced: the temp-dir tests restored
the environment by reassigning `process.env` wholesale; a replaced
`process.env` is a plain object whose writes never reach the real
environment, while `os.tmpdir()` reads the real environ via `safeGetenv`
— so the suite silently leaked env state across tests (and into any test
running later in the same worker). Now saves/restores the individual
variables.

The WSL `%TEMP%`-discovery-failure path (`wsl-vc.ts` parsing garbage on
fallback, #8816 item 4) is deliberately untouched — Windows-specific and
unverifiable here.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: mattgodbolt-molty <mattgodbolt-molty@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
2026-06-14 20:11:35 +01:00

118 lines
5.5 KiB
TypeScript

// Copyright (c) 2025, Compiler Explorer Authors
// All rights reserved.
//
// Redistribution and use in source and binary forms, with or without
// modification, are permitted provided that the following conditions are met:
//
// * Redistributions of source code must retain the above copyright notice,
// this list of conditions and the following disclaimer.
// * Redistributions in binary form must reproduce the above copyright
// notice, this list of conditions and the following disclaimer in the
// documentation and/or other materials provided with the distribution.
//
// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
// AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
// IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
// ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
// LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
// CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
// SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
// CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
// ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
// POSSIBILITY OF SUCH DAMAGE.
import fs from 'node:fs/promises';
import os from 'node:os';
import path from 'node:path';
import {afterEach, describe, expect, it} from 'vitest';
import * as temp from '../lib/temp.js';
import * as utils from '../lib/utils.js';
describe('Creates and tracks temporary directories', () => {
const osTemp = os.tmpdir();
afterEach(async () => {
await temp.cleanup();
temp.resetStats();
});
it('creates directories under $TMPDIR', async () => {
expect(temp.getStats()).toEqual({numCreated: 0, numActive: 0, numRemoved: 0, numAlreadyGone: 0});
const newTemp = await temp.mkdir('prefix');
expect(newTemp).toContain(osTemp);
expect(await utils.dirExists(newTemp)).toBe(true);
expect(temp.getStats()).toEqual({numCreated: 1, numActive: 1, numRemoved: 0, numAlreadyGone: 0});
});
it('creates directories with prefix', async () => {
const newTemp = await temp.mkdir('prefix');
expect(newTemp).toContain('prefix');
});
it('creates uniquely-named directories', async () => {
const temp1 = await temp.mkdir('prefix');
const temp2 = await temp.mkdir('prefix');
const temp3 = await temp.mkdir('prefix');
expect(temp1).not.toEqual(temp2);
expect(temp1).not.toEqual(temp3);
expect(temp2).not.toEqual(temp3);
expect(temp.getStats()).toEqual({numCreated: 3, numActive: 3, numRemoved: 0, numAlreadyGone: 0});
});
it('cleans up directories even if not empty', async () => {
const newTemp1 = await temp.mkdir('prefix');
await utils.ensureFileExists(path.join(newTemp1, 'some', 'dirs', 'under', 'file'));
const newTemp2 = await temp.mkdir('prefix');
const newTemp3 = await temp.mkdir('prefix');
expect(temp.getStats()).toEqual({numCreated: 3, numActive: 3, numRemoved: 0, numAlreadyGone: 0});
await temp.cleanup();
expect(temp.getStats()).toEqual({numCreated: 3, numActive: 0, numRemoved: 3, numAlreadyGone: 0});
expect(await utils.dirExists(newTemp1)).toBe(false);
expect(await utils.dirExists(newTemp2)).toBe(false);
expect(await utils.dirExists(newTemp3)).toBe(false);
});
it('counts already-cleaned-up directiories', async () => {
const newTemp = await temp.mkdir('prefix');
await fs.rm(newTemp, {recursive: true});
expect(temp.getStats()).toEqual({numCreated: 1, numActive: 1, numRemoved: 0, numAlreadyGone: 0});
await temp.cleanup();
expect(temp.getStats()).toEqual({numCreated: 1, numActive: 0, numRemoved: 0, numAlreadyGone: 1});
});
it('uses absolute paths directly when provided', async () => {
const customBase = await fs.mkdtemp(path.join(osTemp, 'custom-base-'));
try {
const absolutePrefix = path.join(customBase, 'myprefix');
const newTemp = await temp.mkdir(absolutePrefix);
expect(newTemp).toContain(customBase);
expect(newTemp).toContain('myprefix');
expect(newTemp).not.toContain(path.join(osTemp, customBase));
expect(await utils.dirExists(newTemp)).toBe(true);
} finally {
await fs.rm(customBase, {recursive: true, force: true});
}
});
it('uses absolute paths directly for mkdirSync', async () => {
const customBase = await fs.mkdtemp(path.join(osTemp, 'custom-base-'));
try {
const absolutePrefix = path.join(customBase, 'syncprefix');
const newTemp = temp.mkdirSync(absolutePrefix);
expect(newTemp).toContain(customBase);
expect(newTemp).toContain('syncprefix');
expect(await utils.dirExists(newTemp)).toBe(true);
} finally {
await fs.rm(customBase, {recursive: true, force: true});
}
});
});
describe('cleanupSync', () => {
it('synchronously removes pending directories', async () => {
const dir = await temp.mkdir('ce-temp-test');
expect(await utils.dirExists(dir)).toBe(true);
temp.cleanupSync();
expect(await utils.dirExists(dir)).toBe(false);
expect(temp.getStats().numActive).toEqual(0);
});
});