scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-08-28 05:27:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

rework git test to see if it helps in the CI build

Browse source
This commit is contained in:
Smartsheet-JB-Brown 2025-04-16 20:27:20 -07:00
parent 68b0a1843d
commit 981c7a1270
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/services/package-manager/__tests__/GitFetcher.test.ts
View file

@ -82,10 +82,15 @@ describe("GitFetcher", () => {
// Reset fs mock defaults
;(fs.mkdir as jest.Mock).mockResolvedValue(undefined)
;(fs.rm as jest.Mock).mockImplementation((path: string, options?: any) => {
if (path === testRepoDir && options?.recursive && options?.force) {
// Always require recursive and force options
if (!options?.recursive || !options?.force) {
return Promise.reject(new Error("Invalid rm call: missing recursive or force options"))
}
// Allow any path under package-manager-cache directory
if (path.includes("package-manager-cache/")) {
return Promise.resolve(undefined)
}
return Promise.reject(new Error("Invalid rm call"))
return Promise.reject(new Error("Invalid rm call: path not in package-manager-cache"))
})
// Setup fs.stat mock for repository structure validation
@ -354,10 +359,7 @@ describe("GitFetcher", () => {
return Promise.reject(new Error("ENOENT"))
})
// Reset fs.rm mock to default behavior
;(fs.rm as jest.Mock).mockImplementation((path: string, options?: any) => {
return Promise.resolve(undefined)
})
// No need to reset fs.rm mock here as it's handled in beforeEach
})
it("should handle paths with spaces when cloning", async () => {