dry_crud 6.0.0 → 8.0.0

data/lib/generators/dry_crud/templates/test/controllers/crud_test_models_controller_test.rb

@@ -1,13 +1,12 @@
-require 'test_helper'
-require 'support/crud_test_helper'
-require 'support/crud_test_model'
-require 'support/crud_controller_test_helper'
+require "test_helper"
+require "support/crud_test_helper"
+require "support/crud_test_model"
+require "support/crud_controller_test_helper"
 
 # Tests all actions of the CrudController based on a dummy model
 # (CrudTestModel). This is useful to test the general behavior
 # of CrudController.
 class CrudTestModelsControllerTest < ActionController::TestCase
-
   include CrudControllerTestHelper
   include CrudTestHelper
 
@@ -20,13 +19,13 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   def test_setup
     assert_equal 6, CrudTestModel.count
     assert_equal CrudTestModelsController, @controller.class
-    assert_recognizes({ controller: 'crud_test_models',
-                        action: 'index' },
-                      '/crud_test_models')
-    assert_recognizes({ controller: 'crud_test_models',
-                        action: 'show',
-                        id: '1' },
-                      '/crud_test_models/1')
+    assert_recognizes({ controller: "crud_test_models",
+                        action: "index" },
+                      "/crud_test_models")
+    assert_recognizes({ controller: "crud_test_models",
+                        action: "show",
+                        id: "1" },
+                      "/crud_test_models/1")
   end
 
   def test_index
@@ -39,14 +38,14 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   def test_index_js
     get :index, params: test_params, xhr: true
     assert_response :success
-    assert_equal 'index js', @response.body
+    assert_equal "index js", @response.body
     assert entries.present?
   end
 
   def test_index_search
     super
     assert_equal 1, entries.size
-    assert_equal({ 'q' => 'AAAA' }, session[:list_params]['/crud_test_models'])
+    assert_equal({ "q" => "AAAA" }, session[:list_params]["/crud_test_models"])
   end
 
   def test_index_with_custom_options
@@ -58,65 +57,65 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   end
 
   def test_index_search_with_custom_options
-    get :index, params: { q: 'DDD', filter: true }
+    get :index, params: { q: "DDD", filter: true }
     assert_response :success
     assert entries.present?
     assert_equal 1, entries.size
-    assert_equal [CrudTestModel.find_by(name: 'BBBBB')], entries
-    assert_equal({ 'q' => 'DDD' }, session[:list_params]['/crud_test_models'])
+    assert_equal [ CrudTestModel.find_by(name: "BBBBB") ], entries
+    assert_equal({ "q" => "DDD" }, session[:list_params]["/crud_test_models"])
   end
 
   def test_sort_given_column
-    get :index, params: { sort: 'children', sort_dir: 'asc' }
+    get :index, params: { sort: "children", sort_dir: "asc" }
     assert_response :success
     assert entries.present?
     assert_equal 6, entries.size
     assert_equal CrudTestModel.all.sort_by(&:children), entries
-    assert_equal({ 'sort' => 'children', 'sort_dir' => 'asc' },
-                 session[:list_params]['/crud_test_models'])
+    assert_equal({ "sort" => "children", "sort_dir" => "asc" },
+                 session[:list_params]["/crud_test_models"])
   end
 
   def test_sort_virtual_column
-    get :index, params: { sort: 'chatty', sort_dir: 'desc' }
+    get :index, params: { sort: "chatty", sort_dir: "desc" }
     assert_response :success
     assert entries.present?
     assert_equal 6, entries.size
-    assert_equal({ 'sort' => 'chatty', 'sort_dir' => 'desc' },
-                 session[:list_params]['/crud_test_models'])
+    assert_equal({ "sort" => "chatty", "sort_dir" => "desc" },
+                 session[:list_params]["/crud_test_models"])
 
     # sort order is ambiguous, use index
     names = entries.map(&:name)
-    assert names.index('BBBBB') < names.index('AAAAA')
-    assert names.index('BBBBB') < names.index('DDDDD')
-    assert names.index('EEEEE') < names.index('AAAAA')
-    assert names.index('EEEEE') < names.index('DDDDD')
-    assert names.index('AAAAA') < names.index('CCCCC')
-    assert names.index('DDDDD') < names.index('CCCCC')
+    assert names.index("BBBBB") < names.index("AAAAA")
+    assert names.index("BBBBB") < names.index("DDDDD")
+    assert names.index("EEEEE") < names.index("AAAAA")
+    assert names.index("EEEEE") < names.index("DDDDD")
+    assert names.index("AAAAA") < names.index("CCCCC")
+    assert names.index("DDDDD") < names.index("CCCCC")
   end
 
   def test_sort_with_search
-    get :index, params: { q: 'DDD', sort: 'chatty', sort_dir: 'asc' }
+    get :index, params: { q: "DDD", sort: "chatty", sort_dir: "asc" }
     assert_response :success
     assert entries.present?
     assert_equal 3, entries.size
     assert_equal %w[CCCCC DDDDD BBBBB], entries.map(&:name)
-    assert_equal({ 'sort' => 'chatty', 'sort_dir' => 'asc', 'q' => 'DDD' },
-                 session[:list_params]['/crud_test_models'])
+    assert_equal({ "sort" => "chatty", "sort_dir" => "asc", "q" => "DDD" },
+                 session[:list_params]["/crud_test_models"])
   end
 
   def test_index_returning
     session[:list_params] = {}
-    session[:list_params]['/crud_test_models'] = { 'q' => 'DDD',
-                                                   'sort' => 'chatty',
-                                                   'sort_dir' => 'desc' }
+    session[:list_params]["/crud_test_models"] = { "q" => "DDD",
+                                                   "sort" => "chatty",
+                                                   "sort_dir" => "desc" }
     get :index, params: { returning: true }
     assert_response :success
     assert entries.present?
     assert_equal 3, entries.size
     assert_equal %w[BBBBB DDDDD CCCCC], entries.map(&:name)
-    assert_equal 'DDD', @controller.params[:q]
-    assert_equal 'chatty', @controller.params[:sort]
-    assert_equal 'desc', @controller.params[:sort_dir]
+    assert_equal "DDD", @controller.params[:q]
+    assert_equal "chatty", @controller.params[:sort]
+    assert_equal "desc", @controller.params[:sort_dir]
   end
 
   def test_new
@@ -128,7 +127,7 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   def test_show_with_custom
     get :show, params: test_params(id: crud_test_models(:BBBBB).id)
     assert_response :success
-    assert_equal 'custom html', @response.body
+    assert_equal "custom html", @response.body
   end
 
   def test_create
@@ -147,8 +146,8 @@ class CrudTestModelsControllerTest < ActionController::TestCase
 
   def test_update
     super
-    assert_equal I18n.t('crud.update.flash.success',
-                        model: 'Crud Test Model <i>foo</i>'),
+    assert_equal I18n.t("crud.update.flash.success",
+                        model: "Crud Test Model <i>foo</i>"),
                  flash[:notice]
     assert flash[:alert].blank?
     assert_equal %i[before_update before_save after_save after_update],
@@ -159,42 +158,42 @@ class CrudTestModelsControllerTest < ActionController::TestCase
     super
     assert_equal %i[before_destroy after_destroy],
                  @controller.called_callbacks
-    assert_equal I18n.t('crud.destroy.flash.success',
-                        model: 'Crud Test Model <i>AAAAA</i>'),
+    assert_equal I18n.t("crud.destroy.flash.success",
+                        model: "Crud Test Model <i>AAAAA</i>"),
                  flash[:notice]
   end
 
   def test_create_with_before_callback
-    assert_no_difference('CrudTestModel.count') do
+    assert_no_difference("CrudTestModel.count") do
       post :create,
-           params: { crud_test_model: { name: 'illegal', children: 2 } }
+           params: { crud_test_model: { name: "illegal", children: 2 } }
     end
     assert_response :success
     assert entry.new_record?
     assert flash[:alert].present?
-    assert_equal 'illegal', entry.name
+    assert_equal "illegal", entry.name
     assert_equal %i[before_render_new before_render_form],
                  @controller.called_callbacks
   end
 
   def test_create_with_before_callback_redirect
     @controller.should_redirect = true
-    assert_no_difference('CrudTestModel.count') do
+    assert_no_difference("CrudTestModel.count") do
       post :create,
-           params: { crud_test_model: { name: 'illegal', children: 2 } }
+           params: { crud_test_model: { name: "illegal", children: 2 } }
     end
-    assert_redirected_to action: 'index'
+    assert_redirected_to action: "index"
     assert_nil @controller.called_callbacks
   end
 
   def test_new_with_before_render_callback_redirect_does_not_set_companions
     @controller.should_redirect = true
     get :new
-    assert_redirected_to action: 'index'
+    assert_redirected_to action: "index"
   end
 
   def test_create_with_failure
-    assert_no_difference('CrudTestModel.count') do
+    assert_no_difference("CrudTestModel.count") do
       post :create, params: { crud_test_model: { children: 2 } }
     end
     assert_response :success
@@ -208,8 +207,8 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   end
 
   def test_create_with_failure_json
-    assert_no_difference('CrudTestModel.count') do
-      post :create, params: { crud_test_model: { children: 2 }, format: 'json' }
+    assert_no_difference("CrudTestModel.count") do
+      post :create, params: { crud_test_model: { children: 2 }, format: "json" }
     end
     assert_response :unprocessable_entity
     assert entry.new_record?
@@ -234,7 +233,7 @@ class CrudTestModelsControllerTest < ActionController::TestCase
           id: test_entry.id,
           crud_test_model: {
             rating: 20,
-            other_ids: [OtherCrudTestModel.first.id]
+            other_ids: [ OtherCrudTestModel.first.id ]
           }
         }
     assert_response :success
@@ -252,7 +251,7 @@ class CrudTestModelsControllerTest < ActionController::TestCase
     put :update,
         params: { id: test_entry.id,
                   crud_test_model: { rating: 20 },
-                  format: 'json' }
+                  format: "json" }
     assert_response :unprocessable_entity
     assert entry.changed?
     assert flash[:notice].blank?
@@ -262,7 +261,7 @@ class CrudTestModelsControllerTest < ActionController::TestCase
 
   def test_destroy_failure
     assert_no_difference("#{model_class.name}.count") do
-      @request.env['HTTP_REFERER'] =
+      @request.env["HTTP_REFERER"] =
         crud_test_model_url(crud_test_models(:BBBBB))
       delete :destroy, params: test_params(id: crud_test_models(:BBBBB).id)
     end
@@ -274,7 +273,7 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   def test_destroy_failure_callback
     e = crud_test_models(:AAAAA)
     # rubocop:disable Rails/SkipsModelValidations
-    e.update_attribute(:name, 'illegal')
+    e.update_attribute(:name, "illegal")
     # rubocop:enable Rails/SkipsModelValidations
     assert_no_difference("#{model_class.name}.count") do
       delete :destroy, params: test_params(id: e.id)
@@ -287,15 +286,15 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   def test_destroy_failure_json
     assert_no_difference("#{model_class.name}.count") do
       delete :destroy, params: test_params(id: crud_test_models(:BBBBB).id,
-                                           format: 'json')
+                                           format: "json")
     end
     assert_response :unprocessable_entity
     assert flash[:notice].blank?
   end
 
   def test_models_label
-    assert_equal 'Crud Test Models', @controller.models_label
-    assert_equal 'Crud Test Model', @controller.models_label(false)
+    assert_equal "Crud Test Models", @controller.models_label
+    assert_equal "Crud Test Model", @controller.models_label(plural: false)
   end
 
   private
@@ -305,24 +304,23 @@ class CrudTestModelsControllerTest < ActionController::TestCase
   end
 
   def new_entry_attrs
-    { name: 'foo',
+    { name: "foo",
       children: 42,
       companion_id: 3,
       rating: 8.5,
       income: 2.42,
-      birthdate: '31-12-1999'.to_date,
+      birthdate: "31-12-1999".to_date,
       human: true,
       remarks: "some custom\n\tremarks" }
   end
 
   def edit_entry_attrs
-    { name: 'foo',
+    { name: "foo",
       children: 42,
       rating: 8.5,
       income: 2.42,
-      birthdate: '31-12-1999'.to_date,
+      birthdate: "31-12-1999".to_date,
       human: true,
       remarks: "some custom\n\tremarks" }
   end
-
 end

data/lib/generators/dry_crud/templates/test/helpers/custom_assertions_test.rb

@@ -1,73 +1,72 @@
-require 'test_helper'
-require 'support/custom_assertions'
-require 'support/crud_test_helper'
-require 'support/crud_test_model'
+require "test_helper"
+require "support/custom_assertions"
+require "support/crud_test_helper"
+require "support/crud_test_model"
 
 # Test CustomAssertions
 class CustomAssertionsTest < ActiveSupport::TestCase
-
   include CustomAssertions
   include CrudTestHelper
 
   setup :reset_db, :setup_db, :create_test_data
   teardown :reset_db
 
-  test 'assert count succeeds if count matches' do
+  test "assert count succeeds if count matches" do
     assert_nothing_raised do
-      assert_count 3, 'ba', 'barbabapa'
+      assert_count 3, "ba", "barbabapa"
     end
   end
 
-  test 'assert count succeeds if count is zero' do
+  test "assert count succeeds if count is zero" do
     assert_nothing_raised do
-      assert_count 0, 'bo', 'barbabapa'
+      assert_count 0, "bo", "barbabapa"
     end
   end
 
-  test 'assert count fails if count does not match' do
-    assert_raise(MiniTest::Assertion) do
-      assert_count 2, 'ba', 'barbabapa'
+  test "assert count fails if count does not match" do
+    assert_raise(Minitest::Assertion) do
+      assert_count 2, "ba", "barbabapa"
     end
   end
 
-  test 'assert valid record succeeds' do
+  test "assert valid record succeeds" do
     assert_nothing_raised do
-      assert_valid crud_test_models('AAAAA')
+      assert_valid crud_test_models("AAAAA")
     end
   end
 
-  test 'assert valid record fails for invalid' do
-    assert_raise(MiniTest::Assertion) do
+  test "assert valid record fails for invalid" do
+    assert_raise(Minitest::Assertion) do
       assert_valid invalid_record
     end
   end
 
-  test 'assert not valid succeeds if record invalid' do
+  test "assert not valid succeeds if record invalid" do
     assert_nothing_raised do
       assert_not_valid invalid_record
     end
   end
 
-  test 'assert not valid succeds if record invalid and invalid attrs given' do
+  test "assert not valid succeds if record invalid and invalid attrs given" do
     assert_nothing_raised do
       assert_not_valid invalid_record, :name, :rating
     end
   end
 
-  test 'assert not valid fails if record valid' do
-    assert_raise(MiniTest::Assertion) do
-      assert_not_valid crud_test_models('AAAAA')
+  test "assert not valid fails if record valid" do
+    assert_raise(Minitest::Assertion) do
+      assert_not_valid crud_test_models("AAAAA")
     end
   end
 
-  test 'assert not valid fails if record invalid and valid attrs given' do
-    assert_raise(MiniTest::Assertion) do
+  test "assert not valid fails if record invalid and valid attrs given" do
+    assert_raise(Minitest::Assertion) do
       assert_not_valid invalid_record, :name, :rating, :children
     end
   end
 
-  test 'assert not valid fails if not all invalid attrs given' do
-    assert_raise(MiniTest::Assertion) do
+  test "assert not valid fails if not all invalid attrs given" do
+    assert_raise(Minitest::Assertion) do
       assert_not_valid invalid_record, :name
     end
   end
@@ -75,10 +74,9 @@ class CustomAssertionsTest < ActiveSupport::TestCase
   private
 
   def invalid_record
-    m = crud_test_models('AAAAA')
+    m = crud_test_models("AAAAA")
     m.name = nil
     m.rating = 42
     m
   end
-
 end