Skip to content

Commit a8697b2

Browse files
committed
test(fs): unskip safeDelete and safeDeleteSync tests
The 'del' module resolution issues have been resolved. The tests now pass successfully with the vendored del wrapper in src/external/del.js and the del 8.0.1 devDependency. Test results: - safeDelete: 5 tests (all passing) - safeDeleteSync: 4 tests (all passing) - Total: 9 previously skipped tests now running The module resolution works correctly through the external wrapper: src/external/del.js → require('del') → node_modules/del
1 parent 3c744b2 commit a8697b2

1 file changed

Lines changed: 2 additions & 8 deletions

File tree

test/fs.test.ts

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -734,10 +734,7 @@ describe('fs', () => {
734734
})
735735
})
736736

737-
describe.skip('safeDelete', () => {
738-
// Note: These tests are skipped because they require the external 'del' module
739-
// which has module resolution issues in the test environment.
740-
// The functionality is covered by integration tests elsewhere.
737+
describe('safeDelete', () => {
741738

742739
it('should delete files in temp directory', async () => {
743740
await runWithTempDir(async tmpDir => {
@@ -812,10 +809,7 @@ describe('fs', () => {
812809
})
813810
})
814811

815-
describe.skip('safeDeleteSync', () => {
816-
// Note: These tests are skipped because they require the external 'del' module
817-
// which has module resolution issues in the test environment.
818-
// The functionality is covered by integration tests elsewhere.
812+
describe('safeDeleteSync', () => {
819813

820814
it('should delete files in temp directory', async () => {
821815
await runWithTempDir(async tmpDir => {

0 commit comments

Comments
 (0)