mirror of
https://github.com/compiler-explorer/compiler-explorer.git
synced 2026-05-16 17:13:59 -04:00
53c7dd328b
Enable Biome's `organizeImports` with groups matching the original ESLint `import/order` configuration: 1. **Node builtins** (`node:fs`, `path`, etc.) 2. *(blank line)* 3. **Third-party packages** (`express`, `@sentry/node`, etc.) 4. *(blank line)* 5. **Local/relative imports** (`../foo.js`, `./bar.js`, aliases) This resolves the inconsistency where Biome wasn't enforcing import grouping, meaning new files would lose the blank-line separation that the old ESLint config enforced. ### Impact - **354 files** updated out of 738 checked (~48%) - **+188 / -240 lines** (net -52) — almost entirely single blank line additions/removals between import groups - No import reordering; purely group separator consistency Fixes #7373 🤖 Generated by LLM (Claude, via OpenClaw)
77 lines
3.1 KiB
TypeScript
77 lines
3.1 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 child_process from 'node:child_process';
|
|
import process from 'node:process';
|
|
|
|
import {afterEach, beforeEach, describe, expect, it, vi} from 'vitest';
|
|
|
|
import {setupTempDir} from '../../lib/app/temp-dir.js';
|
|
|
|
describe('setupTempDir', () => {
|
|
let originalEnv: NodeJS.ProcessEnv;
|
|
|
|
beforeEach(() => {
|
|
originalEnv = {...process.env};
|
|
vi.spyOn(child_process, 'execSync');
|
|
});
|
|
|
|
afterEach(() => {
|
|
process.env = originalEnv;
|
|
vi.restoreAllMocks();
|
|
});
|
|
|
|
it('should set TMP env var with tmpDir option on non-WSL', () => {
|
|
// Skip on Windows as it has different environment variable defaults
|
|
if (process.platform === 'win32') return;
|
|
|
|
setupTempDir('/custom/tmp', false);
|
|
|
|
expect(process.env.TMP).toEqual('/custom/tmp');
|
|
expect(process.env.TEMP).toBeUndefined();
|
|
});
|
|
|
|
it('should set TEMP env var with tmpDir option on WSL', () => {
|
|
// Skip on Windows as it has different environment variable defaults
|
|
if (process.platform === 'win32') return;
|
|
|
|
setupTempDir('/custom/tmp', true);
|
|
|
|
expect(process.env.TEMP).toEqual('/custom/tmp');
|
|
expect(process.env.TMP).toEqual(originalEnv.TMP);
|
|
});
|
|
|
|
it('should try to use Windows TEMP on WSL without tmpDir option', () => {
|
|
// Skip on Windows due to path separator differences (we ironically test this from linux)
|
|
if (process.platform === 'win32') return;
|
|
|
|
vi.mocked(child_process.execSync).mockReturnValue(Buffer.from('C:\\Users\\user\\AppData\\Local\\Temp\n'));
|
|
|
|
setupTempDir(undefined, true);
|
|
|
|
expect(process.env.TEMP).toEqual('/mnt/c/Users/user/AppData/Local/Temp');
|
|
expect(child_process.execSync).toHaveBeenCalledWith('cmd.exe /c echo %TEMP%');
|
|
});
|
|
});
|