Skip to content

Stabilize inline CodeLens test cleanup on Windows - #1913

Merged
Stella Huang (StellaHuang95) merged 1 commit into
microsoft:mainfrom
StellaHuang95:stellahuang95-codelens-windows-test-fix
Oct 8, 2026
Merged

Stella Huang (StellaHuang95) merged 1 commit into
microsoft:mainfrom
StellaHuang95:stellahuang95-codelens-windows-test-fix

Conversation

@StellaHuang95

Copy link
Copy Markdown
Contributor

Summary

  • retry transient Windows filesystem locks when removing the inline CodeLens integration workspace
  • keep cleanup failures visible after a bounded retry window
  • document the Windows integration-test cleanup pattern

Validation

  • npm run lint
  • npm run compile-tests
  • npm run unittest (2812 passing, 7 pending)
  • npm run compile
  • targeted CodeLens integration suite passed once, then 10 consecutive Windows stress runs passed with no leftover temporary workspaces
  • deterministic Windows probe reproduced EBUSY without retries and verified the bounded retry removes the held directory
  • full integration run passed the CodeLens suite; an unrelated environment-creation test timed out

Fixes #1911

Retry transient Windows filesystem locks when removing the inline CodeLens integration workspace.

Fixes microsoft#1911

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Comment on lines +65 to +66
maxRetries: 5,
retryDelay: 100,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason for those numbers in specific? No issue, just curious

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah no specific reason, just put something reasonable there.

@StellaHuang95
Stella Huang (StellaHuang95) merged commit 84d667e into microsoft:main Oct 8, 2026
58 of 59 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stabilize inline CodeLens test cleanup on Windows

2 participants