Raising an error when nil or non-hash is passed to update_attributes.
Remove HelperyTestHelper not used in any test

HelperyTestHelper was introduced in 66ef922 by @josevalim
to pair with HelperyTestController. This test controller was
later removed in e10a253 by @strzalek, leaving HelperyTestHelper unused
exposing target records on the preloader
do not access internal data structures
generate fewer objects when grouping
extract exception raising, clean up group_by statement
just read the attribute rather than `send`ing
lhs preload is always a single preload, so just preload one
stop depending on preloader interals so we can create fewer preloader
objects
the hash should only ever be length one, so decompose it
push `run` up to preload
push running the preloaders up
access preloaders independently of model mutations
fix method name
split up construction and mutation
preserve order on the RHS query
return a list rather than hash
combine methods so we can reuse preloaders
cache associated target records hash
Add notes about database connection pool [ci skip]