Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork960
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
+350 −78
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.