Add separate test to ensure that `delegate` with `:private` option returns correct value

Remove extra comments `# Asking for private method` in activesupport/test/core_ext/module_test.rb
Improve docs of using `delegate` with `:private`
Update changelog of #31944

1 files changed, 5 insertions, 5 deletions

diff --git a/activesupport/lib/active_support/core_ext/module/delegation.rb b/activesupport/lib/active_support/core_ext/module/delegation.rb
index 426e34128f..45d16515d2 100644
--- a/activesupport/lib/active_support/core_ext/module/delegation.rb
+++ b/activesupport/lib/active_support/core_ext/module/delegation.rb
@@ -114,7 +114,7 @@ class Module
   #   invoice.customer_name    # => 'John Doe'
   #   invoice.customer_address # => 'Vimmersvej 13'
   #
-  # If you want the delegated method to be a private method,
+  # If you want the delegate methods to be a private,
   # use the <tt>:private</tt> option.
   #
   #   class User < ActiveRecord::Base
@@ -127,10 +127,10 @@ class Module
   #     end
   #   end
   #
-  #   User.new.age # 2
-  #   User.new.first_name # Tomas
-  #   User.new.date_of_birth # NoMethodError: private method `date_of_birth' called for #<User:0x00000008221340>
-  #   User.new.religion # NoMethodError: private method `religion' called for #<User:0x00000008221340>
+  #   User.new.age # => 2
+  #   User.new.first_name # => "Tomas"
+  #   User.new.date_of_birth # => NoMethodError: private method `date_of_birth' called for #<User:0x00000008221340>
+  #   User.new.religion # => NoMethodError: private method `religion' called for #<User:0x00000008221340>
   #
   # If the target is +nil+ and does not respond to the delegated method a
   # +Module::DelegationError+ is raised. If you wish to instead return +nil+,