8000 python 3.8 vcrpy 4.0.1 No module named 'mock' · Issue #504 · kevin1024/vcrpy · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

python 3.8 vcrpy 4.0.1 No module named 'mock' #504

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

Closed
llybin opened this issue Dec 20, 2019 · 1 comment · Fixed by #505
Closed

python 3.8 vcrpy 4.0.1 No module named 'mock' #504

llybin opened this issue Dec 20, 2019 · 1 comment · Fixed by #505
Labels
bug core issue related to the core of the library verified can replicate This issue has linked a toy repo that replicates the issue
Milestone

Comments

@llybin
Copy link
llybin commented Dec 20, 2019
python3.8/site-packages/vcr/patch.py", line 5, in <module>
    import mock
ModuleNotFoundError: No module named 'mock'

my suggestion: everywhere in projects change:

import mock -> from unittest import mock

@neozenith
Copy link
Collaborator

Good catch thank you!

@neozenith neozenith added bug core issue related to the core of the library verified can replicate This issue has linked a toy repo that replicates the issue labels Dec 20, 2019
@neozenith neozenith added this to the v4.0.x milestone Dec 20, 2019
neozenith added a commit that referenced this issue Dec 20, 2019
neozenith added a commit that referenced this issue Dec 20, 2019
Update changelog

Revert accidental import deletion
evgeni added a commit to evgeni/foreman-ansible-modules that referenced this issue Dec 20, 2019
bagasse pushed a commit to theforeman/foreman-ansible-modules that referenced this issue Dec 20, 2019
neozenith added a commit that referenced this issue Dec 20, 2019
* Correct mock imports. Fixes #504

Update changelog

Revert accidental import deletion

* Remove move dependency that was masking error from tox.ini
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug core issue related to the core of the library verified can replicate This issue has linked a toy repo that replicates the issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants
0