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

Allow disabling factory deps #76

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

charlax
Copy link

@charlax charlax commented May 17, 2019

Proof of concept fix of #69


This change is Reviewable

@charlax charlax force-pushed the ca/allow-disabling-subfactory-deps branch from dc0f49e to 6cad767 Compare July 4, 2019 16:34
@charlax
Copy link
Author

charlax commented Feb 26, 2020

Any interest in this?

@asteinlein
Copy link

This "magic" factory dependencies has bit me more than once. Just today I spent at least a couple of hours before I remembered the "issue"... I had a RelatedFactory with some attribute overrides, that I could not understand why was not working (i.e. the attributes "reset" to the default related factory as registered by the fixture). So I very much want something like this, however, I'd like it to be per-factory. Perhaps a meta class flag?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants