Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve E2E Test Fixtures to be less flaky #2208

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JustinGrote
Copy link
Collaborator

@JustinGrote JustinGrote commented Dec 5, 2024

  • Rewrite Pses Stdio host to be task-based and interface-based
  • Replace LoggingStream with DebugOutputStream to log LSP communication to debug console when debugging tests
  • Wire up LSP Client logging to XUnit ITestOutputHelper
  • Wire up PSES Initial File Logging to debug console

@JustinGrote JustinGrote self-assigned this Dec 5, 2024
@JustinGrote JustinGrote changed the title Initial Pass that implements DebugAdapterServerTestBase Improve E2E Test Fixtures to be less flaky Dec 5, 2024
@JustinGrote JustinGrote force-pushed the test/DapE2ETestUpdates branch 3 times, most recently from 5cb7349 to 1d8a574 Compare December 5, 2024 06:47
@JustinGrote
Copy link
Collaborator Author

@andyleejordan @SeeminglyScience Did a lot of cleanup here, DAP tests run much faster and I should have cleaned up the race conditions, they've been completely reliable for me so far. The tests should also read fairly cleaner now as well.

@JustinGrote JustinGrote force-pushed the test/DapE2ETestUpdates branch from 7f0083c to 7ecdd5a Compare December 6, 2024 17:46
@andyleejordan
Copy link
Member

@JustinGrote running this through the internal pipeline right now!

@andyleejordan
Copy link
Member

@JustinGrote running this through the internal pipeline right now!

Sooo actually per 20fa478 we cannot currently run the tests on OneBranch since it's stuck with PowerShell 7.3...so this will have to wait until January 😵

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants