Merge pull request rails#52371 from jhawthorn/time_with_zone_inspect

jhawthorn · web-flow · commit 88dc93e2c045 · 2024-09-05T15:45:52.000-07:00
Update TimeWithZone#inspect to match Ruby 1.9+ ISO 8601 format
diff --git a/activesupport/CHANGELOG.md b/activesupport/CHANGELOG.md
@@ -1,3 +1,7 @@
+*   `ActiveSupport::TimeWithZone#inspect` now uses ISO 8601 style time like `Time#inspect`
+
+    *John Hawthorn*
+
 *   `ActiveSupport::ErrorReporter#report` now assigns a backtrace to unraised exceptions.
 
     Previously reporting an un-raised exception would result in an error report without
diff --git a/activesupport/lib/active_support/time_with_zone.rb b/activesupport/lib/active_support/time_with_zone.rb
@@ -138,7 +138,7 @@ def zone
     #
     #   Time.zone.now.inspect # => "Thu, 04 Dec 2014 11:00:25.624541392 EST -05:00"
     def inspect
-      "#{time.strftime('%a, %d %b %Y %H:%M:%S.%9N')} #{zone} #{formatted_offset}"
+      "#{time.strftime('%F %H:%M:%S.%9N')} #{zone} #{formatted_offset}"
     end
 
     # Returns a string of the object's date and time in the ISO 8601 standard
diff --git a/activesupport/test/core_ext/time_with_zone_test.rb b/activesupport/test/core_ext/time_with_zone_test.rb

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ def zone`
`138`	`138`	`#`
`139`	`139`	`# Time.zone.now.inspect # => "Thu, 04 Dec 2014 11:00:25.624541392 EST -05:00"`
`140`	`140`	`def inspect`
`141`		`- "#{time.strftime('%a, %d %b %Y %H:%M:%S.%9N')} #{zone} #{formatted_offset}"`
	`141`	`+ "#{time.strftime('%F %H:%M:%S.%9N')} #{zone} #{formatted_offset}"`
`142`	`142`	`end`
`143`	`143`
`144`	`144`	`# Returns a string of the object's date and time in the ISO 8601 standard`