-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Configurize default fixtureStrategy. #18328
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
Conversation
Is it really possible to use the transaction strategy as default? It has very limited use cases, right? |
That's up to the project I guess. |
What other limitations are you referring to? |
@markstory That would be super helpful to quickly change the strategy per app without having to change hundreds or even more files to different base class in between. |
When would you use this? Do you have a plugin or application that can run all of its tests with |
Yeah, any application itself as well. The default one as given right now is sure not helpful beyond a few tests, as it becomes a pain to work with, everywhere relicts after test runs breaking the next tests. |
@markstory can we ship this with tonights release maybe? Docs update in cakephp/docs#8025 |
Docs mention that you can create base classes
https://book.cakephp.org/4/en/development/testing.html#fixture-state-managers
But since those are usually spread across different types and namespaces, this can become cumbersome.
Would it be acceptable to find some kind of default config opt-in?
If not set, nothing would change, so sounds fully BC.