Merge pull request rails#50050 from seanpdoyle/config-x-read-with-arguments-no-method

Raise `ArgumentError` when reading from `config.x` with arguments

Author: rafaelfranca

railties/CHANGELOG.md

@@ -1,3 +1,11 @@
+*   Raise `ArgumentError` when reading `config.x.something` with arguments
+
+    ```ruby
+    config.x.this_works.this_raises true # raises ArgumentError
+    ```
+
+    *Sean Doyle*
+
 *   Add default PWA files for manifest and service-worker that are served from `app/views/pwa` and can be dynamically rendered through erb. Mount these files explicitly at the root with default routes in the generated routes file.
 
     *DHH*

railties/lib/rails/application/configuration.rb

@@ -569,10 +569,14 @@ def initialize
         def method_missing(method, *args)
           if method.end_with?("=")
             @configurations[:"#{method[0..-2]}"] = args.first
-          else
+          elsif args.none?
             @configurations.fetch(method) {
               @configurations[method] = ActiveSupport::OrderedOptions.new
             }
+          else
+            arguments = args.map(&:inspect)
+
+            raise ArgumentError.new("unexpected arguments (%s) while reading `%s` configuration" % [arguments.join(", "), method])
           end
         end
 

railties/test/application/configuration/custom_test.rb

@@ -36,6 +36,10 @@ def teardown
         assert_respond_to x, :i_do_not_exist
         assert_kind_of Method, x.method(:i_do_not_exist)
         assert_kind_of ActiveSupport::OrderedOptions, x.i_do_not_exist
+
+        assert_raises ArgumentError, match: "unexpected arguments (true, false) while reading `i_do_not_exist` configuration" do
+          x.i_do_not_exist(true, false)
+        end
       end
 
       private