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

add a reset_blueprint method #4809

Closed
wants to merge 2 commits into from
Closed

add a reset_blueprint method #4809

wants to merge 2 commits into from

Conversation

rasimandiran
Copy link

Added reset_blueprint method to the Blueprint class. This method re-init the _blueprints list as an empty list and sets the _got_registered_once flag to False

Checklist:

  • Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
  • Add or update relevant docs, in the docs folder and in code.
  • Add an entry in CHANGES.rst summarizing the change and linking to the issue.
  • Add .. versionchanged:: entries in any relevant code docs.
  • Run pre-commit hooks and fix any issues.
  • Run pytest and tox, no tests failed.

@davidism davidism changed the title Bugfix issue 4786 add a reset_blueprint method Sep 7, 2022
@davidism
Copy link
Member

davidism commented Sep 7, 2022

I don't think this is the correct solution, only an application of the workaround mentioned in the issue. This needs more discussion first.

@davidism davidism closed this Sep 7, 2022
@rasimandiran rasimandiran deleted the bugfix-issue-4786 branch September 7, 2022 19:09
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 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.

Nested blueprint registration causes error when using test-scoped app fixtures
2 participants