Skip to content

gh-120418: Don't assume wheeldata is deleted if WHEEL_PKG_DIR is set - #120419

Merged
vstinner merged 1 commit into
python:mainfrom
stefanor:test-makefile-wheeldata
Jun 12, 2024
Merged

gh-120418: Don't assume wheeldata is deleted if WHEEL_PKG_DIR is set#120419
vstinner merged 1 commit into
python:mainfrom
stefanor:test-makefile-wheeldata

Conversation

@stefanor

@stefanorstefanor commented Jun 12, 2024

Copy link
Copy Markdown
Contributor

Remove wheeldata from both sides of the assertEqual, so that we're actually ignoring it from the test set.

This test is only making assertions about the source tree, no code is being executed that would do anything different based on the value of WHEEL_PKG_DIR.

@vstinner

Copy link
Copy Markdown
Member

cc @sobolevn@hroncok

Comment threadLib/test/test_tools/test_makefile.py Outdated
@stefanor
stefanorforce-pushed the test-makefile-wheeldata branch from 7b152d2 to b38ac70CompareJune 12, 2024 16:56
Comment threadLib/test/test_tools/test_makefile.py Outdated
… is set
Remove wheeldata from both sides of the `assertEqual`, so that we're
*actually* ignoring it from the test set.
This test is only making assertions about the source tree, no code is
being executed that would do anything different based on the value of
`WHEEL_PKG_DIR`.
@stefanor
stefanorforce-pushed the test-makefile-wheeldata branch from b38ac70 to f9826d9CompareJune 12, 2024 17:16
@miss-islington-app

Copy link
Copy Markdown

Thanks @stefanor for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 12, 2024
… is set (pythonGH-120419)
Remove wheeldata from both sides of the `assertEqual`, so that we're
*actually* ignoring it from the test set.
This test is only making assertions about the source tree, no code is
being executed that would do anything different based on the value of
`WHEEL_PKG_DIR`.
(cherry picked from commit 030b452)
Co-authored-by: Stefano Rivera <stefano@rivera.za.net>
@bedevere-app

Copy link
Copy Markdown

GH-120432 is a backport of this pull request to the 3.13 branch.

@bedevere-appbedevere-appBot removed the needs backport to 3.13 bugs and security fixes label Jun 12, 2024
vstinner pushed a commit that referenced this pull request Jun 12, 2024
…` is set (GH-120419) (#120432)
gh-120418: Don't assume wheeldata is deleted if `WHEEL_PKG_DIR` is set (GH-120419)
Remove wheeldata from both sides of the `assertEqual`, so that we're
*actually* ignoring it from the test set.
This test is only making assertions about the source tree, no code is
being executed that would do anything different based on the value of
`WHEEL_PKG_DIR`.
(cherry picked from commit 030b452)
Co-authored-by: Stefano Rivera <stefano@rivera.za.net>
mrahtz pushed a commit to mrahtz/cpython that referenced this pull request Jun 30, 2024
… is set (python#120419)
Remove wheeldata from both sides of the `assertEqual`, so that we're
*actually* ignoring it from the test set.
This test is only making assertions about the source tree, no code is
being executed that would do anything different based on the value of
`WHEEL_PKG_DIR`.
@stefanor
stefanor deleted the test-makefile-wheeldata branch July 1, 2024 17:07
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
… is set (python#120419)
Remove wheeldata from both sides of the `assertEqual`, so that we're
*actually* ignoring it from the test set.
This test is only making assertions about the source tree, no code is
being executed that would do anything different based on the value of
`WHEEL_PKG_DIR`.
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
… is set (python#120419)
Remove wheeldata from both sides of the `assertEqual`, so that we're
*actually* ignoring it from the test set.
This test is only making assertions about the source tree, no code is
being executed that would do anything different based on the value of
`WHEEL_PKG_DIR`.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@stefanor@vstinner@hroncok@sobolevn