much-rails 0.2.4 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b0abf64ca287edfd1a6546db5766e878424a9165a661ef991ca7fda1dba6f78
-  data.tar.gz: 3aa006e90bb01924786952077cb5a72c595bb1c0fa28e4f32f3701063fc2ac06
+  metadata.gz: ba7d46f07c2592e152fe836a0cff5a7bf8061e9f531aac1c6a748f1365cc383b
+  data.tar.gz: 19e41c6e470a0021b6b78ea9c7eff7e6e530758d0522a591b4f8623b610f9fa2
 SHA512:
-  metadata.gz: 92884c3070b8cbaf78ff384b7ea4bc12476ee175a56e73b4ea58431f84c3539c1d27d97e180b9ddcfbc16ba908127a6783c72f3716890ac5e879773dde7ff4e1
-  data.tar.gz: '0755183185e5f64a4c0e289e31d6bf0545438c96631847f306f83c0aa588308d215583ab2249e856c35592162438dc8cc4315ac959bb478a14baa9aa5c08584f'
+  metadata.gz: 37bdf6aa9a624f270cd53e5ba22f3e400f331b9b91c29aed2c6169627350f9c49e2f929ff6b9bc83665c789953559f67f546c11fd4cc9660814b46f61bfb2bb9
+  data.tar.gz: eb78925cf707aef2f6484665963da6dee38f495cad04de680f7ea349292eae41b77bdd36f014b05a2e5fcda48e8c8d03b23485f60786020100794f4d1cb3c15f

data/lib/much-rails/action.rb

@@ -31,7 +31,7 @@ module MuchRails::Action
 
     add_config :much_rails_action
 
-    attr_reader :params, :current_user, :request, :errors
+    attr_reader :params, :request, :errors
   end
 
   mixin_class_methods do
@@ -107,17 +107,13 @@ module MuchRails::Action
 
     def default_action_template_name
       @default_action_template_name ||=
-        to_s
-          .remove(/\A#{MuchRails.config.action.namespace}/)
-          .tableize
-          .singularize
+        to_s.remove(/\A#{MuchRails.config.action.namespace}/).underscore
     end
   end
 
   mixin_instance_methods do
-    def initialize(params: nil, current_user: nil, request: nil)
+    def initialize(params: nil, request: nil)
       @params = params.to_h.with_indifferent_access
-      @current_user = current_user
       @request = request
       @errors = Hash.new{ |hash, key| hash[key] = [] }
     end
@@ -153,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
@@ -257,6 +262,10 @@ module MuchRails::Action
           template: template || default_action_template_name,
         }.merge(**kargs)
 
+      if view_model&.respond_to?(:much_rails_action=)
+        view_model.much_rails_action = self
+      end
+
       @much_rails_action_result =
         MuchRails::Action::RenderResult.new(view_model, *args, **result_kargs)
       halt

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

@@ -149,7 +149,7 @@ class MuchRails::Action::BaseRouter
   #       get    "/new",    "Root::New"
   #       post   "/",       "Root::Create"
   #       get    "/edit",   "Root::Edit"
-  #       put    "/",       "Root::Update"
+  #       put    "/",       "Root::Upsert"
   #       patch  "/",       "Root::Update"
   #       get    "/remove", "Root::Remove"
   #       delete "/",       "Root::Destroy"
@@ -301,6 +301,14 @@ class MuchRails::Action::BaseRouter
       def constraints_lambda
         request_type.constraints_lambda
       end
+
+      def class_constant
+        @class_constant ||= class_name.constantize
+      end
+
+      def format
+        class_constant.format
+      end
     end
 
   class URLSet
@@ -460,6 +468,16 @@ class MuchRails::Action::BaseRouter
       !@default_action_class_name.nil?
     end
 
+    def default_action_class_constant
+      return unless has_default_action_class_name?
+
+      @default_action_class_constant ||= default_action_class_name.constantize
+    end
+
+    def default_action_format
+      default_action_class_constant&.format
+    end
+
     def ==(other)
       return super unless other.is_a?(self.class)
 

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_user: current_user,
               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
@@ -38,7 +49,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 +60,12 @@ 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
 
     definitions.each do |definition|
       definition.request_type_actions.each do |request_type_action|
@@ -58,10 +73,12 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
           definition.http_method,
           definition.path,
           to: draw_route_to,
-          as: definition.name,
+          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,
         )
@@ -73,10 +90,12 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
         definition.http_method,
         definition.path,
         to: draw_route_to,
-        as: definition.name,
+        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,
           }),
       )
     end
@@ -93,14 +112,14 @@ class MuchRails::Action::Router < MuchRails::Action::BaseRouter
     def path_for(**kargs)
       MuchRails::RailsRoutes.instance.public_send(
         "#{name}_path",
-        **kargs.symbolize_keys,
+        **kargs.symbolize_keys.except(:format),
       )
     end
 
     def url_for(**kargs)
       MuchRails::RailsRoutes.instance.public_send(
         "#{name}_url",
-        **kargs.symbolize_keys,
+        **kargs.symbolize_keys.except(:format),
       )
     end
   end

data/lib/much-rails/input_value.rb

@@ -7,7 +7,7 @@ module MuchRails::InputValue
   def self.strip(value)
     return if value.blank?
 
-    value.to_s.strip
+    value.to_s.strip.presence
   end
 
   def self.strip_all(values)

data/lib/much-rails/version.rb

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

data/much-rails.gemspec

@@ -37,6 +37,6 @@ Gem::Specification.new do |gem|
   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.1"])
+  gem.add_dependency("much-slug",      ["~> 0.1.2"])
   gem.add_dependency("oj",             ["~> 3.10"])
 end

data/test/support/actions/show.rb

@@ -7,5 +7,7 @@ module Actions; end
 module Actions::Show
   include MuchRails::Action
 
+  format :html
+
   params_root :nested
 end

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/base_router_tests.rb

@@ -296,20 +296,30 @@ class MuchRails::Action::BaseRouter
     desc "when init"
     subject{ request_type_action_class.new(request_type1, action_class_name1) }
 
+    setup do
+      Assert.stub(action_class_name1, :constantize){ action_class1 }
+    end
+
     let(:name1){ Factory.symbol }
     let(:constraints_lambda1){ ->(request){} }
     let(:request_type1) do
       unit_class::RequestType.new(name1, constraints_lambda1)
     end
     let(:action_class_name1){ Factory.string }
+    let(:action_class1) do
+      Struct.new(:format).new([:html, :any].sample)
+    end
 
     should have_imeths :request_type, :class_name, :constraints_lambda
+    should have_imeths :class_constant, :format
 
     should "know its attributes" do
       assert_that(subject.request_type).equals(request_type1)
       assert_that(subject.class_name).equals(action_class_name1)
       assert_that(subject.constraints_lambda)
         .equals(request_type1.constraints_lambda)
+      assert_that(subject.class_constant).equals(action_class1)
+      assert_that(subject.format).equals(subject.class_constant.format)
     end
   end
 
@@ -506,6 +516,15 @@ class MuchRails::Action::BaseRouter
       )
     end
 
+    setup do
+      Assert.stub(default_action_class_name1, :constantize) do
+        default_action_class1
+      end
+    end
+
+    let(:default_action_class1) do
+      Struct.new(:format).new([:html, :any].sample)
+    end
     let(:default_params1) do
       { Factory.string => Factory.string }
     end
@@ -514,6 +533,7 @@ class MuchRails::Action::BaseRouter
     should have_readers :default_action_class_name, :request_type_actions
     should have_readers :called_from
     should have_imeths :path, :name, :has_default_action_class_name?
+    should have_imeths :default_action_class_constant, :default_action_format
 
     should "know its attributes" do
       assert_that(subject.http_method).equals(http_method1)
@@ -525,6 +545,10 @@ class MuchRails::Action::BaseRouter
       assert_that(subject.called_from).equals(caller1)
       assert_that(subject.path).equals(url_path1)
       assert_that(subject.name).equals(url_name1)
+      assert_that(subject.default_action_class_constant)
+        .equals(default_action_class1)
+      assert_that(subject.default_action_format)
+        .equals(subject.default_action_class_constant.format)
     end
   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,15 +29,32 @@ 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
     desc "when init"
     subject{ receiver_class.new(params1) }
 
+    setup do
+      Assert.stub(::Actions::Show, :format){ nil }
+    end
+
     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: {
@@ -49,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
@@ -65,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(
@@ -75,10 +92,9 @@ module MuchRails::Action::Controller
 
       assert_that(subject.much_rails_action_class).equals(Actions::Show)
 
-      Actions::Show.format(:html)
+      Assert.stub(::Actions::Show, :format){ :html }
       receiver = receiver_class.new(params1)
       assert_that(receiver.much_rails_action_class_format).equals(:html)
-      Actions::Show.format(nil)
     end
   end
 
@@ -88,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
@@ -64,7 +64,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,12 +99,15 @@ 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,
+        }
 
       assert_that(application_routes.get_calls.size).equals(3)
       assert_that(application_routes.get_calls[0].pargs).equals([url_path])
@@ -113,14 +116,17 @@ class MuchRails::Action::Router
           to: expected_draw_route_to,
           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,
         )
       assert_that(application_routes.get_calls[1].pargs).equals([url_path])
       assert_that(application_routes.get_calls[1].kargs)
         .equals(
           to: expected_draw_route_to,
-          as: url_name,
+          as: nil,
           defaults: expected_default_defaults,
         )
       assert_that(application_routes.get_calls[2].pargs).equals([url2_path])
@@ -135,7 +141,7 @@ class MuchRails::Action::Router
       assert_that(application_routes.post_calls.last.kargs)
         .equals(
           to: expected_draw_route_to,
-          as: url_name,
+          as: nil,
           defaults: expected_default_defaults,
         )
 
@@ -153,7 +159,7 @@ class MuchRails::Action::Router
       assert_that(application_routes.patch_calls.last.kargs)
         .equals(
           to: expected_draw_route_to,
-          as: url_name,
+          as: nil,
           defaults: expected_default_defaults,
         )
 
@@ -162,7 +168,7 @@ class MuchRails::Action::Router
       assert_that(application_routes.delete_calls.last.kargs)
         .equals(
           to: expected_draw_route_to,
-          as: url_name,
+          as: nil,
           defaults: expected_default_defaults,
         )
     end
@@ -206,12 +212,12 @@ class MuchRails::Action::Router
     should have_imeths :path_for, :url_for
 
     should "know its attributes" do
-      path_string = subject.path_for(test: "args")
+      path_string = subject.path_for(test: "args", format: "html")
       assert_that(path_string).equals("TEST PATH OR URL STRING")
       assert_that(@rails_routes_method_missing_call.args)
         .equals(["#{url_name1}_path".to_sym, { test: "args" }])
 
-      url_string = subject.url_for(test: "args")
+      url_string = subject.url_for(test: "args", format: "xml")
       assert_that(url_string).equals("TEST PATH OR URL STRING")
       assert_that(@rails_routes_method_missing_call.args)
         .equals(["#{url_name1}_url".to_sym, { test: "args" }])

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

@@ -167,7 +167,7 @@ module MuchRails::Action
       assert_that(subject.default_action_template_name)
         .equals(
           "some/namespace/for/"\
-          "#{MuchRails.config.action.namespace.tableize.singularize}"\
+          "#{MuchRails.config.action.namespace.underscore}"\
           "thing/show",
         )
     end
@@ -176,11 +176,7 @@ module MuchRails::Action
   class InitTests < ReceiverTests
     desc "when init"
     subject do
-      receiver_class.new(
-        params: params1,
-        current_user: current_user1,
-        request: request1,
-      )
+      receiver_class.new(params: params1, request: request1)
     end
 
     let(:receiver_class) do
@@ -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_user1){ "CURRENT USER 1" }
-    let(:request1){ "REQUEST 1" }
+    let(:request1){ FakeRequest.new }
 
-    should have_readers :params, :current_user, :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_user).equals(current_user1)
       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
@@ -318,6 +343,7 @@ module MuchRails::Action
           layout: false,
         )
 
+      view_model = Object.new
       receiver_class.on_call do
         render(view_model, "some/view/template", layout: false)
       end
@@ -330,18 +356,29 @@ module MuchRails::Action
           layout: false,
         )
 
+      view_model =
+        Class
+          .new{
+            def much_rails_action
+              "NOT A MUCH RAILS ACTION"
+            end
+          }
+          .new
       receiver_class.on_call do
         render(view_model, "some/view/template", template: "other/template")
       end
       action = receiver_class.new(params: params1)
       result = action.call
       assert_that(result.render_view_model).is(view_model)
+      assert_that(result.render_view_model.much_rails_action).is_not(action)
       assert_that(result.render_kargs).equals(template: "other/template")
 
+      view_model = Struct.new(:much_rails_action).new(nil)
       receiver_class.on_call{ render(view_model, template: "other/template") }
       action = receiver_class.new(params: params1)
       result = action.call
       assert_that(result.render_view_model).is(view_model)
+      assert_that(result.render_view_model.much_rails_action).is(action)
       assert_that(result.render_kargs).equals(template: "other/template")
     end
 
@@ -397,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/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_user: 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_user: 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_user: 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_user: 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_user: 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_user: 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_user: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     let(:receiver_class) do
       Class.new do

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_user: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     should have_imeths :destroy_result
 

data/test/unit/input_value_tests.rb

@@ -9,12 +9,21 @@ module MuchRails::InputValue
     subject{ unit_class }
 
     let(:unit_class){ MuchRails::InputValue }
+    let(:non_blank_but_empty_to_s_object) do
+      Class
+        .new{
+          def to_s
+            ["", " "].sample
+          end
+        }
+        .new
+    end
 
     should have_imeths :strip, :strip_all
 
     should "know its attributes" do
       # strip
-      input_value = [nil, "", " "].sample
+      input_value = [nil, "", " ", non_blank_but_empty_to_s_object].sample
       assert_that(subject.strip(input_value)).is_nil
 
       input_value = [" VALUE  ", "\r VALUE\n"].sample

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_user: nil,
-        request: nil,
-      )
-    end
+    subject{ receiver_class.new(params: {}, request: nil) }
 
     should have_imeths :save_result
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: much-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.3.0
 platform: ruby
 authors:
 - Kelly Redding
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-09 00:00:00.000000000 Z
+date: 2021-05-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: much-style-guide
@@ -217,14 +217,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.1
+        version: 0.1.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.1
+        version: 0.1.2
 - !ruby/object:Gem::Dependency
   name: oj
   requirement: !ruby/object:Gem::Requirement