-
-
Notifications
You must be signed in to change notification settings - Fork 715
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
3217 [Enterprise Fee Summary] Move setup code in tests to factories #3312
3217 [Enterprise Fee Summary] Move setup code in tests to factories #3312
Conversation
We should not presume what is not default for an enterprise fee object.
eef993f
to
73e15c1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome @kristinalim we finally have /spec/factories/ 🎉
It's also good you are just adding new stuff, moving things from factories.rb to here will be better done after v2 :-)
I approve the PR, please follow my suggestions below if you agree, otherwise ignore.
Thanks for your feedback, @luisramos0! I renamed the files in
|
Make these follow the "#{SINGULAR_OBJECT}_factory.rb" convention.
d2ac046
to
2e14251
Compare
awesome, ready for testing. |
And it's a |
ah, yes, thanks for the heads up @sigmundpetersen |
Move setup code in Enterprise Fee Summary tests to factories.
What? Why?
Closes #3217
This cleans up the code, and allows customization and reuse of setup logic in the tests.
What should we test?
No user-observable changes expected.
Release notes
Changelog Category: Added
Dependencies
This requires #3115 to be merged to the transitional branch first.