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
Convert test_tools to directory #66117
Comments
Lib/test/test_tools.py becomes too large. It includes tests of unrelated command-lines tools and scripts. It would be good to convert it to directory containing separate test files for different tools. |
How about this? |
Here's the same patch in --git format, which should make the actual changes clearer (though probably not in Rietveld). |
Here's a new version of the patch in response to review comments. Patch is in --git format, which means no Rietveld link for this one. Changes from the first patch:
|
About load_tests() -- look at Lib/test/test_email/init.py. Otherwise LGTM. |
New changeset e69f037332a6 by Zachary Ware in branch '3.4': New changeset 1d0ca204c36f by Zachary Ware in branch 'default': |
Committed, thanks for the reviews! I had modeled load_tests after test_json's load_tests. The idiom used in test_email causes a warning from regrtest about sys.path being modified, so I modified the idiom slightly and used that. I have declined to backport the patch to 2.7; it's more trouble than it's worth to me. |
Thanks for your patch. Do you want to correct test_json's and others |
Sure, I'll work on that some time this week. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: