Skip to content

Commit 3bb87e2

Browse files
committed
make test more robutst to implementation changes
1 parent 2c299d5 commit 3bb87e2

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

packages/core/src/codewhisperer/service/transformByQ/transformApiHandler.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -354,9 +354,7 @@ export async function zipCode({ dependenciesFolder, humanInTheLoopFlag, modulePa
354354

355355
tempFilePath = path.join(os.tmpdir(), 'zipped-code.zip')
356356
await fs.writeFile(tempFilePath, zip.toBuffer())
357-
if (await fs.exists(dependenciesFolder.path)) {
358-
await fs.delete(dependenciesFolder.path, { recursive: true, force: true })
359-
}
357+
await fs.delete(dependenciesFolder.path, { recursive: true, force: true })
360358
} catch (e: any) {
361359
throw Error('Failed to zip project')
362360
} finally {

packages/core/src/test/codewhisperer/zipcode.test.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ describe('zipCode', function () {
5757
fileNameSuffix: '.md',
5858
})
5959
).uri.fsPath
60+
const writeSpy = sinon.spy(fs, 'writeFile')
6061
const transformQManifest = new ZipManifest()
6162
transformByQState.setProjectPath(tempDir)
6263
transformQManifest.customBuildCommand = CodeWhispererConstants.skipUnitTestsBuildCommand
@@ -75,10 +76,10 @@ describe('zipCode', function () {
7576
},
7677
verify: async (setup: SetupResult) => {
7778
// writes a zip to disk.
78-
assert.ok(setup.writeSpy.called)
7979
assert.ok(
80-
setup.writeSpy.getCalls()[0].args[0].includes('.zip') ||
81-
setup.writeSpy.getCalls()[1].args[0].includes('.zip')
80+
setup.writeSpy.args.find((arg) => {
81+
return arg[0].endsWith('.zip')
82+
})
8283
)
8384
},
8485
}

0 commit comments

Comments
 (0)