Skip to content
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

break up tox test runs with underscores where they should be unique #3173

Merged
merged 2 commits into from Dec 18, 2023

Conversation

fselmo
Copy link
Collaborator

@fselmo fselmo commented Dec 15, 2023

What was wrong?

Related to PR #3157

How was it fixed?

tox is looking at commands in the tox.ini testenv section and running similarly named hyphenated commands together. Breaking up the naming with _ underscores where they need to be unique seems to fix this.

Todo:

Cute Animal Picture

Big thanks to Eric Lam for the watermarked bug picture I just found

Put a link to a cute animal picture inside the parenthesis-->

@fselmo fselmo force-pushed the name-ethtester-tox-runs-similarly branch from 1b2ea67 to 6a2ad06 Compare December 15, 2023 23:07
@fselmo fselmo marked this pull request as ready for review December 15, 2023 23:11
@fselmo fselmo force-pushed the name-ethtester-tox-runs-similarly branch from 6a2ad06 to c212c68 Compare December 15, 2023 23:18
fselmo added a commit to fselmo/web3.py that referenced this pull request Dec 15, 2023
@fselmo fselmo requested a review from kclowes December 15, 2023 23:20
- ``tox`` seems to be picking up multiple patterns otherwise, when looking at the ``commands`` in ``testenv``
@fselmo fselmo force-pushed the name-ethtester-tox-runs-similarly branch from c212c68 to b724530 Compare December 15, 2023 23:32
@fselmo fselmo changed the title Name the flaky and regular eth-tester integration test runs similarly break up tox test runs with underscores where they should be unique Dec 15, 2023
Copy link
Collaborator

@kclowes kclowes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🔨

@fselmo fselmo merged commit f30c6ad into ethereum:main Dec 18, 2023
99 checks passed
@fselmo fselmo deleted the name-ethtester-tox-runs-similarly branch December 18, 2023 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants