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 rspec for CI and development to 3.8.0+ #699
Conversation
It supresses ERB.new deprecation warnings. rspec/rspec-core@c1b5695
We many need to pass the last optional Hash to `.with()` as Hash literal.
@@ -71,7 +71,7 @@ class Address; end | |||
'development' => {'hosts' => ['127.0.0.1:27017'], 'database' => 'test', 'read' => 'primary'} | |||
} | |||
logger = double('logger') | |||
Mongo::Client.should_receive(:new).with(['127.0.0.1:27017'], :logger => logger, :read => :primary, :database => 'test') | |||
Mongo::Client.should_receive(:new).with(['127.0.0.1:27017'], { :logger => logger, :read => :primary, :database => 'test' }) |
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.
MongoMapper.connect(environment, options={})
's last argument is a positional argument, not a keyword arguments. So we have to expect it as so.
These changes are related to rspec/rspec-mocks#1394.
Hmmm, coveraged for each file are calculated correctly, but total coverage are not. Related? |
Let me merge. It may be one-off. |
It supresses ERB.new deprecation warnings.
rspec/rspec-core@c1b5695
Please check the CI outputs for Ruby 3.0+.
https://github.com/mongomapper/mongomapper/runs/6594866261?check_suite_focus=true#step:6:9 (this is for Ruby 3.1).
Fixed #697