action_args 2.3.0 → 2.6.0

data/test/controllers/action_args_controller_test.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-
-class BooksControllerTest < ActionController::TestCase
-  sub_test_case 'GET index (having an optional parameter)' do
-    setup do
-      @books = []
-      Book.delete_all
-      100.times {|i| @books << Book.create!(title: 'book'+i.to_s) }
-    end
-
-    test 'without page parameter' do
-      get :index
-      assert 200, response.code
-      assert_equal @books[0..9], assigns(:books)
-    end
-
-    test 'with page parameter' do
-      get :index, params: {page: 3}
-      assert 200, response.code
-      assert_equal @books[20..29], assigns(:books)
-    end
-
-    test 'first param is nil and second is not nil' do
-      rhg = Book.create! title: 'RHG'
-      Book.create! title: 'AWDwR'
-      get :index, params: {q: 'RH'}
-      assert_equal [rhg], assigns(:books)
-    end
-  end
-
-  test 'GET show' do
-    rhg = Book.create! title: 'RHG'
-    get :show, params: {id: rhg.id}
-    assert_equal rhg, assigns(:book)
-  end
-
-  test 'POST create' do
-    Book.create! title: 'RHG'
-    books_count_was = Book.count
-    post :create, params: {book: {title: 'AWDwR', price: 24}}
-    assert_equal 1, Book.count - books_count_was
-  end
-end

data/test/controllers/hooks_test.rb

@@ -1,30 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-
-class BooksControllerTest < ActionController::TestCase
-  setup do
-    Book.delete_all
-    @book = Book.create! title: 'Head First ActionArgs'
-    get :show, params: {id: @book.id}
-  end
-
-  sub_test_case 'before_action' do
-    test 'via Symbol' do
-      assert_equal @book, assigns(:book)
-    end
-
-    if Rails.version < '5.1'
-      test 'via String' do
-        assert assigns(:string_filter_executed)
-      end
-    end
-
-    test 'via Proc' do
-      assert assigns(:proc_filter_executed)
-    end
-  end
-
-  test 'around_action' do
-    assert_not_nil assigns(:elapsed_time)
-  end
-end

data/test/controllers/kwargs_controller_test.rb

@@ -1,31 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-
-class KwBooksControllerTest < ActionController::TestCase
-  sub_test_case 'GET index (having an optional parameter)' do
-    test 'without giving any kw parameter (not even giving :required one)' do
-      assert_raises(ActionController::BadRequest) { get :index }
-    end
-
-    test 'without giving any optional kw parameter' do
-      get :index, params: {author_name: 'nari'}
-      assert 200, response.code
-    end
-
-    test 'with kw parameter defaults to non-nil value' do
-      get :index, params: {author_name: 'nari', page: 3}
-      body = eval response.body
-      assert_equal 'nari', body[:author_name]
-      assert_equal '3', body[:page]
-      assert_nil body[:q]
-    end
-
-    test 'with kw parameter defaults to nil' do
-      get :index, params: {author_name: 'nari', q: 'Rails'}
-      body = eval response.body
-      assert_equal 'nari', body[:author_name]
-      assert_equal '1', body[:page]
-      assert_equal 'Rails', body[:q]
-    end
-  end
-end

data/test/controllers/kwargs_keyreq_controller_test.rb

@@ -1,31 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-
-class KwKeyreqBooksControllerTest < ActionController::TestCase
-  sub_test_case 'GET index (having an optional parameter)' do
-    test 'without giving any kw parameter (not even giving :required one)' do
-      assert_raises(ActionController::BadRequest) { get :index }
-    end
-
-    test 'without giving any kw parameter' do
-      get :index, params: {author_name: 'nari'}
-      assert 200, response.code
-    end
-
-    test 'with kw parameter defaults to non-nil value' do
-      get :index, params: {author_name: 'nari', page: 3}
-      body = eval response.body
-      assert_equal 'nari', body[:author_name]
-      assert_equal '3', body[:page]
-      assert_nil body[:q]
-    end
-
-    test 'with kw parameter defaults to nil' do
-      get :index, params: {author_name: 'nari', q: 'Rails'}
-      body = eval response.body
-      assert_equal 'nari', body[:author_name]
-      assert_equal '1', body[:page]
-      assert_equal 'Rails', body[:q]
-    end
-  end
-end if RUBY_VERSION >= '2.1'

data/test/controllers/ordinal_controller_test.rb

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-
-class AuthorsControllerTest < ActionController::TestCase
-  test 'GET show' do
-    matz = Author.create! name: 'Matz'
-    get :show, params: {id: matz.id}
-    assert_equal matz, assigns(:author)
-  end
-end

data/test/controllers/strong_parameters_test.rb

@@ -1,58 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-
-class StoresControllerTest < ActionController::TestCase
-  test 'GET show' do
-    tatsu_zine = Store.create! name: 'Tatsu-zine'
-    get :show, params: {id: tatsu_zine.id}
-
-    assert_equal tatsu_zine, assigns(:store)
-  end
-
-  sub_test_case 'GET new' do
-    test 'without store parameter' do
-      get :new
-      assert 200, response.code
-      assert_equal 'PragProg', assigns(:store).name
-    end
-    test 'with store parameter' do
-      get :new, params: {store: {name: 'Tatsu-zine'}}
-      assert 200, response.code
-      assert_equal 'Tatsu-zine', assigns(:store).name
-    end
-  end
-
-  test 'POST create' do
-    store_count_was = Store.count
-    post :create, params: {store: {name: 'Tatsu-zine', url: 'http://tatsu-zine.com'}}
-
-    assert_equal 1, Store.count - store_count_was
-  end
-end
-
-class MoviesControllerTest < ActionController::TestCase
-  test 'POST create' do
-    movie_count_was = Movie.count
-    post :create, params: {movie: {title: 'Dr. No', actors_attributes: [{name: 'Bernard Lee'}]}}
-
-    assert_equal 1, Movie.count - movie_count_was
-  end
-end
-
-# this controller doesn't permit price of new book do
-class Admin::BooksControllerTest < ActionController::TestCase
-  test 'POST create' do
-    post :create, params: {book: {title: 'naruhoUnix', price: 30}}
-
-    assert_nil Book.last.price
-  end
-end
-
-class Admin::AccountsControllerTest < ActionController::TestCase
-  test 'POST create' do
-    admin_account_count_was = Admin::Account.count
-    post :create, params: {admin_account: {name: 'amatsuda'}}
-
-    assert_equal 1, Admin::Account.count - admin_account_count_was
-  end
-end

data/test/fake_app.rb

@@ -1,184 +0,0 @@
-# coding: utf-8
-# frozen_string_literal: true
-
-# config
-ActiveRecord::Base.establish_connection(:adapter => 'sqlite3', :database => ':memory:')
-
-module ActionArgsTestApp
-  class Application < Rails::Application
-    config.secret_key_base = config.secret_token = [*'A'..'z'].join
-    config.session_store :cookie_store, :key => '_myapp_session'
-    config.active_support.deprecation = :log
-    config.eager_load = false
-  end
-end
-ActionArgsTestApp::Application.initialize!
-
-# routes
-ActionArgsTestApp::Application.routes.draw do
-  resources :authors
-  resources :books
-  resources :kw_books  # 2.0+ only
-  resources :kw_keyreq_books  # 2.1+ only
-  resources :stores
-  resources :movies
-
-  namespace :admin do
-    resources :accounts
-    resources :books
-  end
-end
-
-# models
-class Author < ActiveRecord::Base
-end
-class Book < ActiveRecord::Base
-end
-class Store < ActiveRecord::Base
-end
-class Movie < ActiveRecord::Base
-  has_many :actors
-  accepts_nested_attributes_for :actors
-end
-class Actor < ActiveRecord::Base
-  belongs_to :movie
-end
-module Admin
-  def self.table_name_prefix() 'admin_' end
-  class Account < ActiveRecord::Base
-  end
-end
-
-# mailers
-require "action_mailer/railtie"
-class UserMailer < ActionMailer::Base
-  def send_email_without_args
-    mail(
-      to:      'to@example.com',
-      from:    'from@example.com',
-      subject: 'Action Args!!!',
-      body:    'test'
-    )
-  end
-end
-
-# helpers
-module ApplicationHelper; end
-
-# controllers
-class ApplicationController < ActionController::Base
-end
-class AuthorsController < ApplicationController
-  def show
-    @author = Author.find params[:id]
-    render plain: @author.name
-  end
-end
-class BooksController < ApplicationController
-  before_action :set_book, only: :show
-  before_action -> { @proc_filter_executed = true }, only: :show
-  if Rails.version < '5.1'
-    before_action '@string_filter_executed = true', only: :show
-  end
-  around_action :benchmark_action
-  before_action :omg
-  skip_before_action :omg
-
-  # optional parameter
-  def index(page = 1, q = nil, limit = 10)
-    @books = Book.limit(limit.to_i).offset(([page.to_i - 1, 0].max) * 10)
-    @books = @books.where('title like ?', "%#{q}%") unless q.blank?
-    render plain: 'index', books: @books
-  end
-
-  def show(id)
-    render plain: @book.title
-  end
-
-  def create(book)
-    book = book.permit :title, :price
-    @book = Book.create! book
-    render plain: @book.title
-  end
-
-  private
-    def set_book(id)
-      @book = Book.find(id)
-    end
-
-    def benchmark_action
-      start  = Time.now
-      yield
-      @elapsed_time = Time.now - start
-    end
-
-    def omg
-      raise '💣'
-    end
-end
-class MoviesController < ApplicationController
-  permits :title, actors_attributes: [:name]
-
-  def create(movie)
-    @movie = Movie.create! movie
-    render plain: @movie.title
-  end
-end
-class StoresController < ApplicationController
-  permits :name, :url
-
-  def show(id)
-    @store = Store.find(id)
-    render plain: @store.name
-  end
-
-  def new(store = {name: 'PragProg'})
-    @store = Store.new store
-    render plain: @store.name
-  end
-
-  def create(store)
-    @store = Store.create! store
-    render plain: @store.name
-  end
-end
-module Admin
-  class AccountsController < ::ApplicationController
-    permits :name, model_name: 'Admin::Account'
-
-    def create(admin_account)
-      @admin_account = Admin::Account.create! admin_account
-      render plain: @admin_account.name
-    end
-  end
-
-  class BooksController < ::ApplicationController
-    permits :title
-
-    def create(book)
-      @book = Book.create! book
-      render plain: @book.title
-    end
-  end
-end
-
-require_relative 'kwargs_controllers'
-require_relative 'kwargs_keyreq_controllers' if RUBY_VERSION >= '2.1'
-
-# migrations
-class CreateAllTables < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
-  def self.up
-    create_table(:authors) {|t| t.string :name}
-    create_table(:books) {|t| t.string :title; t.integer :price}
-    create_table(:stores) {|t| t.string :name; t.string :url}
-    create_table(:admin_accounts) {|t| t.string :name}
-    create_table(:movies) {|t| t.string :title}
-    create_table(:actors) {|t| t.string :name; t.references :movie}
-  end
-end
-
-if ActiveRecord::Base.connection.respond_to? :data_source_exists?
-  CreateAllTables.up unless ActiveRecord::Base.connection.data_source_exists? 'authors'
-else
-  CreateAllTables.up unless ActiveRecord::Base.connection.table_exists? 'authors'
-end

data/test/kwargs_controllers.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-# this file should not be loaded from Ruby <2.0
-
-class KwBooksController < ApplicationController
-  # keyword arguments
-  def index(author_name, page: '1', q: nil)
-    render plain: {author_name: author_name, page: page, q: q}.inspect
-  end
-end

data/test/kwargs_keyreq_controllers.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-# this file should not be loaded from Ruby <2.1
-
-class KwKeyreqBooksController < ApplicationController
-  # keyword arguments
-  def index(author_name:, page: '1', q: nil)
-    render plain: {author_name: author_name, page: page, q: q}.inspect
-  end
-end

data/test/mailers/action_mailer_test.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-
-class UserMailerTest < ActionMailer::TestCase
-  test '#send_email_without_args' do
-    #it should not raise NameError: undefined local variable or method `params' for ...
-    assert UserMailer.send_email_without_args
-  end
-end

data/test/params_handler/params_handler_test.rb

@@ -1,193 +0,0 @@
-# frozen_string_literal: true
-require 'test_helper'
-using ActionArgs::ParamsHandler
-
-class ActionArgs::ParamsHandlerTest < ActiveSupport::TestCase
-  sub_test_case 'extract_method_arguments_from_params' do
-    setup do
-      params = {a: '1', b: '2'}
-      @controller = Class.new(ApplicationController).new.tap {|c| c.params = params }
-    end
-    test 'no parameters' do
-      def @controller.m() end
-
-      assert_equal [], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test '1 req' do
-      def @controller.m(a) end
-
-      assert_equal ['1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test '2 reqs' do
-      def @controller.m(a, b) end
-
-      assert_equal ['1', '2'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test '1 opt with value' do
-      def @controller.m(a = 'a') end
-
-      assert_equal ['1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test '1 opt without value' do
-      def @controller.m(x = 'x') end
-
-      assert_equal [], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'req, opt with value' do
-      def @controller.m(a, b = 'b') end
-
-      assert_equal ['1', '2'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'req, opt without value' do
-      def @controller.m(a, x = 'x') end
-
-      assert_equal ['1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt with value, opt with value' do
-      def @controller.m(a = 'a', b = 'b') end
-
-      assert_equal ['1', '2'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt with value, opt without value' do
-      def @controller.m(a = 'a', x = 'x') end
-
-      assert_equal ['1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt without value, opt with value' do
-      def @controller.m(x = 'x', a = 'a') end
-
-      assert_equal [nil, '1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt without value, opt without value' do
-      def @controller.m(x = 'x', y = 'y') end
-
-      assert_equal [], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt with value, req' do
-      def @controller.m(a = 'a', b) end
-
-      assert_equal ['1', '2'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt without value, req' do
-      def @controller.m(x = 'x', a) end
-
-      assert_equal ['1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt without value, opt with value, req' do
-      def @controller.m(x = 'x', b = 'b', a) end
-
-      assert_equal [nil, '2', '1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'opt with value, opt without value, req' do
-      def @controller.m(b = 'b', x = 'x', a) end
-
-      assert_equal ['2', '1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'req without a value' do
-      def @controller.m(x) end
-
-      assert_raises(ActionController::BadRequest) { @controller.extract_method_arguments_from_params(:m) }
-    end
-
-    test 'key' do
-      def @controller.m(a: nil) end
-
-      assert_equal [a: '1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    test 'key, key without value' do
-      def @controller.m(a: nil, x: 'x') end
-
-      assert_equal [a: '1'], @controller.extract_method_arguments_from_params(:m)
-    end
-
-    if RUBY_VERSION >= '2.1'
-      eval <<-KWARGS_KEYREQ_TEST
-        test 'keyreq' do
-          def @controller.m(a:) end
-
-          assert_equal [a: '1'], @controller.extract_method_arguments_from_params(:m)
-        end
-
-        test 'keyreq, keyreq without value' do
-          def @controller.m(a:, x:) end
-
-          assert_raises(ActionController::BadRequest) { @controller.extract_method_arguments_from_params(:m) }
-        end
-      KWARGS_KEYREQ_TEST
-    end
-  end
-
-  sub_test_case 'strengthen_params!' do
-    setup do
-      @params = ActionController::Parameters.new(x: '1', y: '2', foo: {a: 'a', b: 'b'}, bar: {a: 'a', b: 'b'}, baz: {a: 'a', b: 'b'}, hoge: {a: 'a', b: 'b'}, fuga: {a: 'a', b: 'b'})
-    end
-
-    def execute_strengthen_params!(controller)
-      c = controller.new
-      c.instance_variable_set :@_params, @params
-      c.strengthen_params! :a
-    end
-
-    test 'requiring via :req, permitting all scalars' do
-      execute_strengthen_params! FooController ||= Class.new(ApplicationController) { permits :a, :b; def a(foo) end }
-
-      assert @params[:foo].permitted?
-      assert_not_nil @params[:foo][:a]
-      assert_not_nil @params[:foo][:b]
-    end
-
-    test 'requiring via :req, not permitting all scalars' do
-      execute_strengthen_params! BarController ||= Class.new(ApplicationController) { permits :a; def a(bar, x = 'x') end }
-
-      assert @params[:bar].permitted?
-      assert_not_nil @params[:bar][:a]
-      assert_nil @params[:bar][:b]
-    end
-
-    test 'requiring via :req, not permitting any scalars' do
-      execute_strengthen_params! BazController ||= Class.new(ApplicationController) { def a(baz, aho = 'omg') end }
-
-      refute @params[:baz].permitted?
-    end
-
-    test 'requiring via :opt, permitting all scalars' do
-      execute_strengthen_params! HogeController ||= Class.new(ApplicationController) { permits :a, :b; def a(hoge = {}) end }
-
-      assert @params[:hoge].permitted?
-      assert_not_nil @params[:hoge][:a]
-      assert_not_nil @params[:hoge][:b]
-    end
-
-    test 'requiring via :key, permitting all scalars' do
-      execute_strengthen_params! FugaController ||= Class.new(ApplicationController) { permits :a, :b; def a(fuga: {}) end }
-
-      assert @params[:fuga].permitted?
-      assert_not_nil @params[:fuga][:a]
-      assert_not_nil @params[:fuga][:b]
-    end
-
-    test '"model_name" option' do
-      execute_strengthen_params! PiyoController ||= Class.new(ApplicationController) { permits :a, :b, model_name: 'Foo'; def a(foo) end }
-
-      assert @params[:foo].permitted?
-      assert_not_nil @params[:foo][:a]
-      assert_not_nil @params[:foo][:b]
-    end
-  end
-end