much-rails 0.2.8 → 0.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b16874b1abd3bada37a15e501101bf2b1f3b0f49949e939078a06fe5df861ee
-  data.tar.gz: abf58e64838b2a14a2a2b3d243f6fb601dbe93fb1537f00a9cfca12e09c2c16a
+  metadata.gz: 4e21c445f3eec6146fc636f67a3e98b4f40dde3c916c6f25f6bb02af8f6dfc38
+  data.tar.gz: ffcc8deab5152d91d1e433d5bfc174e359fa02a2d5e43b57c12f2b7dfed9fdab
 SHA512:
-  metadata.gz: 3a971f04f2f90b7c5adc8284d7fe7b6ff11a115d637183921c6bc2f12867e226f6ce741513068cf9418f6094778e9b2adf27e74fafaf11fcf3e621913271058f
-  data.tar.gz: c0ba8ef4a6242a9e0d1ccfe56d1d6b1a6aeec49f994e47e9d4c52553bc13f6311fe3f5e2a41ce7105c30b3d7bbac79f68399d6e2f75b5bbfbde85aad6d457b23
+  metadata.gz: 3299e07aff0d7304b3cf84ca6df14e53e3dcdd0d63690ec9bed261217c04fdae8058e0314840b276a1e183a18b87e2ba47d930389c77eda82638f7637577437a
+  data.tar.gz: 5fe7641d69715ffe58daa889edf8635b3d0ae614d500e9205f44c4798aadd5d6c8c021d4a40f7c0afcc58db8c31ea10252a91e087fb7d8aa44f110fd883e1628

data/lib/much-rails.rb

@@ -18,6 +18,7 @@ require "much-rails/destroy_action"
 require "much-rails/destroy_service"
 require "much-rails/has_slug"
 require "much-rails/input_value"
+require "much-rails/invalid_error"
 require "much-rails/json"
 require "much-rails/layout"
 require "much-rails/not_given"

data/lib/much-rails/action.rb

@@ -31,7 +31,7 @@ module MuchRails::Action
 
     add_config :much_rails_action
 
-    attr_reader :params, :current_session, :request, :errors
+    attr_reader :params, :request, :errors
   end
 
   mixin_class_methods do
@@ -112,11 +112,10 @@ module MuchRails::Action
   end
 
   mixin_instance_methods do
-    def initialize(params: nil, current_session: nil, request: nil)
+    def initialize(params: nil, request: nil)
       @params = params.to_h.with_indifferent_access
-      @current_session = current_session
       @request = request
-      @errors = Hash.new{ |hash, key| hash[key] = [] }
+      @errors = HashWithIndifferentAccess.new{ |hash, key| hash[key] = [] }
     end
 
     def on_call
@@ -150,8 +149,17 @@ module MuchRails::Action
       @much_rails_successful_action
     end
 
+    def controller_session=(value)
+      @controller_session = value
+    end
+
     private
 
+    def controller_session
+      @controller_session ||=
+        request&.env&.[]("action_controller.instance")&.session
+    end
+
     def default_action_success_result
       MuchRails::Action::HeadResult.new(:ok)
     end

data/lib/much-rails/action/controller.rb

@@ -9,30 +9,31 @@ module MuchRails::Action; end
 # MuchRails::Action::Controller defines the behaviors for controllers processing
 # MuchRails::Actions.
 module MuchRails::Action::Controller
-  DEFAULT_ACTION_CLASS_FORMAT = :any
-
   include MuchRails::Mixin
 
+  def self.DEFAULT_ACTION_CLASS_FORMAT
+    :any
+  end
+
   mixin_included do
     attr_reader :much_rails_action_class
 
-    before_action(
+    prepend_before_action(
       :require_much_rails_action_class,
-      only: MuchRails::Action::Router::CONTROLLER_CALL_ACTION_METHOD_NAME,
+      only: MuchRails::Action::Router.CONTROLLER_CALL_ACTION_METHOD_NAME,
     )
     before_action :permit_all_much_rails_action_params
   end
 
   mixin_instance_methods do
     define_method(
-      MuchRails::Action::Router::CONTROLLER_CALL_ACTION_METHOD_NAME,
+      MuchRails::Action::Router.CONTROLLER_CALL_ACTION_METHOD_NAME,
     ) do
       respond_to do |format|
         format.public_send(much_rails_action_class_format) do
           result =
             much_rails_action_class.call(
               params: much_rails_action_params,
-              current_session: current_session,
               request: request,
             )
           instance_exec(result, &result.execute_block)
@@ -41,7 +42,7 @@ module MuchRails::Action::Controller
     end
 
     define_method(
-      MuchRails::Action::Router::CONTROLLER_NOT_FOUND_METHOD_NAME,
+      MuchRails::Action::Router.CONTROLLER_NOT_FOUND_METHOD_NAME,
     ) do
       respond_to do |format|
         format.html do
@@ -51,11 +52,12 @@ module MuchRails::Action::Controller
     end
 
     def much_rails_action_class_name
-      "::#{params[MuchRails::Action::Router::ACTION_CLASS_PARAM_NAME]}"
+      "::#{params[MuchRails::Action::Router.ACTION_CLASS_PARAM_NAME]}"
     end
 
     def much_rails_action_class_format
-      much_rails_action_class.format || DEFAULT_ACTION_CLASS_FORMAT
+      much_rails_action_class.format ||
+      MuchRails::Action::Controller.DEFAULT_ACTION_CLASS_FORMAT
     end
 
     def much_rails_action_params
@@ -67,7 +69,7 @@ module MuchRails::Action::Controller
           params
             .to_h
             .except(
-              MuchRails::Action::Router::ACTION_CLASS_PARAM_NAME,
+              MuchRails::Action::Router.ACTION_CLASS_PARAM_NAME,
               :controller,
               :action,
             ),

data/lib/much-rails/action/router.rb

@@ -6,10 +6,21 @@ module MuchRails; end
 module MuchRails::Action; end
 
 class MuchRails::Action::Router < MuchRails::Action::BaseRouter
-  DEFAULT_CONTROLLER_NAME = "application"
-  CONTROLLER_CALL_ACTION_METHOD_NAME = :much_rails_call_action
-  CONTROLLER_NOT_FOUND_METHOD_NAME = :much_rails_not_found
-  ACTION_CLASS_PARAM_NAME = :much_rails_action_class_name
+  def self.DEFAULT_CONTROLLER_NAME
+    "application"
+  end
+
+  def self.CONTROLLER_CALL_ACTION_METHOD_NAME
+    :much_rails_call_action
+  end
+
+  def self.CONTROLLER_NOT_FOUND_METHOD_NAME
+    :much_rails_not_found
+  end
+
+  def self.ACTION_CLASS_PARAM_NAME
+    :much_rails_action_class_name
+  end
 
   def self.url_class
     MuchRails::Action::Router::URL
@@ -28,7 +39,10 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
       raise ArgumentError, "routes file `#{file_path.inspect}` does not exist."
     end
 
-    new(routes_file_name, controller_name: controller_name) do
+    new(
+      routes_file_name.to_s.split(File::SEPARATOR).join("_"),
+      controller_name: controller_name,
+    ) do
       instance_eval(File.read(file_path), file_path.to_s, 1)
     end
   end
@@ -38,7 +52,7 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
   def initialize(name = nil, controller_name: nil, &block)
     super(name, &block)
 
-    @controller_name = controller_name || DEFAULT_CONTROLLER_NAME
+    @controller_name = controller_name || self.class.DEFAULT_CONTROLLER_NAME
   end
 
   # Example:
@@ -49,8 +63,10 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
   #   end
   def apply_to(application_routes_draw_scope)
     validate!
-    draw_url_to   = "#{controller_name}##{CONTROLLER_NOT_FOUND_METHOD_NAME}"
-    draw_route_to = "#{controller_name}##{CONTROLLER_CALL_ACTION_METHOD_NAME}"
+    draw_url_to =
+      "#{controller_name}##{self.class.CONTROLLER_NOT_FOUND_METHOD_NAME}"
+    draw_route_to =
+      "#{controller_name}##{self.class.CONTROLLER_CALL_ACTION_METHOD_NAME}"
 
     definition_names = Set.new
 
@@ -63,7 +79,8 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
           as: (definition.name if definition_names.add?(definition.name)),
           defaults:
             definition.default_params.merge({
-              ACTION_CLASS_PARAM_NAME => request_type_action.class_name,
+              self.class.ACTION_CLASS_PARAM_NAME =>
+                request_type_action.class_name,
               "format" => request_type_action.format,
             }),
           constraints: request_type_action.constraints_lambda,
@@ -79,7 +96,8 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
         as: (definition.name if definition_names.add?(definition.name)),
         defaults:
           definition.default_params.merge({
-            ACTION_CLASS_PARAM_NAME => definition.default_action_class_name,
+            self.class.ACTION_CLASS_PARAM_NAME =>
+              definition.default_action_class_name,
             "format" => definition.default_action_format,
           }),
       )

data/lib/much-rails/change_action_result.rb

@@ -27,12 +27,15 @@ class MuchRails::ChangeActionResult
 
     @service_result = save_service_result
 
-    @service_result.validation_errors ||= {}
+    @service_result.validation_errors ||= HashWithIndifferentAccess.new
   end
 
   def validation_errors
     @validation_errors ||=
-      service_result.get_for_all_results(:validation_errors).to_h
+      service_result
+        .get_for_all_results(:validation_errors)
+        .to_h
+        .with_indifferent_access
   end
 
   def validation_error_messages
@@ -40,7 +43,7 @@ class MuchRails::ChangeActionResult
   end
 
   def extract_validation_error(field_name)
-    validation_errors.delete(field_name)
+    Array.wrap(validation_errors.delete(field_name)).compact
   end
 
   def any_unextracted_validation_errors?

data/lib/much-rails/config.rb

@@ -1,55 +1,7 @@
 # frozen_string_literal: true
 
-require "much-rails/mixin"
+require "much-config"
 
-module MuchRails; end
-
-# MuchRails::Config is a mix-in to implement object DSL configuration.
-module MuchRails::Config
-  include MuchRails::Mixin
-
-  mixin_class_methods do
-    def add_config(name = nil, method_name: nil)
-      config_method_name, config_class_name, configure_method_name =
-        much_rails_config_names(name, method_name)
-
-      instance_eval(<<~RUBY, __FILE__, __LINE__ + 1)
-        def #{config_method_name}
-          @#{config_method_name} ||= self::#{config_class_name}.new
-        end
-
-        def #{configure_method_name}
-          yield(#{config_method_name}) if block_given?
-        end
-      RUBY
-    end
-
-    def add_instance_config(name = nil, method_name: nil)
-      config_method_name, config_class_name, configure_method_name =
-        much_rails_config_names(name, method_name)
-
-      instance_eval(<<~RUBY, __FILE__, __LINE__ + 1)
-        define_method(:#{config_method_name}) do
-          @#{config_method_name} ||= self.class::#{config_class_name}.new
-        end
-
-        define_method(:#{configure_method_name}) do |&block|
-          block.call(#{config_method_name}) if block
-        end
-      RUBY
-    end
-
-    private
-
-    def much_rails_config_names(name, method_name)
-      name_prefix = name.nil? ? "" : "#{name.to_s.underscore}_"
-      config_method_name = (method_name || "#{name_prefix}config").to_s
-      config_class_name = "#{name_prefix.classify}Config"
-
-      name_suffix = name.nil? ? "" : "_#{name.to_s.underscore}"
-      configure_method_name = "configure#{name_suffix}"
-
-      [config_method_name, config_class_name, configure_method_name]
-    end
-  end
+module MuchRails
+  Config = MuchConfig
 end

data/lib/much-rails/destroy_service.rb

@@ -1,10 +1,12 @@
 # frozen_string_literal: true
 
+require "much-rails/invalid_error"
 require "much-rails/mixin"
 require "much-rails/records"
 require "much-rails/records/validate_destroy"
 require "much-rails/result"
 require "much-rails/service"
+require "much-rails/service_validation_errors"
 
 module MuchRails; end
 
@@ -51,6 +53,13 @@ module MuchRails::DestroyService
                 validation_error_messages: ex.error_full_messages.to_a,
               )
             end
+            e.add(MuchRails::InvalidError) do |ex|
+              MuchRails::SaveService::FailureResult.new(
+                exception: ex,
+                validation_errors: ex.errors,
+                validation_error_messages: ex.error_messages,
+              )
+            end
           end
     end
   end

data/lib/much-rails/invalid_error.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module MuchRails; end
+
+# MuchRails::InvalidError allows for raising and auto-rescueing general
+# validation errors outside the scope of e.g. an ActiveRecord save.
+class MuchRails::InvalidError < StandardError
+  attr_reader :errors
+
+  def initialize(backtrace: nil, **errors)
+    @errors = errors
+
+    super(@errors.inspect)
+    set_backtrace(backtrace) if backtrace
+  end
+
+  def error_messages
+    @errors.to_a.map do |(field_name, message)|
+      "#{field_name}: #{Array.wrap(message).to_sentence}"
+    end
+  end
+end

data/lib/much-rails/save_service.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "active_record"
+require "much-rails/invalid_error"
 require "much-rails/mixin"
 require "much-rails/result"
 require "much-rails/service"
@@ -52,6 +53,13 @@ module MuchRails::SaveService
                   ex.record&.errors&.full_messages.to_a,
               )
             end
+            e.add(MuchRails::InvalidError) do |ex|
+              MuchRails::SaveService::FailureResult.new(
+                exception: ex,
+                validation_errors: ex.errors,
+                validation_error_messages: ex.error_messages,
+              )
+            end
           end
     end
   end

data/lib/much-rails/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MuchRails
-  VERSION = "0.2.8"
+  VERSION = "0.4.3"
 end

data/much-rails.gemspec

@@ -23,20 +23,21 @@ Gem::Specification.new do |gem|
 
   gem.required_ruby_version = "~> 2.5"
 
-  gem.add_development_dependency("much-style-guide", ["~> 0.6.0"])
-  gem.add_development_dependency("assert",           ["~> 2.19.5"])
+  gem.add_development_dependency("much-style-guide", ["~> 0.6.6"])
+  gem.add_development_dependency("assert",           ["~> 2.19.6"])
   gem.add_development_dependency("rails",            ["> 5.0", "< 7.0"])
 
   gem.add_dependency("activerecord",   ["> 5.0", "< 7.0"])
   gem.add_dependency("activesupport",  ["> 5.0", "< 7.0"])
-  gem.add_dependency("dassets",        ["~> 0.15.2"])
+  gem.add_dependency("dassets",        ["~> 0.15.3"])
   gem.add_dependency("dassets-erubi",  ["~> 0.1.1"])
   gem.add_dependency("dassets-sass",   ["~> 0.5.1"])
   gem.add_dependency("much-boolean",   ["~> 0.2.1"])
+  gem.add_dependency("much-config",    ["~> 0.1.0"])
   gem.add_dependency("much-decimal",   ["~> 0.1.3"])
   gem.add_dependency("much-mixin",     ["~> 0.2.4"])
   gem.add_dependency("much-not-given", ["~> 0.1.2"])
   gem.add_dependency("much-result",    ["~> 0.1.3"])
   gem.add_dependency("much-slug",      ["~> 0.1.2"])
-  gem.add_dependency("oj",             ["~> 3.10"])
+  gem.add_dependency("oj",             ["~> 3.11"])
 end

data/test/support/config/routes/test/subrouter.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+url :root, "/subrouter"

data/test/support/fake_action_controller.rb

@@ -13,20 +13,31 @@ module FakeActionController
   end
 
   mixin_class_methods do
-    def before_action(method_name, **)
-      before_actions << method_name
+    def before_action(*args, &block)
+      before_action_calls << Assert::StubCall.new(*args, &block)
     end
 
-    def before_actions
-      @before_actions ||= []
+    def prepend_before_action(*args, &block)
+      prepend_before_action_calls << Assert::StubCall.new(*args, &block)
+    end
+
+    def before_action_calls
+      @before_action_calls ||= []
+    end
+
+    def prepend_before_action_calls
+      @prepend_before_action_calls ||= []
     end
   end
 
   mixin_instance_methods do
     def initialize(params)
       @params = FakeParams.new(params)
-      self.class.before_actions.each do |before_action|
-        public_send(before_action)
+      self.class.prepend_before_action_calls.each do |before_action_call|
+        public_send(before_action_call.args.first)
+      end
+      self.class.before_action_calls.each do |before_action_call|
+        public_send(before_action_call.args.first)
       end
     end
 

data/test/unit/action/controller_tests.rb

@@ -17,8 +17,8 @@ module MuchRails::Action::Controller
       assert_that(subject).includes(MuchRails::Mixin)
     end
 
-    should "know its constants" do
-      assert_that(subject::DEFAULT_ACTION_CLASS_FORMAT).equals(:any)
+    should "know its attributes" do
+      assert_that(subject.DEFAULT_ACTION_CLASS_FORMAT).equals(:any)
     end
   end
 
@@ -29,6 +29,19 @@ module MuchRails::Action::Controller
     let(:receiver_class) do
       Class.new{ include FakeActionController }
     end
+
+    should "be configured as expected" do
+      assert_that(subject.prepend_before_action_calls.size).equals(1)
+      assert_that(subject.prepend_before_action_calls.last.args)
+        .equals([
+          :require_much_rails_action_class,
+          only: MuchRails::Action::Router.CONTROLLER_CALL_ACTION_METHOD_NAME,
+        ])
+
+      assert_that(subject.before_action_calls.size).equals(1)
+      assert_that(subject.before_action_calls.last.args)
+        .equals([:permit_all_much_rails_action_params])
+    end
   end
 
   class ReceiverInitTests < ReceiverTests
@@ -41,7 +54,7 @@ module MuchRails::Action::Controller
 
     let(:params1) do
       {
-        MuchRails::Action::Router::ACTION_CLASS_PARAM_NAME => "Actions::Show",
+        MuchRails::Action::Router.ACTION_CLASS_PARAM_NAME => "Actions::Show",
         controller: "actions",
         action: "show",
         nested: {
@@ -53,10 +66,10 @@ module MuchRails::Action::Controller
     should have_readers :much_rails_action_class
 
     should have_imeths(
-      MuchRails::Action::Router::CONTROLLER_CALL_ACTION_METHOD_NAME,
+      MuchRails::Action::Router.CONTROLLER_CALL_ACTION_METHOD_NAME,
     )
     should have_imeths(
-      MuchRails::Action::Router::CONTROLLER_NOT_FOUND_METHOD_NAME,
+      MuchRails::Action::Router.CONTROLLER_NOT_FOUND_METHOD_NAME,
     )
     should have_imeths :much_rails_action_class_name
     should have_imeths :much_rails_action_class_format
@@ -69,7 +82,7 @@ module MuchRails::Action::Controller
         .equals("::Actions::Show")
 
       assert_that(subject.much_rails_action_class_format)
-        .equals(unit_module::DEFAULT_ACTION_CLASS_FORMAT)
+        .equals(unit_module.DEFAULT_ACTION_CLASS_FORMAT)
 
       assert_that(subject.much_rails_action_params)
         .equals(
@@ -91,7 +104,7 @@ module MuchRails::Action::Controller
 
     let(:params1) do
       {
-        MuchRails::Action::Router::ACTION_CLASS_PARAM_NAME =>
+        MuchRails::Action::Router.ACTION_CLASS_PARAM_NAME =>
           "Actions::Unknown",
       }
     end

data/test/unit/action/router_tests.rb

@@ -16,17 +16,17 @@ class MuchRails::Action::Router
 
     should have_imeths :url_class
 
-    should "be a BaseRouter" do
+    should "be configured as expected" do
       assert_that(subject < MuchRails::Action::BaseRouter).is_true
     end
 
-    should "know its constants" do
-      assert_that(subject::DEFAULT_CONTROLLER_NAME).equals("application")
-      assert_that(subject::CONTROLLER_CALL_ACTION_METHOD_NAME)
+    should "know its attributes" do
+      assert_that(subject.DEFAULT_CONTROLLER_NAME).equals("application")
+      assert_that(subject.CONTROLLER_CALL_ACTION_METHOD_NAME)
         .equals(:much_rails_call_action)
-      assert_that(subject::CONTROLLER_NOT_FOUND_METHOD_NAME)
+      assert_that(subject.CONTROLLER_NOT_FOUND_METHOD_NAME)
         .equals(:much_rails_not_found)
-      assert_that(subject::ACTION_CLASS_PARAM_NAME)
+      assert_that(subject.ACTION_CLASS_PARAM_NAME)
         .equals(:much_rails_action_class_name)
       assert_that(subject.url_class).equals(unit_class::URL)
     end
@@ -45,6 +45,12 @@ class MuchRails::Action::Router
       assert_that(router.url_set.fetch(:root).path).equals("/")
     end
 
+    should "eval the nested named routes file" do
+      router = unit_class.load("test/subrouter")
+      assert_that(router.url_set).is_not_empty
+      assert_that(router.url_set.fetch(:root).path).equals("/subrouter")
+    end
+
     should "complain if no file name given" do
       assert_that{ unit_class.load([nil, "", "  "]) }.raises(ArgumentError)
     end
@@ -64,7 +70,7 @@ class MuchRails::Action::Router
 
     should "know its attributes" do
       assert_that(subject.controller_name)
-        .equals(unit_class::DEFAULT_CONTROLLER_NAME)
+        .equals(unit_class.DEFAULT_CONTROLLER_NAME)
 
       router = unit_class.new(controller_name: controller_name1)
       assert_that(router.controller_name).equals(controller_name1)
@@ -99,13 +105,13 @@ class MuchRails::Action::Router
 
       expected_draw_url_to =
         "#{subject.controller_name}"\
-        "##{unit_class::CONTROLLER_NOT_FOUND_METHOD_NAME}"
+        "##{unit_class.CONTROLLER_NOT_FOUND_METHOD_NAME}"
       expected_draw_route_to =
         "#{subject.controller_name}"\
-        "##{unit_class::CONTROLLER_CALL_ACTION_METHOD_NAME}"
+        "##{unit_class.CONTROLLER_CALL_ACTION_METHOD_NAME}"
       expected_default_defaults =
         {
-          unit_class::ACTION_CLASS_PARAM_NAME => default_class_name,
+          unit_class.ACTION_CLASS_PARAM_NAME => default_class_name,
           "format" => :html,
         }
 
@@ -117,7 +123,7 @@ class MuchRails::Action::Router
           as: url_name,
           defaults:
             {
-              unit_class::ACTION_CLASS_PARAM_NAME => request_type_class_name,
+              unit_class.ACTION_CLASS_PARAM_NAME => request_type_class_name,
               "format" => :html,
             },
           constraints: request_type_proc,

data/test/unit/action/unprocessable_entity_result_tests.rb

@@ -20,7 +20,7 @@ class MuchRails::Action::UnprocessableEntityResult
     let(:controller1){ FakeController.new(params1) }
     let(:params1) do
       {
-        MuchRails::Action::Router::ACTION_CLASS_PARAM_NAME => "Actions::Show",
+        MuchRails::Action::Router.ACTION_CLASS_PARAM_NAME => "Actions::Show",
       }
     end
     let(:errors1) do

data/test/unit/action_tests.rb

@@ -176,11 +176,7 @@ module MuchRails::Action
   class InitTests < ReceiverTests
     desc "when init"
     subject do
-      receiver_class.new(
-        params: params1,
-        current_session: current_session1,
-        request: request1,
-      )
+      receiver_class.new(params: params1, request: request1)
     end
 
     let(:receiver_class) do
@@ -203,7 +199,7 @@ module MuchRails::Action
           end
 
           if params[:fail_custom_validation]
-            errors[:custom_validation] << "ERROR1"
+            errors["custom_validation"] << "ERROR1"
           end
         end
 
@@ -218,6 +214,10 @@ module MuchRails::Action
         on_after_call do
           @after_call_called = true
         end
+
+        def action_controller_session
+          controller_session
+        end
       end
     end
 
@@ -234,16 +234,41 @@ module MuchRails::Action
         active: "true",
       }
     end
-    let(:current_session1){ "CURRENT SESSION 1" }
-    let(:request1){ "REQUEST 1" }
+    let(:request1){ FakeRequest.new }
 
-    should have_readers :params, :current_session, :request, :errors
+    should have_readers :params, :request, :errors
+    should have_writers :controller_session
     should have_imeths :on_call, :valid_action?, :successful_action?
 
     should "know its attributes" do
       assert_that(subject.params).equals(params1.with_indifferent_access)
-      assert_that(subject.current_session).equals(current_session1)
       assert_that(subject.request).equals(request1)
+      assert_that(subject.action_controller_session).is_a(FakeSession)
+
+      receiver = receiver_class.new(params: params1)
+      assert_that(receiver.action_controller_session).is_nil
+
+      controller_session = FakeRequest.new
+      receiver.controller_session = controller_session
+      assert_that(receiver.action_controller_session).is(controller_session)
+
+      receiver =
+        receiver_class.new(params: params1, request: FakeRequestWithNoEnv.new)
+      assert_that(receiver.action_controller_session).is_nil
+
+      receiver =
+        receiver_class.new(
+          params: params1,
+          request: FakeRequestWithNoControllerInstance.new,
+        )
+      assert_that(receiver.action_controller_session).is_nil
+
+      receiver =
+        receiver_class.new(
+          params: params1,
+          request: FakeRequestWithNoControllerInstance.new,
+        )
+      assert_that(receiver.action_controller_session).is_nil
     end
 
     should "return the expected Result" do
@@ -409,4 +434,31 @@ module MuchRails::Action
       assert_that(result.errors[:custom_validation]).includes("ERROR1")
     end
   end
+
+  class FakeController
+    def session
+      FakeSession.new
+    end
+  end
+
+  class FakeSession
+  end
+
+  class FakeRequest
+    def env
+      { "action_controller.instance" => FakeController.new }
+    end
+  end
+
+  class FakeRequestWithNoControllerInstance
+    def env
+      { "action_controller.instance" => nil }
+    end
+  end
+
+  class FakeRequestWithNoEnv
+    def env
+      nil
+    end
+  end
 end

data/test/unit/assets_tests.rb

@@ -48,7 +48,7 @@ module MuchRails::Assets
 
     should "configure the app's app/assets/css folder as a source" do
       source =
-        subject.config.sources.detect do |source|
+        subject.config.sources.find do |source|
           source.path == FakeRails.root.join("app", "assets", "css").to_s
         end
 
@@ -63,7 +63,7 @@ module MuchRails::Assets
 
     should "configure the app's app/assets/img folder as a source" do
       source =
-        subject.config.sources.detect do |source|
+        subject.config.sources.find do |source|
           source.path == FakeRails.root.join("app", "assets", "img").to_s
         end
 
@@ -74,7 +74,7 @@ module MuchRails::Assets
 
     should "configure the app's app/assets/js folder as a source" do
       source =
-        subject.config.sources.detect do |source|
+        subject.config.sources.find do |source|
           source.path == FakeRails.root.join("app", "assets", "js").to_s
         end
 
@@ -87,7 +87,7 @@ module MuchRails::Assets
 
     should "configure the app's app/assets/vendor folder as a source" do
       source =
-        subject.config.sources.detect do |source|
+        subject.config.sources.find do |source|
           source.path == FakeRails.root.join("app", "assets", "vendor").to_s
         end
 

data/test/unit/change_action_result_tests.rb

@@ -64,19 +64,27 @@ class MuchRails::ChangeActionResult
   class ValidationMethodsTests < UnitTests
     desc "validation methods"
     subject do
-      unit_class.failure(validation_errors: {
-        name: ["NAME ERROR"],
-        other: ["OTHER ERROR"],
-      })
+      unit_class.failure(
+        validation_errors: {
+          name: ["NAME ERROR"],
+          other: ["OTHER ERROR"],
+          empty: [],
+          none: nil,
+        },
+      )
     end
 
     should "know its attributes" do
       # validation_errors
       assert_that(subject.validation_errors)
         .equals({
-          name: ["NAME ERROR"],
-          other: ["OTHER ERROR"],
+          "name" => ["NAME ERROR"],
+          "other" => ["OTHER ERROR"],
+          "empty" => [],
+          "none" => [nil],
         })
+      assert_that(subject.validation_errors[:name]).equals(["NAME ERROR"])
+      assert_that(subject.validation_errors["name"]).equals(["NAME ERROR"])
 
       # validation_error_messages
       assert_that(subject.validation_error_messages)
@@ -88,8 +96,14 @@ class MuchRails::ChangeActionResult
       # extract_validation_error
       assert_that(subject.extract_validation_error(:name))
         .equals(["NAME ERROR"])
-      assert_that(subject.extract_validation_error(:other))
+      assert_that(subject.extract_validation_error("other"))
         .equals(["OTHER ERROR"])
+      assert_that(subject.extract_validation_error(:empty))
+        .equals([])
+      assert_that(subject.extract_validation_error("none"))
+        .equals([])
+      assert_that(subject.extract_validation_error(:unknown))
+        .equals([])
       assert_that(subject.validation_errors).is_empty
 
       # any_unextracted_validation_errors?

data/test/unit/change_action_tests.rb

@@ -57,14 +57,7 @@ module MuchRails::ChangeAction
 
   class InitTests < ReceiverTests
     desc "when init"
-    subject{ receiver_class.new(params: {}) }
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     setup do
       Assert.stub(subject, :any_unextracted_change_result_validation_errors?) do
@@ -84,13 +77,7 @@ module MuchRails::ChangeAction
 
   class RecordErrorsWithResultExceptionTests < InitTests
     desc "with record errors and a result exception"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     setup do
       Assert.stub(subject, :any_unextracted_change_result_validation_errors?) do
@@ -118,13 +105,7 @@ module MuchRails::ChangeAction
 
   class RecordErrorsWithNoResultExceptionTests < InitTests
     desc "with record errors and no result exception"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     setup do
       Assert.stub(subject, :any_unextracted_change_result_validation_errors?) do
@@ -149,13 +130,7 @@ module MuchRails::ChangeAction
 
   class ChangeResultMethodTests < InitTests
     desc "#change_result method"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     should "memoize and return the expected Result" do
       result = subject.change_result
@@ -178,13 +153,7 @@ module MuchRails::ChangeAction
 
   class AnyUnextractedChangeResultValidationErrorsMethodTests < ReceiverTests
     desc "#any_unextracted_change_result_validation_errors? method"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     let(:receiver_class) do
       Class.new do
@@ -205,13 +174,7 @@ module MuchRails::ChangeAction
   class NoValidationErrorsTests <
           AnyUnextractedChangeResultValidationErrorsMethodTests
     desc "with no validation errors"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     let(:receiver_class) do
       Class.new do
@@ -232,13 +195,7 @@ module MuchRails::ChangeAction
   class ValidationErrorsTests <
           AnyUnextractedChangeResultValidationErrorsMethodTests
     desc "with validation errors"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     let(:receiver_class) do
       Class.new do

data/test/unit/config_tests.rb

@@ -10,59 +10,8 @@ module MuchRails::Config
 
     let(:unit_class){ MuchRails::Config }
 
-    should "include MuchRails::Mixin" do
-      assert_that(subject).includes(MuchRails::Mixin)
-    end
-  end
-
-  class ReceiverTests < UnitTests
-    desc "receiver"
-    subject{ receiver_class }
-
-    setup do
-      class receiver_class::Config
-        attr_accessor :value
-      end
-
-      class receiver_class::AnotherConfig
-        include MuchRails::Config
-
-        add_instance_config :sub, method_name: :sub
-
-        attr_accessor :another_value
-
-        class SubConfig
-          attr_accessor :sub_value
-        end
-      end
-    end
-
-    let(:receiver_class) do
-      Class.new do
-        include MuchRails::Config
-
-        add_config
-        add_config :another
-      end
-    end
-
-    should have_imeths :config, :another_config
-
-    should "know its attributes" do
-      assert_that(subject.config).is_instance_of(subject::Config)
-      subject.configure{ |config| config.value = "VALUE 1" }
-      assert_that(subject.config.value).equals("VALUE 1")
-
-      assert_that(subject.another_config).is_instance_of(subject::AnotherConfig)
-      subject.configure_another{ |config| config.another_value = "VALUE 2" }
-      assert_that(subject.another_config.another_value).equals("VALUE 2")
-
-      assert_that(subject.another_config.sub)
-        .is_instance_of(subject::AnotherConfig::SubConfig)
-      subject.another_config.configure_sub do |sub|
-        sub.sub_value = "VALUE 3"
-      end
-      assert_that(subject.another_config.sub.sub_value).equals("VALUE 3")
+    should "be MuchConfig" do
+      assert_that(unit_class).is(MuchConfig)
     end
   end
 end

data/test/unit/destroy_action_tests.rb

@@ -50,13 +50,7 @@ module MuchRails::DestroyAction
 
   class InitTests < ReceiverTests
     desc "when init"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     should have_imeths :destroy_result
 

data/test/unit/destroy_service_tests.rb

@@ -73,6 +73,7 @@ module MuchRails::DestroyService
         RuntimeError.new(Factory.string),
         ArgumentError.new(Factory.string),
         MuchRails::Records::DestructionInvalid.new(FakeRecord.new),
+        MuchRails::InvalidError.new,
       ]
     end
     let(:exception_classes){ exceptions.map(&:class) }
@@ -102,6 +103,8 @@ module MuchRails::DestroyService
         .is_an_instance_of(MuchRails::ServiceValidationErrors)
       assert_that(subject.service_validation_errors.exception_classes)
         .includes(MuchRails::Records::DestructionInvalid)
+      assert_that(subject.service_validation_errors.exception_classes)
+        .includes(MuchRails::InvalidError)
     end
   end
 
@@ -185,6 +188,22 @@ module MuchRails::DestroyService
     end
   end
 
+  class ValidationErrorsResultForInvalidErrorTests < ValidationErrorsTests
+    desc "when .result_for is passed an MuchRails::InvalidError"
+
+    let(:exception){ MuchRails::InvalidError.new }
+
+    should "return a failure result with the record and validation errors" do
+      result = subject.result_for(exception)
+
+      assert_that(result.failure?).is_true
+      assert_that(result.exception).equals(exception)
+      assert_that(result.validation_errors).equals(exception.errors)
+      assert_that(result.validation_error_messages)
+        .equals(exception.error_messages)
+    end
+  end
+
   class FailureResultTests < UnitTests
     desc "FailureResult"
     subject{ unit_class::FailureResult }

data/test/unit/invalid_error_tests.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require "assert"
+require "much-rails/invalid_error"
+
+class MuchRails::InvalidError
+  class UnitTests < Assert::Context
+    desc "MuchRails::InvalidError"
+    subject{ unit_class }
+
+    let(:unit_class){ MuchRails::InvalidError }
+
+    should "be configured as expected" do
+      assert_that(subject < StandardError).is_true
+    end
+  end
+
+  class InitSetupTests < UnitTests
+    desc "when init"
+    subject{ unit_class.new }
+
+    should have_readers :errors
+    should have_imeths :error_messages
+
+    should "know its attributes" do
+      assert_that(subject.backtrace).is_nil
+      assert_that(subject.errors).equals({})
+      assert_that(subject.message).equals(subject.errors.inspect)
+      assert_that(subject.error_messages).equals([])
+
+      backtrace = Array.new(Factory.integer(3)){ Factory.path }
+      errors =
+        {
+          field1: ["ERROR 1A", "ERROR 2B"],
+          field2: "ERROR 2A",
+        }
+      exception = unit_class.new(backtrace: backtrace, **errors)
+      assert_that(exception.backtrace).equals(backtrace)
+      assert_that(exception.errors).equals(errors)
+      assert_that(exception.message).equals(exception.errors.inspect)
+      assert_that(exception.error_messages)
+        .equals(["field1: ERROR 1A and ERROR 2B", "field2: ERROR 2A"])
+    end
+  end
+end

data/test/unit/save_action_tests.rb

@@ -50,13 +50,7 @@ module MuchRails::SaveAction
 
   class InitTests < ReceiverTests
     desc "when init"
-    subject do
-      receiver_class.new(
-        params: {},
-        current_session: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     should have_imeths :save_result
 

data/test/unit/save_service_tests.rb

@@ -73,6 +73,7 @@ module MuchRails::SaveService
         RuntimeError.new(Factory.string),
         ArgumentError.new(Factory.string),
         ActiveRecord::RecordInvalid.new(FakeRecord.new),
+        MuchRails::InvalidError.new,
       ]
     end
     let(:exception_classes){ exceptions.map(&:class) }
@@ -102,6 +103,8 @@ module MuchRails::SaveService
         .is_an_instance_of(MuchRails::ServiceValidationErrors)
       assert_that(subject.service_validation_errors.exception_classes)
         .includes(ActiveRecord::RecordInvalid)
+      assert_that(subject.service_validation_errors.exception_classes)
+        .includes(MuchRails::InvalidError)
     end
   end
 
@@ -197,6 +200,22 @@ module MuchRails::SaveService
     end
   end
 
+  class ValidationErrorsResultForInvalidErrorTests < ValidationErrorsTests
+    desc "when .result_for is passed an MuchRails::InvalidError"
+
+    let(:exception){ MuchRails::InvalidError.new }
+
+    should "return a failure result with the record and validation errors" do
+      result = subject.result_for(exception)
+
+      assert_that(result.failure?).is_true
+      assert_that(result.exception).equals(exception)
+      assert_that(result.validation_errors).equals(exception.errors)
+      assert_that(result.validation_error_messages)
+        .equals(exception.error_messages)
+    end
+  end
+
   class FailureResultTests < UnitTests
     desc "FailureResult"
     subject{ unit_class::FailureResult }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: much-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.4.3
 platform: ruby
 authors:
 - Kelly Redding
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-04-24 00:00:00.000000000 Z
+date: 2021-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: much-style-guide
@@ -17,28 +17,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: 0.6.6
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: 0.6.6
 - !ruby/object:Gem::Dependency
   name: assert
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.19.5
+        version: 2.19.6
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.19.5
+        version: 2.19.6
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -105,14 +105,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.15.2
+        version: 0.15.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.15.2
+        version: 0.15.3
 - !ruby/object:Gem::Dependency
   name: dassets-erubi
   requirement: !ruby/object:Gem::Requirement
@@ -155,6 +155,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.2.1
+- !ruby/object:Gem::Dependency
+  name: much-config
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.1.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.1.0
 - !ruby/object:Gem::Dependency
   name: much-decimal
   requirement: !ruby/object:Gem::Requirement
@@ -231,14 +245,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.10'
+        version: '3.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.10'
+        version: '3.11'
 description: Rails utilities.
 email:
 - kelly@kellyredding.com
@@ -277,6 +291,7 @@ files:
 - lib/much-rails/destroy_service.rb
 - lib/much-rails/has_slug.rb
 - lib/much-rails/input_value.rb
+- lib/much-rails/invalid_error.rb
 - lib/much-rails/json.rb
 - lib/much-rails/layout.rb
 - lib/much-rails/layout/helper.rb
@@ -304,6 +319,7 @@ files:
 - test/helper.rb
 - test/support/actions/show.rb
 - test/support/config/routes/test.rb
+- test/support/config/routes/test/subrouter.rb
 - test/support/factory.rb
 - test/support/fake_action_controller.rb
 - test/system/.keep
@@ -333,6 +349,7 @@ files:
 - test/unit/destroy_service_tests.rb
 - test/unit/has_slug_tests.rb
 - test/unit/input_value_tests.rb
+- test/unit/invalid_error_tests.rb
 - test/unit/json_tests.rb
 - test/unit/layout_tests.rb
 - test/unit/mixin_tests.rb
@@ -371,7 +388,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.4
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Rails utilities.
@@ -379,6 +396,7 @@ test_files:
 - test/helper.rb
 - test/support/actions/show.rb
 - test/support/config/routes/test.rb
+- test/support/config/routes/test/subrouter.rb
 - test/support/factory.rb
 - test/support/fake_action_controller.rb
 - test/system/.keep
@@ -408,6 +426,7 @@ test_files:
 - test/unit/destroy_service_tests.rb
 - test/unit/has_slug_tests.rb
 - test/unit/input_value_tests.rb
+- test/unit/invalid_error_tests.rb
 - test/unit/json_tests.rb
 - test/unit/layout_tests.rb
 - test/unit/mixin_tests.rb