-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update database_cleaner #125
base: master
Are you sure you want to change the base?
Conversation
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.
Some files could not be reviewed due to errors:
Error: unrecognized cop Rails found in .rubocop.yml
Error: unrecognized cop Rails found in .rubocop.yml
Codecov Report
@@ Coverage Diff @@
## master #125 +/- ##
=======================================
Coverage 99.48% 99.48%
=======================================
Files 26 26
Lines 388 388
=======================================
Hits 386 386
Misses 2 2
Continue to review full report at Codecov.
|
Upgrading Database Cleaner to version 2.x required some configuration changes: - "Truncation" strategy for Mongoid has been renamed to "deletion". - "Only" configuration option now requires an array.
Use DatabaseCleaner::clean rather than DatabaseCleaner::clean_with, as it honors adapter configuration.
0fac7ca
to
7deac77
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.
Some files could not be reviewed due to errors:
Error: unrecognized cop Rails found in .rubocop.yml
Error: unrecognized cop Rails found in .rubocop.yml
There are issues that need resolving if we upgrade database_cleaner:
This means we need to drop Rails 4 and Ruby 2.4. Which is fine? Ping @ribose-jeffreylau . |
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.
Some files could not be reviewed due to errors:
Error: unrecognized cop Rails found in .rubocop.yml
Error: unrecognized cop Rails found in .rubocop.yml
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.
Some files could not be reviewed due to errors:
Error: unrecognized cop Rails found in .rubocop.yml
Error: unrecognized cop Rails found in .rubocop.yml
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.
Some files could not be reviewed due to errors:
Error: unrecognized cop Rails found in .rubocop.yml
Error: unrecognized cop Rails found in .rubocop.yml
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.
Some files could not be reviewed due to errors:
Error: unrecognized cop Rails found in .rubocop.yml
Error: unrecognized cop Rails found in .rubocop.yml
The remaining failures are all in e.g.
Seems related to: rspec/rspec-core#2603 |
#123 is dependent on this PR as DatabaseCleaner 2.0+ provides Rails 7 support. |
No description provided.