actionfacade 0.1.0.1 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7c80521a86c6ac425e16ff4fe0305d8885d2da3f4cd6c278cf116dc83a9cf059
-  data.tar.gz: 53ab2594a91e9399cfa8c0081d88213865844f822b2b1f572c63d6219f230da5
+  metadata.gz: 36b54b632384d17a03da67a3c06b17af8f34d3860f4982d34a2085a0435d2050
+  data.tar.gz: d5b6cc6da5f073f084b42718f5cf00dbb5c33842aec0ba86eec5107ead413957
 SHA512:
-  metadata.gz: 1a0eca5888e77d3c6092fdc9d5382a87ab8c75bb75c97956eea1aa4956492110179cda1e508b87c788d0c68472860046a56027439952da8ff86647ccb30a8e21
-  data.tar.gz: 706f73248be326ca32e2d2cd8b5fa243ef17bf653ca3b799d56d16c1157966ddc800c2505f26eee694c842c5996edad599d21f0ad8dc5c982d031efc498f4425
+  metadata.gz: 9cc9020127e0d5684dc29330a2a2d060b5466c56b2a14d59b2c26207b188ef20cf2a5010963b47c6b34b57f627afdd5107addb24c5e04badf3ff7bfb18141112
+  data.tar.gz: ec3730ca6f2a004aa8180e1d209ee9e5c5f400edb5e167c167afbc74b842d4bb162477f78e0b0202123c3cdc2552030a00ab9bfa0708eaac7730f6ed1c4142d7

data/actionfacade.gemspec

@@ -24,6 +24,7 @@ Gem::Specification.new do |s|
   s.test_files  = s.files.grep(/^test/)
 
   s.add_dependency "bundler", ">= 1.3"
+  s.add_dependency "activesupport", ">= 5.2"
   s.add_development_dependency "rake", ">= 12.3.3"
   s.add_development_dependency "test-unit", ">= 3.3"
 end

data/lib/action_facade/retrieval.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "active_support/core_ext/string/inflections"
+
 module ActionFacade
   # == Action \Facade \Retrieval
   #
@@ -11,13 +13,80 @@ module ActionFacade
   module Retrieval
     # Retrieve data from `facade`
     #
+    # If `facade_or_other` is a facade, retrieve data from it.
+    # If it is a string or a symbol, it will be merged to `variable_names` and
+    # retrieve_from method called with controller params.
+    # If it is a hash, retrieve_from method will be called with the hash as a payload.
+    #
     # `variable_names` are symbols for the method names in the facade and
     # they will be set as instance variables in the class that
     # includes the module.
-    def retrieve(facade, *variable_names)
-      variable_names.each do |name|
+    def retrieve(facade_or_other, *variable_names)
+      facade = nil
+      names = variable_names.dup
+      case facade_or_other
+      when ActionFacade::Base
+        facade = facade_or_other
+      when String, Symbol
+        if defined?(params)
+          names.unshift(facade_or_other)
+          return retrieve_from(params, *names)
+        else
+          raise ArgumentError.new("Can't call the method with #{facade_or_other.class} if params is undefined.")
+        end
+      when Hash
+        return retrieve_from(facade_or_other, *variable_names)
+      else
+        raise ArgumentError.new("First parameter must be instance of ActionFacade::Base, String, Symbol or Hash")
+      end
+      names.each do |name|
         instance_variable_set("@#{name}", facade.send(name.to_sym))
       end
     end
+
+    # Retrieve data from given payload
+    #
+    # If the class which includes the module is Rails controller,
+    # guessed facade name will be "Controller" is replaced by the action name + "Facade".
+    # If the class is not Rails controller, the name will be suffixed by "Facade".
+    #
+    # `payload` is the initialization parameter for the facade.
+    #
+    # `variable_names` are symbols for the method names in the facade and
+    # they will be set as instance variables in the class that
+    # includes the module.
+    def retrieve_from(payload, *variable_names)
+      facade = guess_facade
+      if facade.nil?
+        raise FacadeNotFoundError.new("Could not find Facade class #{guess_facade_name}.")
+      end
+      retrieve(facade.new(payload), *variable_names)
+    end
+
+    class FacadeNotFoundError < StandardError; end
+
+    private
+
+    def guess_facade
+      facade_name = guess_facade_name
+      begin
+        facade_name.constantize
+      rescue NameError
+        nil
+      end
+    end
+
+    def guess_facade_name
+      klass_name = self.class.name
+      if klass_name.end_with?("Controller")
+        if defined?(params) && params[:action]
+          klass_name.delete_suffix("Controller") + "::#{params[:action].camelize}Facade"
+        else
+          klass_name.delete_suffix("Controller") + "Facade"
+        end
+      else
+        klass_name + "Facade"
+      end
+    end
   end
 end

data/test/base_test.rb

@@ -1,5 +1,4 @@
-require "test/unit"
-require_relative "../lib/actionfacade"
+require_relative "./test_helper"
 
 class BaseTest < Test::Unit::TestCase
   test ".new does not raise error" do

data/test/inheritance_test.rb

@@ -1,5 +1,4 @@
-require "test/unit"
-require_relative "../lib/actionfacade"
+require_relative "./test_helper"
 
 class UserFacade < ActionFacade::Base
   USER_DATA = [{ id: 1, name: "john" }, { id: 2, name: "taro" }]

data/test/rails_app/retrieval_test.rb

@@ -0,0 +1,77 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+require_relative "./test_app"
+
+module RailsApp
+  USER_DATA = [{ id: 1, name: "john" }, { id: 2, name: "bob" }]
+end
+
+module Home; end
+
+class Home::IndexFacade < ActionFacade::Base
+  def all_users
+    RailsApp::USER_DATA
+  end
+end
+
+class HomeController < ActionController::Base
+  include Rails.application.routes.url_helpers
+  include ActionFacade::Retrieval
+
+  def index
+    case params[:retrieve]
+    when "facade"
+      facade = Home::IndexFacade.new
+      retrieve(facade, :all_users)
+    when "string"
+      retrieve("all_users")
+    when "symbol"
+      retrieve(:all_users)
+    when "hash"
+      retrieve({}, :all_users)
+    else
+      retrieve_from({}, :all_users)
+    end
+    render plain: @all_users
+  end
+end
+
+require "rack/test"
+
+module RailsApp; end
+
+class RailsApp::RetrievalTest < Test::Unit::TestCase
+  include Rack::Test::Methods
+
+  test "@all_users is set after retrieve_from({}, :all_users)" do
+    get "/"
+    assert_equal(last_response.body, RailsApp::USER_DATA.to_s)
+  end
+
+  test "@all_users is set after retrieve(facade, :all_users)" do
+    get "/", params: { retrieve: "facade" }
+    assert_equal(last_response.body, RailsApp::USER_DATA.to_s)
+  end
+
+  test "@all_users is set after retrieve(:all_users)" do
+    get "/", params: { retrieve: "symbol" }
+    assert_equal(last_response.body, RailsApp::USER_DATA.to_s)
+  end
+
+  test "@all_users is set after retrieve(\"all_users\")" do
+    get "/", params: { retrieve: "string" }
+    assert_equal(last_response.body, RailsApp::USER_DATA.to_s)
+  end
+
+  test "@all_users is set after retrieve({}, :all_users)" do
+    get "/", params: { retrieve: "hash" }
+    assert_equal(last_response.body, RailsApp::USER_DATA.to_s)
+  end
+
+  private
+
+  def app
+    Rails.application
+  end
+end

data/test/rails_app/test_app.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require "action_controller/railtie"
+
+class TestApp < Rails::Application
+  config.root = __dir__
+  config.hosts << "example.org"
+  secrets.secret_key_base = "secret_key_base"
+
+  config.logger = Logger.new($stdout)
+  Rails.logger  = config.logger
+
+  routes.draw do
+    get "/" => "home#index"
+  end
+end

data/test/retrieval_test.rb

@@ -1,15 +1,21 @@
-require "test/unit"
-require_relative "../lib/action_facade/base.rb"
-require_relative "../lib/action_facade/retrieval.rb"
+require_relative "./test_helper"
 
-USER_DATA = [{ id: 1, name: "john" }]
+USER_DATA = [{ id: 1, name: "john" }, { id: 2, name: "bob" }]
 
-class UserFacade < ActionFacade::Base
+class UsersFacade < ActionFacade::Base
   def john
     USER_DATA.find { |user| user[:name] == "john" }
   end
 end
 
+module Admin
+  class ShowFacade < ActionFacade::Base
+    def bob
+      USER_DATA.find { |user| user[:name] == "bob" }
+    end
+  end
+end
+
 class UsersController
   include ActionFacade::Retrieval
 
@@ -17,9 +23,26 @@ class UsersController
 
   def show
     payload = {}
-    facade = UserFacade.new(payload)
+    facade = UsersFacade.new(payload)
     retrieve(facade, :john)
   end
+
+  def show_symbol
+    retrieve(:john)
+  end
+
+  def show_string
+    retrieve("john")
+  end
+
+  def show_hash
+    retrieve({}, :john)
+  end
+
+  def show_from
+    payload = {}
+    retrieve_from(payload, :john)
+  end
 end
 
 class RetrievalTest < Test::Unit::TestCase
@@ -33,4 +56,36 @@ class RetrievalTest < Test::Unit::TestCase
     controller.show
     assert_equal(controller.john, { id: 1, name: "john" })
   end
+
+  test "raised exception when retrieve(:john)" do
+    controller = UsersController.new
+    assert_nil(controller.john)
+    error = assert_raises ArgumentError do
+      controller.show_symbol
+    end
+    assert_equal(error.message, "Can't call the method with Symbol if params is undefined.")
+  end
+
+  test "raised exception when retrieve(\"john\")" do
+    controller = UsersController.new
+    assert_nil(controller.john)
+    error = assert_raises ArgumentError do
+      controller.show_string
+    end
+    assert_equal(error.message, "Can't call the method with String if params is undefined.")
+  end
+
+  test "@john is set after retrieve({}, :john)" do
+    controller = UsersController.new
+    assert_nil(controller.john)
+    controller.show_hash
+    assert_equal(controller.john, { id: 1, name: "john" })
+  end
+
+  test "@john is set after retrieve_from(payload, :john)" do
+    controller = UsersController.new
+    assert_nil(controller.john)
+    controller.show_from
+    assert_equal(controller.john, { id: 1, name: "john" })
+  end
 end

data/test/test_helper.rb

@@ -0,0 +1,2 @@
+require "test/unit"
+require_relative "../lib/actionfacade"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: actionfacade
 version: !ruby/object:Gem::Version
-  version: 0.1.0.1
+  version: 0.2.4
 platform: ruby
 authors:
 - Ryo Hashimoto
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-02 00:00:00.000000000 Z
+date: 2021-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '5.2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -66,7 +80,10 @@ files:
 - lib/actionfacade.rb
 - test/base_test.rb
 - test/inheritance_test.rb
+- test/rails_app/retrieval_test.rb
+- test/rails_app/test_app.rb
 - test/retrieval_test.rb
+- test/test_helper.rb
 homepage: https://github.com/ryohashimoto/lightrails
 licenses:
 - MIT
@@ -86,7 +103,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.8.11
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Action Facade provides a simple interface for data required by view / controller
@@ -94,4 +111,7 @@ summary: Action Facade provides a simple interface for data required by view / c
 test_files:
 - test/base_test.rb
 - test/inheritance_test.rb
+- test/rails_app/retrieval_test.rb
+- test/rails_app/test_app.rb
 - test/retrieval_test.rb
+- test/test_helper.rb