scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-10-11 03:38:15 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

test: stabilize checkpoint service temp dir cleanup

Browse source
This commit is contained in:
Hannes Rudolph 2026-01-19 19:48:53 -07:00
parent 8c142755be
commit d1bdbd56b8
1 changed files with 19 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
src/services/checkpoints/__tests__/ShadowCheckpointService.spec.ts
View file

@ -12,7 +12,9 @@ import * as fileSearch from "../../../services/search/file-search"
import { RepoPerTaskCheckpointService } from "../RepoPerTaskCheckpointService"
const tmpDir = path.join(os.tmpdir(), "CheckpointService")
// Use a unique tmp directory per test run to avoid collisions with other
// Vitest workers/processes and to keep cleanup fast/reliable on Windows.
const tmpDir = path.join(os.tmpdir(), `CheckpointService-${process.pid}-${Date.now()}`)
const initWorkspaceRepo = async ({
workspaceDir,
@ -55,10 +57,12 @@ describe.each([[RepoPerTaskCheckpointService, "RepoPerTaskCheckpointService"]])(
let workspaceGit: SimpleGit
let testFile: string
let service: RepoPerTaskCheckpointService
let shadowDir: string
let workspaceDir: string
beforeEach(async () => {
const shadowDir = path.join(tmpDir, `${prefix}-${Date.now()}`)
const workspaceDir = path.join(tmpDir, `workspace-${Date.now()}`)
shadowDir = path.join(tmpDir, `${prefix}-${Date.now()}`)
workspaceDir = path.join(tmpDir, `workspace-${Date.now()}`)
const repo = await initWorkspaceRepo({ workspaceDir })
workspaceGit = repo.git
@ -70,10 +74,21 @@ describe.each([[RepoPerTaskCheckpointService, "RepoPerTaskCheckpointService"]])(
afterEach(async () => {
vitest.restoreAllMocks()
// Clean up per-test directories to prevent a huge accumulated tmp tree.
// This makes Windows CI much less likely to hit slow/blocked recursive deletes.
await Promise.all([
fs
.rm(shadowDir, { recursive: true, force: true, maxRetries: 10, retryDelay: 100 })
.catch(() => undefined),
fs
.rm(workspaceDir, { recursive: true, force: true, maxRetries: 10, retryDelay: 100 })
.catch(() => undefined),
])
})
afterAll(async () => {
await fs.rm(tmpDir, { recursive: true, force: true })
await fs.rm(tmpDir, { recursive: true, force: true, maxRetries: 10, retryDelay: 100 })
}, 60_000) // 60 second timeout for Windows cleanup
describe(`${klass.name}#getDiff`, () => {