[#5775 state:resolved]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

This was a regression introduced in 5c858220. We bring
sorting back because people rely on it, eg for constructing consistent cache keys.

Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

[#5601 state:committed]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

In [32b82e4c], the committer has switch
methods in AS::Duration to use `Time.current` to return the correct
duration based on the `Time.default_zone` instead of using `Time.now`.

[#5607 state:committed]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

- as_json in ActiveModel should return a hash
  and handle :only/:except/:methods options
- Array and Hash should call as_json on their elements
- json methods should not modify options argument

[#5374 state:committed]
Signed-off-by: NJeremy Kemper <jeremy@bitsweat.net>

Signed-off-by: NXavier Noria <fxn@hashref.com>

[#5375 state:resolved]
Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Signed-off-by: NJosé Valim <jose.valim@gmail.com>

no test was failing. Tests have been added to ensure that commenting
out the code within if loop would cause test failure.
Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

Signed-off-by: Nwycats <wycats@gmail.com>

code gardening: we have assert_(nil|blank|present), more concise, with better default failure messages - let's use them

adds test coverage to ensure DateTime#advance processes first the date deltas, and then the time deltas

adds test coverage for edge-cases of Array.wrap, and better documentation for how it differs from Kernel#Array

This resolves issues for libraries which use '_' prefixed keys in their
attributes hash, such as Mongoid. A key like "_id" or "_type" will no
longer be converted to "<-id>" and "<-type>".
Signed-off-by: Nwycats <wycats@gmail.com>