Skip to content

Commit 9338d7a

Browse files
committed
docs: fix skip comments to explain actual reason tests are skipped
workspace.updateWorkspaceFolders() corrupts workspace state for subsequent tests, not just CI crashes. Verified locally.
1 parent c379689 commit 9338d7a

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/unit/commands/batch-organizer.integration.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -536,7 +536,7 @@ suite('BatchOrganizer - REAL Integration (calls actual methods!)', () => {
536536
}
537537
});
538538

539-
// TODO: Convert to stub-based test (uses workspace.updateWorkspaceFolders which crashes CI)
539+
// SKIPPED: workspace.updateWorkspaceFolders() corrupts workspace state for subsequent tests
540540
test.skip('organizeWorkspace() should show warning when no workspace folder open', async function() {
541541
this.timeout(5000);
542542

@@ -683,7 +683,7 @@ suite('BatchOrganizer - REAL Integration (calls actual methods!)', () => {
683683
}
684684
});
685685

686-
// TODO: Convert to stub-based test (uses workspace.updateWorkspaceFolders which crashes CI)
686+
// SKIPPED: workspace.updateWorkspaceFolders() corrupts workspace state for subsequent tests
687687
test.skip('organizeFolder() should handle symlink edge case (VS Code bug #44964)', async function() {
688688
this.timeout(15000);
689689

@@ -764,7 +764,7 @@ suite('BatchOrganizer - REAL Integration (calls actual methods!)', () => {
764764
}
765765
});
766766

767-
// TODO: Convert to stub-based test (uses workspace.updateWorkspaceFolders which crashes CI)
767+
// SKIPPED: workspace.updateWorkspaceFolders() corrupts workspace state for subsequent tests
768768
test.skip('organizeWorkspace() should respect per-root excludePatterns in multi-root workspace', async function() {
769769
this.timeout(20000);
770770

0 commit comments

Comments
 (0)