forked from DonJayamanne/pythonVSCode
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Switch to using named pipes for test communication #22390
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
Merged
Merged
Changes from all commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
cc353a2
Add named pipe utility classes
karthiknadig f9943a5
Get test ids server working with named pipes
karthiknadig 364ed9f
Add named pipes to pytest discovery and execution
karthiknadig 5edfb26
get named sockets functioning for unix
eleanorjboyd 7b387a8
move socket stuff to different file
eleanorjboyd c2fc87e
curr progress
eleanorjboyd 59500b2
renamed vars
eleanorjboyd b43d073
cleanup
eleanorjboyd 215af8d
update pytest unittests to get them working
eleanorjboyd 585893c
fix tests
eleanorjboyd 57e0c9b
fix unittests python tests
eleanorjboyd ca279f6
fix utils rename
eleanorjboyd 322c9fc
Merge branch 'main' into named-pipes
eleanorjboyd a4d6367
windows work-in-progress
eleanorjboyd 46abb41
fixing env vars pytest discovery adapter
eleanorjboyd 544caed
Merge branch 'main' into named-pipes
eleanorjboyd 219629e
pytest tests working mac
eleanorjboyd 1b150ce
fix path
eleanorjboyd e02cc71
fixing some errors to run CI tests
eleanorjboyd b0fcf45
fix types
eleanorjboyd 4d7fc7e
spot a
eleanorjboyd 6f67ee3
working one pytest mac
eleanorjboyd 784c318
spot b
eleanorjboyd 2812465
edits for windows 14/16
eleanorjboyd 89278ba
add conditional requirement for windows
eleanorjboyd b098067
Merge branch 'main' into named-pipes
eleanorjboyd f61086a
fix build errors
eleanorjboyd c999593
fix python types
eleanorjboyd a9f123d
fix python tests, return new helper
eleanorjboyd e8da064
type fixes
eleanorjboyd c16629b
rename to python_files to align with main
eleanorjboyd ef4f468
untracked align with main
eleanorjboyd a84b29f
Merge remote-tracking branch 'upstream/main' into named-pipes
eleanorjboyd 0ab99d9
merge issues
eleanorjboyd 071fc39
more merge conflicts
eleanorjboyd 2fb7c83
merge with main again
eleanorjboyd 0229592
updates to support large repos
eleanorjboyd e965362
Merge branch 'main' into named-pipes
eleanorjboyd 63187b1
cancelationtokensworking
eleanorjboyd 1235e7b
unittest discovery tests working
eleanorjboyd f4432f7
fix ruff requirements
eleanorjboyd 9f55664
Merge remote-tracking branch 'upstream/main' into named-pipes
eleanorjboyd 1855128
cleanup
eleanorjboyd ed52d0c
add logging to catch testing cwd issue
eleanorjboyd 5911a44
fix import
eleanorjboyd dbee9bc
remove extra print lines
eleanorjboyd e2afae5
remove unneeded sections in helpers
eleanorjboyd 5a1ba8f
add pytest.ini file to .data directory
eleanorjboyd 8af151e
cleanup
eleanorjboyd 4cf2cd4
fixing windows test
eleanorjboyd a513fff
remove socket manager reader for windows
eleanorjboyd 8419dab
fix launch debug tests
eleanorjboyd da64ff5
add exception handling for reader
eleanorjboyd 32c1af3
Merge branch 'main' into named-pipes
eleanorjboyd 800c671
remove unneeded print
eleanorjboyd 53beb2a
add reader to read method
eleanorjboyd 921c924
Fix bug with windows named pipes
karthiknadig 5f4d71c
Fix test and formatting
karthiknadig b128b68
Fix linting
karthiknadig a868c29
Fix pyright complaints
karthiknadig File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# Copyright (c) Microsoft Corporation. All rights reserved. | ||
# Licensed under the MIT License. | ||
|
||
# pytest.ini is specified here so the root directory of the tests is kept at .data instead of referencing | ||
# the parent python_files/pyproject.toml for test_discovery.py and test_execution.py for pytest-adapter tests. |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.