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

[3.6] bpo-28655: Fix test bdb for isolate mode #10220

Merged
merged 1 commit into from Oct 29, 2018

Conversation

maigimenez
Copy link

@maigimenez maigimenez commented Oct 29, 2018

Fix test_bdb when running Python is isolated mode (python3 -I).

https://bugs.python.org/issue28655

Fix test_bdb when running Python is isolated mode.
(cherry picked from commit c0799ec)
@the-knights-who-say-ni
Copy link

the-knights-who-say-ni commented Oct 29, 2018

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately we couldn't find an account corresponding to your GitHub username on bugs.python.org (b.p.o) to verify you have signed the CLA (this might be simply due to a missing "GitHub Name" entry in your b.p.o account settings). This is necessary for legal reasons before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for your contribution, we look forward to reviewing it!

@bedevere-bot bedevere-bot added tests Tests in the Lib/test dir awaiting review labels Oct 29, 2018
@maigimenez maigimenez changed the title bpo-28655: Fix test bdb for isolate mode [3.6] bpo-28655: Fix test bdb for isolate mode Oct 29, 2018
@vstinner
Copy link
Member

vstinner commented Oct 29, 2018

FYI I am mentoring @maigimenez and I asked her to fix the few tests in Python 3.6 which break when Python is run in isolated mode. Tests are fine in 3.7 and newer.

Copy link
Member

@vstinner vstinner left a comment

LGTM. I confirm that the change fix test_bdb as expected.

vstinner@apu$ ./python -m test test_bdb
...
Tests result: SUCCESS

vstinner@apu$ ./python -I  -m test test_bdb
...
Tests result: FAILURE

vstinner@apu$ git pr 10220  # download this PR

vstinner@apu$ ./python -I  -m test test_bdb  # it works!
...
Tests result: SUCCESS

@vstinner
Copy link
Member

vstinner commented Oct 29, 2018

I will merge this PR once you sign the CLA and the PSF will review your signature.

Copy link
Member

@vstinner vstinner left a comment

Hum. Just to prevent myself to merge the PR by mistake, I block the PR until the CLA is signed :-)

It seems like technically, the PR can be merged even if the PR has the label "CLA not signed".

@bedevere-bot
Copy link

bedevere-bot commented Oct 29, 2018

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

Copy link
Member

@vstinner vstinner left a comment

Aha, "CLA signed" is now green.

@vstinner vstinner merged commit 4687bc9 into python:3.6 Oct 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants