Just use one `Event` class. Reduces duplication, makes the tests easier
to read. It might seem like each tests needs a different kind of Struct,
since we make a new one for each test case.

Add missing require for `remove_possible_method`

https://github.com/rails/rails/commit/505537082849d912e8e29819655b80a573e93c0c
added a call to `remove_possible_method`, but didn't require the file
that defines it.

@prathamesh-sonpatki added the last missing commits in #28911.

[ci skip]

[ci skip]

Will backport to 5-1-stable, since the original PR landed there.

[ci skip]

Fix indentation [ci skip]

Added some more release notes for Active Record [ci skip]

Ensure input to distance_of_time_in_words is not nil

Remove unnecessary `skip_default_ids` and `allow_method_names_outside_object` attributes of select tag in `form_with`

* Internally all input is converted to time so that it can be treated
  uniformly.

Remove now-unneeded condition

* Now that all input is treated is converted to time, we no longer need
  to type check it.

Rename variables to clarify their purpose

Extract private method to normalize distance_of_time args to time

Update actionview changelog

Update the "Working with javascript" guide

Fix #28905

Evaluate belongs_to :default option against the owner, not the association

`form_with` is the new preferred method to deal with generation of
HTML forms and it enables Ajax support by default.

[ci skip]

Restore `fixtures :author_addresses`

This change reverted in eac6f369 but it is needed for data integrity.
See #25328.

Added a shared section to config/database.yml that will be loaded for all envs

PostgreSQL 10 does not convert `CURRENT_DATE` into `('now'::text)::date`

`sort_query_string_params` method is no more used

Do not try to encoding the parameters when the controller is not defined

Set consistent typecast ENV["VERBOSE"]

Pass request params to ActionMailer::Preview

Remove useless test case

Fix `current_page?` regression:

Remove mysql2 database adapter default username root

When you have a route that points to an nonexistent controller we raise
an exception.

This exception was being caught by the DebugExceptions middleware in
development, but when trying to render the error page, we are reading
the request format[[1][]]. To determine the request format we are reading
the format parameters[[2][]], and to be able to read the parameters we need
to encode them[[3][]]. This was raising another exception that to encode the
parameter we try to load the controller to determine if we need to
encode the parameters are binary[[4][]]. This new exception inside the
DebugExceptions middleware makes Rails to render a generic error page.

To avoid this new exception now we only encode the parameters when the
controller can be loaded.

Fixes #28892

[1]: https://github.com/rails/rails/blob/f52cdaac6336f99d13622ff9bda556a3124a4121/actionpack/lib/action_dispatch/middleware/debug_exceptions.rb#L80
[2]: https://github.com/rails/rails/blob/f52cdaac6336f99d13622ff9bda556a3124a4121/actionpack/lib/action_dispatch/http/mime_negotiation.rb#L63
[3]: https://github.com/rails/rails/blob/f52cdaac6336f99d13622ff9bda556a3124a4121/actionpack/lib/action_dispatch/http/parameters.rb#L58
[4]: https://github.com/rails/rails/blob/f52cdaac6336f99d13622ff9bda556a3124a4121/actionpack/lib/action_dispatch/http/parameters.rb#L88

[ci skip]

@dhh most changes that I mentioned in the talk were already there:
https://speakerdeck.com/claudiob/rails-5-dot-1-upcoming-features

I just added a few that were missing.

Some improvements to the 5.1 release notes