Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork937
Test native Windows on CI#1745
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes fromall commits
Commits
Show all changes
29 commits Select commitHold shift + click to select a range
2fd79f4
Add native Windows test jobs to CI matrix
EliahKagan6e477e3
Add xfail marks for IndexFile.from_tree failures
EliahKagancd9d7a9
Mark test_clone_command_injection xfail on Windows
EliahKaganf72e282
Mark test_diff_submodule xfail on Windows
EliahKagan42a3d74
Mark TestSubmodule.test_rename xfail on Windows
EliahKagan4abab92
Mark test_conditional_includes_from_git_dir xfail on Windows
EliahKagan799c853
Improve ordering/grouping of a few imports
EliahKaganb284ad7
Mark test_create_remote_unsafe_url_allowed xfail on Windows
EliahKagan61d1fba
Mark unsafe-options "allowed" tests xfail on Windows
EliahKaganad07ecb
Show PATH on CI
EliahKagan2784e40
Show bash and other WSL-relevant info but not PATH
EliahKagan9717b8d
Install WSL system on CI for hook tests
EliahKagan5d11394
Fix and expand bash.exe xfail marks on hook tests
EliahKaganb215357
Simplify/clarify bash.exe check for hook tests; do it only once
EliahKagancabb572
Temporarily don't install WSL system to test xfail
EliahKagan2875ffa
Put back WSL on Windows CI; pare down debug info
EliahKagan0f8cd4c
Treat XPASS status as a test failure
EliahKagan82c361e
Correct TestSubmodule.test_rename xfail condition
EliahKagan0ae5dd1
Revert "Treat XPASS status as a test failure"
EliahKagan0b7ee17
Refine TestSubmodule.test_rename xfail condition
EliahKagan8621e89
Reword comment in _WinBashStatus.check for clarity
EliahKagan7ff3cee
Make _WinBashStatus instances carry all their info
EliahKagand5ed266
Use bytes in bash.exe check; retest no-distro case
EliahKagan496acaa
Handle multiple encodings for WSL error messages
EliahKagand779a75
Don't assume WSL-related bash.exe error is English
EliahKagan9ac2438
Handle encodings better; make the sum type "public"
EliahKaganb07e5c7
Put back WSL on Windows CI
EliahKagan3303c74
Improve readability of WinBashStatus class
EliahKagane00fffc
Shorten comments on _decode steps
EliahKaganFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
4 changes: 1 addition & 3 deletions.github/workflows/cygwin-test.yml
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
26 changes: 20 additions & 6 deletions.github/workflows/pythonpackage.yml
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
1 change: 1 addition & 0 deletionstest-requirements.txt
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 |
---|---|---|
@@ -9,3 +9,4 @@ pytest-cov | ||
pytest-instafail | ||
pytest-mock | ||
pytest-sugar | ||
sumtypes |
17 changes: 9 additions & 8 deletionstest/test_config.py
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
30 changes: 13 additions & 17 deletionstest/test_diff.py
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
11 changes: 9 additions & 2 deletionstest/test_docs.py
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
17 changes: 14 additions & 3 deletionstest/test_fun.py
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
Oops, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
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.