Skip to content

Commit a6a6a80

Browse files
DeviaVirCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot <[email protected]>
1 parent 7f4536d commit a6a6a80

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

test/main.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,8 @@ const forceRemoveSync = (targetPath) => {
6060
fs.rmSync(targetPath, { recursive: true, force: true })
6161
return
6262
}
63+
// fs-extra's removeSync always removes recursively and handles non-existent paths gracefully,
64+
// so no options are needed here. This is functionally equivalent to the rmSync call above.
6365
fs.removeSync(targetPath)
6466
}
6567

@@ -148,7 +150,7 @@ const _awsRestore = () => {
148150
describe('lib/main', function () {
149151
if (['win32', 'darwin'].includes(process.platform)) {
150152
// It seems that it takes time for file operation in Windows and Mac.
151-
// So set `timeout(180000)` for the whole test.
153+
// So set timeout(180000) (sec: 180) for the whole test.
152154
this.timeout(180000)
153155
}
154156

0 commit comments

Comments
 (0)