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
configure rspec with disable_monkey_patching! #4802
Conversation
b222539
to
124c163
Compare
I think you only need to namespace the outer |
124c163
to
96b5f47
Compare
@deivid-rodriguez , yes wrong regexp ... |
96b5f47
to
aa7007e
Compare
@deivid-rodriguez , rebased and fixed |
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.
Looks good to me other than some minor comments.
spec/unit/application_spec.rb
Outdated
|
||
let(:application) do | ||
ActiveAdmin::Application.new.tap do |app| | ||
# Manually override the load paths becuase RSpec messes these up | ||
# Manually override the load paths becuase RSpec.messes these up |
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.
Another bad substitution. Maybe you can correct the "becuase" typo since you're here.
@@ -92,10 +92,10 @@ | |||
end | |||
end | |||
end | |||
end # describe rendering rows | |||
end #describe rendering rows |
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.
This line should be left alone.
|
||
end | ||
end # describe dsl styles | ||
end #describe dsl styles |
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.
ditto.
@@ -232,7 +232,7 @@ | |||
end | |||
end | |||
end | |||
end # describe rendering rows | |||
end #describe rendering rows |
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.
Ditto.
see https://www.relishapp.com/rspec/rspec-core/v/3-0/docs/configuration/zero-monkey-patching-mode disable all monkey patching done by RSpec
aa7007e
to
5c579a8
Compare
@deivid-rodriguez , thanks ! it was too late for me , sorry for my inattention |
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.
Looks good to me!
No problem! |
see https://www.relishapp.com/rspec/rspec-core/v/3-0/docs/configuration/zero-monkey-patching-mode
disable all monkey patching done by RSpec