-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Update test/libregrtest to CPython 3.13.10
#6410
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
Changes from 1 commit
f297888
5d253d1
08762a3
f436c51
a8e7633
e4fb263
32fd6be
404836f
4c833fc
2789ff7
4a559f3
bedcb22
1f56a0f
c76e725
cadac34
8aafbf2
8d2b215
472c12c
17cc559
1827af1
e831124
3aa6bf8
34fe0f2
dc9475c
55c71aa
9500f66
99168e0
3427215
0d128cc
59b80e3
0a82673
2769c08
40da807
8765d86
12811eb
d170b02
b76c0a2
98d8ebc
d1e75c0
55c3096
a09ed1b
e6d2ba0
def6974
ae6deb7
1c07d35
1642959
daaae22
717265a
80af69e
8d505dc
f9a1f2b
7445b27
1dcded4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -341,13 +341,20 @@ jobs: | |
| done | ||
| if [ ${exit_code} -ne 3 ]; then | ||
| echo "Test ${thing} is no longer polluting the environment after ${i} attempts!" | ||
| echo "Please remove ${thing} from either ENV_POLLUTING_TESTS_COMMON or ENV_POLLUTING_TESTS_LINUX in '.github/workflows/ci.yaml'." | ||
| echo "Please also remove the skip decorators that include the word 'POLLUTERS' in ${thing}." | ||
| if [ ${exit_code} -ne 0 ]; then | ||
| echo "Test ${thing} failed with exit code ${exit_code}." | ||
| echo "Please investigate which test item in ${thing} is failing and either mark it as an expected failure or a skip." | ||
| fi | ||
| exit 1 | ||
| fi | ||
| done | ||
| timeout-minutes: 15 | ||
|
|
||
| - if: runner.os == 'macOS' | ||
| name: run cpython tests to check if env polluters have stopped polluting (Common/macOS) | ||
| shell: bash | ||
| run: | | ||
| for thing in ${{ env.ENV_POLLUTING_TESTS_COMMON }} ${{ env.ENV_POLLUTING_TESTS_MACOS }}; do | ||
| for i in $(seq 1 10); do | ||
|
|
@@ -362,6 +369,12 @@ jobs: | |
| done | ||
| if [ ${exit_code} -ne 3 ]; then | ||
| echo "Test ${thing} is no longer polluting the environment after ${i} attempts!" | ||
| echo "Please remove ${thing} from either ENV_POLLUTING_TESTS_COMMON or ENV_POLLUTING_TESTS_MACOS in '.github/workflows/ci.yaml'." | ||
| echo "Please also remove the skip decorators that include the word 'POLLUTERS' in ${thing}." | ||
| if [ ${exit_code} -ne 0 ]; then | ||
| echo "Test ${thing} failed with exit code ${exit_code}." | ||
| echo "Please investigate which test item in ${thing} is failing and either mark it as an expected failure or a skip." | ||
| fi | ||
| exit 1 | ||
| fi | ||
| done | ||
|
|
@@ -384,6 +397,12 @@ jobs: | |
| done | ||
| if [ ${exit_code} -ne 3 ]; then | ||
| echo "Test ${thing} is no longer polluting the environment after ${i} attempts!" | ||
| echo "Please remove ${thing} from either ENV_POLLUTING_TESTS_COMMON or ENV_POLLUTING_TESTS_WINDOWS in '.github/workflows/ci.yaml'." | ||
| echo "Please also remove the skip decorators that include the word 'POLLUTERS' in ${thing}." | ||
| if [ ${exit_code} -ne 0 ]; then | ||
| echo "Test ${thing} failed with exit code ${exit_code}." | ||
| echo "Please investigate which test item in ${thing} is failing and either mark it as an expected failure or a skip." | ||
| fi | ||
| exit 1 | ||
| fi | ||
| done | ||
|
Comment on lines
+383
to
+404
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @coderabbitai translate this to powershell
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @fanninpm Here's the PowerShell translation of the Windows environment pollution check script: - if: runner.os == 'Windows'
name: run cpython tests to check if env polluters have stopped polluting (Common/windows)
shell: pwsh
run: |
$tests = "${{ env.ENV_POLLUTING_TESTS_COMMON }} ${{ env.ENV_POLLUTING_TESTS_WINDOWS }}" -split '\s+' | Where-Object { $_ }
foreach ($thing in $tests) {
for ($i = 1; $i -le 10; $i++) {
target/release/rustpython -m test -j 1 --slowest --fail-env-changed --timeout 600 -v $thing
$exit_code = $LASTEXITCODE
if ($exit_code -eq 3) {
Write-Host "Test $thing polluted the environment on attempt $i."
break
}
}
if ($exit_code -ne 3) {
Write-Host "Test $thing is no longer polluting the environment after $i attempts!"
Write-Host "Please remove $thing from either ENV_POLLUTING_TESTS_COMMON or ENV_POLLUTING_TESTS_WINDOWS in ``.github/workflows/ci.yaml``."
Write-Host "Please also remove the skip decorators that include the word 'POLLUTERS' in $thing."
if ($exit_code -ne 0) {
Write-Host "Test $thing failed with exit code $exit_code."
Write-Host "Please investigate which test item in $thing is failing and either mark it as an expected failure or a skip."
}
exit 1
}
}Key changes from bash:
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.