Update RecordNotFound exception cases to include a message with the
Model that the Record was not found in.
[ci skip] Reword a comment in concern's documentation

Nothing biggie. Skimmed through `ActiveSupport::Concern` docs these days
and this one comment seemed a bit off.
Merge pull request #16981 from gsamokovarov/concern-docs-comment-reword

[ci skip] Reword a comment in concern's documentation
Merge pull request #15791 from zev/add_model_to_recordnotfound_message

Update RecordNotFound exception cases to include a message with the
Quick fix to address the borken build caused by #15791

This is pretty lame though, so feel free to send a PR with a better fix! ;)

cc @zev @tenderlove
Fix find_by with associations not working with adequate record

For now, we will just skip the cache when a non-column key is used in the hash.
If the future, we can probably move some of the logic in PredicateBuilder.expand
up the chain to make caching possible for association queries.

Closes #16903
Fixes #16884
No need to call to_sym here

The hash is now string-keyed, and [_]reflect_on_association calls `to_s` on the
argument anyway.
ActiveRecord CHANGELOG improvements [skip ci]
Merge pull request #16987 from jonatack/patch-8

ActiveRecord CHANGELOG improvements [skip ci]
Split the redirection tests out of dispatch/routing_test.rb into dispatch/routing/redirection_test.rb

Nest dispatch/routing/concerns_test consistent with the other routing test files.

Removed some unnecessary requires from dispatch/routing_test
Fail informatively when redirect() is called with an opaque uri.

Absent this fix the call fails at runtime with “undefined method `empty?' for nil:NilClass” for uri.path.empty? because opaque uris have no path.