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

fix pycharm unit test debug execution by renaming http package #6071

Merged
merged 1 commit into from May 17, 2022

Conversation

alexrashed
Copy link
Member

This PR renames the module unit.http to unit.http_.
It fixes a developer experience issue where unit tests in the "root" folder of the unit tests could not be executed in debug mode in PyCharm.
You can easily reproduce this by trying to execute any test in the unit folder in debug mode using PyCharm's pytest integration (f.e. test_generic_proxy).
PyCharm tries to load the http module (which contains unit tests for the localstack.http module) instead of the builtin http module.
PyCharm YouTrack Issue: https://youtrack.jetbrains.com/issue/PY-54265

@alexrashed alexrashed requested a review from thrau May 17, 2022 08:41
@alexrashed alexrashed temporarily deployed to localstack-ext-tests May 17, 2022 08:41 Inactive
@github-actions
Copy link

LocalStack integration with Pro

       3 files  ±0         3 suites  ±0   59m 19s ⏱️ + 1m 48s
1 028 tests ±0     970 ✔️ ±0  58 💤 ±0  0 ±0 
1 324 runs  ±0  1 239 ✔️ ±0  85 💤 ±0  0 ±0 

Results for commit 40355c0. ± Comparison against base commit edf832e.

@thrau thrau merged commit a70d9f5 into master May 17, 2022
@thrau thrau deleted the fix-pycharm-unit-test-exec branch May 17, 2022 11:46
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants