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

Increased test coverage of django.db.migrations.questioner. #15212

Merged
merged 2 commits into from Dec 20, 2021

Conversation

jacobtylerwalls
Copy link
Member

Only uncovered line now is this return False for ambiguous namespace packages:

if len(migrations_module.__path__) > 1:
return False

@felixxm felixxm changed the title Increased test coverage of migration questioner. Increased test coverage of django.db.migrations.questioner. Dec 20, 2021
@felixxm felixxm self-assigned this Dec 20, 2021
@felixxm
Copy link
Member

felixxm commented Dec 20, 2021

@jacobtylerwalls Thanks 👍 I made small edits.

@felixxm felixxm force-pushed the migration-questioner-coverage branch from d18defe to 03cadb9 Compare December 20, 2021 10:12
@felixxm felixxm merged commit 03cadb9 into django:main Dec 20, 2021
@jacobtylerwalls jacobtylerwalls deleted the migration-questioner-coverage branch December 20, 2021 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants