Phil Pirozhkov

> figure out which one is better I'd go with `+=`, keeping in mind inheritance and mixins.

Ping @tejasbubane

Absolutely. The guide is behind the reality in this regard. There's a mention of request specs here https://github.com/rubocop-hq/rspec-style-guide/#rails-integration, but the problem with slow request specs [is long gone](https://www.youtube.com/watch?v=oT74HLvDo_A). I'm against...

Also this https://github.com/rails/rails/issues/22496

Agree, functional controller tests (that map to RSpec Rails' Controller specs) are said to [use `ActionDispatch::IntegrationTest` in Rails's testing guide](https://guides.rubyonrails.org/testing.html#functional-tests-for-your-controllers), not [`ActionController::TestCase::Behavior` as RSpec Rails states](https://relishapp.com/rspec/rspec-rails/v/4-0/docs/controller-specs). This needs to be...

https://discuss.rubyonrails.org/t/need-controller-side-effects-testing-advise/76103

Nice find, @klyonrad! So, from the Rails Guidelines on [Functional Tests](https://guides.rubyonrails.org/testing.html#functional-tests-for-your-controllers) (RSpec's Controller Specs): > testing how your actions handle the requests and the expected result or response, in some...

Yep, local to an example group, so that it's not accessible in the other example groups. It depends on the number of examples in the group and the complexity of...

As we've discussed previously, `let` are overused/often abused. Not sure if parts of the test code being extracted to a plain method fall into the same category, and is not...

This recommendation is taken from an Effective Testing with RSpec book. I did not take the example from it, however, it is using the same argumentation, the difference that in...