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
[deliver][spaceship][scan] fix build warnings in rspec #21660
Conversation
f1fc587
to
7ddd032
Compare
7ddd032
to
6e0ee4b
Compare
6e0ee4b
to
8a0a740
Compare
78acb96
to
4b651ad
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.
Great changes! Expect methods to not raise any error and expect them to raise a specific error 🙌
LGTM, just left some nitpick comments in case you wanna address them 🙇
@@ -1,5 +1,12 @@ | |||
describe Spaceship::Tunes::DeviceType do | |||
describe "type identifiers" do | |||
before(:each) do | |||
# let's catch those calls to avoid polluting the output | |||
# Note: warning has a different signature in newer versions of ruby |
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.
Is this the reason why we have 2 allow
calls? If so, can we specify this in the comment? I think it'd be more clear, like:
# Note: warning has a different signature in newer versions of ruby | |
# Note: warnings have different signature depending on the Ruby version, hence why we need more than one allow(…) |
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.
I made an update.
Co-authored-by: Roger Oba <rogerluan.oba@gmail.com>
* Remove 'warning: already initialized constant ScreenSize' * Remove warning of rspec raise_error being used too widely * Remove warning related to rspec around usage * Reduce amount of deprecation output. * Remove chruby as we are not installing it. Is it needed? * Rubocop fix * Update spaceship/spec/tunes/device_type_spec.rb Co-authored-by: Roger Oba <rogerluan.oba@gmail.com> * Clean comment --------- Co-authored-by: Roger Oba <rogerluan.oba@gmail.com>
build output is cluttered with unecessary error messages. Removing some of them.
WIP
Checklist
bundle exec rspec
from the root directory to see all new and existing tests passbundle exec rubocop -a
to ensure the code style is validci/circleci
builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)Motivation and Context
Description
Testing Steps