").html_safe?
end
- unless ENV['NON_LOCALIZED'] # localization dependent tests
- test 'format Floats' do
- assert_equal '1.000', f(1.0)
- assert_equal '1.200', f(1.2)
- assert_equal '3.142', f(3.14159)
+ unless ENV["NON_LOCALIZED"] # localization dependent tests
+ test "format Floats" do
+ assert_equal "1.000", f(1.0)
+ assert_equal "1.200", f(1.2)
+ assert_equal "3.142", f(3.14159)
end
- test 'format Booleans' do
- assert_equal 'yes', f(true)
- assert_equal 'no', f(false)
+ test "format Booleans" do
+ assert_equal "yes", f(true)
+ assert_equal "no", f(false)
end
- test 'format attr with fallthrough to f' do
- assert_equal '12.234', format_attr('12.23424', :to_f)
+ test "format attr with fallthrough to f" do
+ assert_equal "12.234", format_attr("12.23424", :to_f)
end
end
- test 'format attr with custom format_string_size method' do
- assert_equal '4 chars', format_attr('abcd', :size)
+ test "format attr with custom format_string_size method" do
+ assert_equal "4 chars", format_attr("abcd", :size)
end
- test 'format attr with custom format_size method' do
- assert_equal '2 items', format_attr([1, 2], :size)
+ test "format attr with custom format_size method" do
+ assert_equal "2 items", format_attr([ 1, 2 ], :size)
end
- test 'format integer column' do
+ test "format integer column" do
m = crud_test_models(:AAAAA)
- assert_equal '9', format_type(m, :children)
+ assert_equal "9", format_type(m, :children)
m.children = 10_000
- assert_equal '10,000', format_type(m, :children)
+ assert_equal "10,000", format_type(m, :children)
end
- unless ENV['NON_LOCALIZED'] # localization dependent tests
- test 'format float column' do
+ unless ENV["NON_LOCALIZED"] # localization dependent tests
+ test "format float column" do
m = crud_test_models(:AAAAA)
- assert_equal '1.100', format_type(m, :rating)
+ assert_equal "1.100", format_type(m, :rating)
m.rating = 3.145001 # you never know with these floats..
- assert_equal '3.145', format_type(m, :rating)
+ assert_equal "3.145", format_type(m, :rating)
end
- test 'format decimal column' do
+ test "format decimal column" do
m = crud_test_models(:AAAAA)
- assert_equal '10,000,000.1111', format_type(m, :income)
+ assert_equal "10,000,000.1111", format_type(m, :income)
end
- test 'format date column' do
+ test "format date column" do
m = crud_test_models(:AAAAA)
- assert_equal '1910-01-01', format_type(m, :birthdate)
+ assert_equal "1910-01-01", format_type(m, :birthdate)
end
- test 'format datetime column' do
+ test "format datetime column" do
m = crud_test_models(:AAAAA)
- assert_equal '2010-01-01 11:21', format_type(m, :last_seen)
+ assert_equal "2010-01-01 11:21", format_type(m, :last_seen)
end
end
- test 'format time column' do
+ test "format time column" do
m = crud_test_models(:AAAAA)
- assert_equal '01:01', format_type(m, :gets_up_at)
+ assert_equal "01:01", format_type(m, :gets_up_at)
end
- test 'format text column' do
+ test "format text column" do
m = crud_test_models(:AAAAA)
assert_equal "AAAAA BBBBB CCCCC\n
AAAAA BBBBB CCCCC\n
",
format_type(m, :remarks)
assert format_type(m, :remarks).html_safe?
end
- test 'format boolean false column' do
+ test "format boolean false column" do
m = crud_test_models(:AAAAA)
m.human = false
- assert_equal 'no', format_type(m, :human)
+ assert_equal "no", format_type(m, :human)
end
- test 'format boolean true column' do
+ test "format boolean true column" do
m = crud_test_models(:AAAAA)
m.human = true
- assert_equal 'yes', format_type(m, :human)
+ assert_equal "yes", format_type(m, :human)
end
- test 'format belongs to column without content' do
+ test "format belongs to column without content" do
m = crud_test_models(:AAAAA)
- assert_equal t('global.associations.no_entry'),
+ assert_equal t("global.associations.no_entry"),
format_attr(m, :companion)
end
- test 'format belongs to column with content' do
+ test "format belongs to column with content" do
m = crud_test_models(:BBBBB)
- assert_equal 'AAAAA', format_attr(m, :companion)
+ assert_equal "AAAAA", format_attr(m, :companion)
end
- test 'format has one without content' do
+ test "format has one without content" do
m = crud_test_models(:FFFFF)
- assert_equal t('global.associations.no_entry'),
+ assert_equal t("global.associations.no_entry"),
format_attr(m, :comrad)
end
- test 'format has one with content' do
+ test "format has one with content" do
m = crud_test_models(:AAAAA)
- assert_equal 'BBBBB', format_attr(m, :comrad)
+ assert_equal "BBBBB", format_attr(m, :comrad)
end
- test 'format has_many column with content' do
+ test "format has_many column with content" do
m = crud_test_models(:CCCCC)
- assert_equal '',
+ assert_equal "",
format_attr(m, :others)
end
- test 'captionize' do
- assert_equal 'Camel Case', captionize(:camel_case)
- assert_equal 'All Upper Case', captionize('all upper case')
- assert_equal 'With Object', captionize('With object', Object.new)
- assert_not captionize('bad ').html_safe?
+ test "captionize" do
+ assert_equal "Camel Case", captionize(:camel_case)
+ assert_equal "All Upper Case", captionize("all upper case")
+ assert_equal "With Object", captionize("With object", Object.new)
+ assert_not captionize("bad <title>").html_safe?
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/helpers/i18n_helper_test.rb b/lib/generators/dry_crud/templates/test/helpers/i18n_helper_test.rb
index b332ff56..3d05a539 100644
--- a/lib/generators/dry_crud/templates/test/helpers/i18n_helper_test.rb
+++ b/lib/generators/dry_crud/templates/test/helpers/i18n_helper_test.rb
@@ -1,79 +1,78 @@
-require 'test_helper'
-require 'support/crud_test_model'
-require 'support/crud_test_models_controller'
+require "test_helper"
+require "support/crud_test_model"
+require "support/crud_test_models_controller"
# Test I18nHelper
class I18nHelperTest < ActionView::TestCase
-
include CrudTestHelper
- test 'translate inheritable lookup' do
+ test "translate inheritable lookup" do
# current controller is :crud_test_models, action is :index
@controller = CrudTestModelsController.new
I18n.backend.store_translations(
I18n.locale,
- global: { test_key: 'global' }
+ global: { test_key: "global" }
)
- assert_equal 'global', ti(:test_key)
+ assert_equal "global", ti(:test_key)
I18n.backend.store_translations(
I18n.locale,
- list: { global: { test_key: 'list global' } }
+ list: { global: { test_key: "list global" } }
)
- assert_equal 'list global', ti(:test_key)
+ assert_equal "list global", ti(:test_key)
I18n.backend.store_translations(
I18n.locale,
- list: { index: { test_key: 'list index' } }
+ list: { index: { test_key: "list index" } }
)
- assert_equal 'list index', ti(:test_key)
+ assert_equal "list index", ti(:test_key)
I18n.backend.store_translations(
I18n.locale,
- crud: { global: { test_key: 'crud global' } }
+ crud: { global: { test_key: "crud global" } }
)
- assert_equal 'crud global', ti(:test_key)
+ assert_equal "crud global", ti(:test_key)
I18n.backend.store_translations(
I18n.locale,
- crud: { index: { test_key: 'crud index' } }
+ crud: { index: { test_key: "crud index" } }
)
- assert_equal 'crud index', ti(:test_key)
+ assert_equal "crud index", ti(:test_key)
I18n.backend.store_translations(
I18n.locale,
- crud_test_models: { global: { test_key: 'test global' } }
+ crud_test_models: { global: { test_key: "test global" } }
)
- assert_equal 'test global', ti(:test_key)
+ assert_equal "test global", ti(:test_key)
I18n.backend.store_translations(
I18n.locale,
- crud_test_models: { index: { test_key: 'test index' } }
+ crud_test_models: { index: { test_key: "test index" } }
)
- assert_equal 'test index', ti(:test_key)
+ assert_equal "test index", ti(:test_key)
end
- test 'translate association lookup' do
+ test "translate association lookup" do
assoc = CrudTestModel.reflect_on_association(:companion)
I18n.backend.store_translations(
I18n.locale,
- global: { associations: { test_key: 'global' } }
+ global: { associations: { test_key: "global" } }
)
- assert_equal 'global', ta(:test_key, assoc)
+ assert_equal "global", ta(:test_key, assoc)
I18n.backend.store_translations(
I18n.locale,
activerecord: {
associations: {
crud_test_model: {
- test_key: 'model'
+ test_key: "model"
}
}
}
)
- assert_equal 'model', ta(:test_key, assoc)
+ assert_equal "model", ta(:test_key, assoc)
I18n.backend.store_translations(
I18n.locale,
@@ -82,16 +81,15 @@ class I18nHelperTest < ActionView::TestCase
models: {
crud_test_model: {
companion: {
- test_key: 'companion'
+ test_key: "companion"
}
}
}
}
}
)
- assert_equal 'companion', ta(:test_key, assoc)
+ assert_equal "companion", ta(:test_key, assoc)
- assert_equal 'global', ta(:test_key)
+ assert_equal "global", ta(:test_key)
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/helpers/table_helper_test.rb b/lib/generators/dry_crud/templates/test/helpers/table_helper_test.rb
index 7904c913..a2cb8cd0 100644
--- a/lib/generators/dry_crud/templates/test/helpers/table_helper_test.rb
+++ b/lib/generators/dry_crud/templates/test/helpers/table_helper_test.rb
@@ -1,10 +1,9 @@
-require 'test_helper'
-require 'support/custom_assertions'
-require 'support/crud_test_model'
+require "test_helper"
+require "support/custom_assertions"
+require "support/crud_test_model"
# Test TableHelper
class TableHelperTest < ActionView::TestCase
-
include UtilityHelper
include FormatHelper
include I18nHelper
@@ -30,13 +29,13 @@ def empty_params
@params = {}
end
- test 'empty table renders message' do
+ test "empty table renders message" do
result = plain_table_or_message([])
assert result.html_safe?
assert_match(/<div class=["']table["']>.*<\/div>/, result)
end
- test 'non empty table renders table' do
+ test "non empty table renders table" do
result = plain_table_or_message(%w[foo bar]) do |t|
t.attrs :size, :upcase
end
@@ -44,10 +43,10 @@ def empty_params
assert_match(/^<table.*<\/table>$/, result)
end
- test 'table with attrs' do
+ test "table with attrs" do
expected = DryCrud::Table::Builder.table(
%w[foo bar], self,
- class: 'table table-striped table-hover'
+ class: "table table-striped table-hover"
) do |t|
t.attrs :size, :upcase
end
@@ -56,7 +55,7 @@ def empty_params
assert_equal expected, actual
end
- test 'standard list table' do
+ test "standard list table" do
@entries = CrudTestModel.all
table = with_test_routing do
@@ -67,7 +66,7 @@ def empty_params
assert_count 14, REGEXP_SORT_HEADERS, table
end
- test 'custom list table with attributes' do
+ test "custom list table with attributes" do
@entries = CrudTestModel.all
table = with_test_routing do
@@ -78,13 +77,13 @@ def empty_params
assert_count 3, REGEXP_SORT_HEADERS, table
end
- test 'custom list table with block' do
+ test "custom list table with block" do
@entries = CrudTestModel.all
table = with_test_routing do
list_table do |t|
t.attrs :name, :children, :companion_id
- t.col('head') { |e| tag.span(e.income.to_s) }
+ t.col("head") { |e| tag.span(e.income.to_s) }
end
end
@@ -94,12 +93,12 @@ def empty_params
assert_count 6, /<span>.+?<\/span>/, table
end
- test 'custom list table with attributes and block' do
+ test "custom list table with attributes and block" do
@entries = CrudTestModel.all
table = with_test_routing do
list_table :name, :children, :companion_id do |t|
- t.col('head') { |e| tag.span(e.income.to_s) }
+ t.col("head") { |e| tag.span(e.income.to_s) }
end
end
@@ -109,8 +108,8 @@ def empty_params
assert_count 6, /<span>.+?<\/span>/, table
end
- test 'standard list table with ascending sort params' do
- @params = { sort: 'children', sort_dir: 'asc' }
+ test "standard list table with ascending sort params" do
+ @params = { sort: "children", sort_dir: "asc" }
@entries = CrudTestModel.all
table = with_test_routing do
@@ -123,8 +122,8 @@ def empty_params
assert_count 1, sort_header_desc, table
end
- test 'standard list table with descending sort params' do
- @params = { sort: 'children', sort_dir: 'desc' }
+ test "standard list table with descending sort params" do
+ @params = { sort: "children", sort_dir: "desc" }
@entries = CrudTestModel.all
table = with_test_routing do
@@ -137,8 +136,8 @@ def empty_params
assert_count 1, sort_header_asc, table
end
- test 'list table with custom column sort params' do
- @params = { sort: 'chatty', sort_dir: 'asc' }
+ test "list table with custom column sort params" do
+ @params = { sort: "chatty", sort_dir: "asc" }
@entries = CrudTestModel.all
table = with_test_routing do
@@ -151,7 +150,7 @@ def empty_params
assert_count 1, sort_header_desc, table
end
- test 'standard crud table' do
+ test "standard crud table" do
@entries = CrudTestModel.all
table = with_test_routing do
@@ -163,7 +162,7 @@ def empty_params
assert_count 12, REGEXP_ACTION_CELL, table # edit, delete links
end
- test 'custom crud table with attributes' do
+ test "custom crud table with attributes" do
@entries = CrudTestModel.all
table = with_test_routing do
@@ -175,13 +174,13 @@ def empty_params
assert_count 12, REGEXP_ACTION_CELL, table # edit, delete links
end
- test 'custom crud table with block' do
+ test "custom crud table with block" do
@entries = CrudTestModel.all
table = with_test_routing do
crud_table do |t|
t.attrs :name, :children, :companion_id
- t.col('head') { |e| tag.span(e.income.to_s) }
+ t.col("head") { |e| tag.span(e.income.to_s) }
end
end
@@ -191,12 +190,12 @@ def empty_params
assert_count 12, REGEXP_ACTION_CELL, table # edit, delete links
end
- test 'custom crud table with attributes and block' do
+ test "custom crud table with attributes and block" do
@entries = CrudTestModel.all
table = with_test_routing do
crud_table :name, :children, :companion_id do |t|
- t.col('head') { |e| tag.span(e.income.to_s) }
+ t.col("head") { |e| tag.span(e.income.to_s) }
end
end
@@ -210,5 +209,4 @@ def empty_params
def entry
@entry ||= CrudTestModel.first
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/helpers/utility_helper_test.rb b/lib/generators/dry_crud/templates/test/helpers/utility_helper_test.rb
index b301f84d..c26335e0 100644
--- a/lib/generators/dry_crud/templates/test/helpers/utility_helper_test.rb
+++ b/lib/generators/dry_crud/templates/test/helpers/utility_helper_test.rb
@@ -1,47 +1,46 @@
-require 'test_helper'
-require 'support/crud_test_model'
+require "test_helper"
+require "support/crud_test_model"
# Test UtilityHelper
class UtilityHelperTest < ActionView::TestCase
-
include CrudTestHelper
setup :reset_db, :setup_db, :create_test_data
teardown :reset_db
- test 'content_tag_nested escapes safe correctly' do
+ test "content_tag_nested escapes safe correctly" do
html = content_tag_nested(:div, %w[a b]) { |e| tag.span(e) }
- assert_equal '<div><span>a</span><span>b</span></div>', html
+ assert_equal "<div><span>a</span><span>b</span></div>", html
end
- test 'content_tag_nested escapes unsafe correctly' do
+ test "content_tag_nested escapes unsafe correctly" do
html = content_tag_nested(:div, %w[a b]) { |e| "<#{e}>" }
- assert_equal '<div><a><b></div>', html
+ assert_equal "<div><a><b></div>", html
end
- test 'content_tag_nested without block' do
+ test "content_tag_nested without block" do
html = content_tag_nested(:div, %w[a b])
- assert_equal '<div>ab</div>', html
+ assert_equal "<div>ab</div>", html
end
- test 'safe_join without block' do
- html = safe_join(['<a>', '<b>'.html_safe])
- assert_equal '<a><b>', html
+ test "safe_join without block" do
+ html = safe_join([ "<a>", "<b>".html_safe ])
+ assert_equal "<a><b>", html
end
- test 'safe_join with block' do
+ test "safe_join with block" do
html = safe_join(%w[a b]) { |e| tag.span(e) }
- assert_equal '<span>a</span><span>b</span>', html
+ assert_equal "<span>a</span><span>b</span>", html
end
- test 'default attributes do not include id and password' do
+ test "default attributes do not include id and password" do
assert_equal %i[name email whatever children companion_id rating
income birthdate gets_up_at last_seen human
remarks created_at updated_at],
default_crud_attrs
end
- test 'column types' do
+ test "column types" do
m = crud_test_models(:AAAAA)
assert_equal :string, column_type(m, :name)
assert_equal :integer, column_type(m, :children)
@@ -55,5 +54,4 @@ class UtilityHelperTest < ActionView::TestCase
assert_equal :boolean, column_type(m, :human)
assert_equal :text, column_type(m, :remarks)
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/support/crud_controller_test_helper.rb b/lib/generators/dry_crud/templates/test/support/crud_controller_test_helper.rb
index 071ddd7c..0ab0c201 100644
--- a/lib/generators/dry_crud/templates/test/support/crud_controller_test_helper.rb
+++ b/lib/generators/dry_crud/templates/test/support/crud_controller_test_helper.rb
@@ -3,7 +3,6 @@
# #test_entry_attrs to test the basic crud functionality. Override the test
# methods if you changed the behaviour in your subclass controller.
module CrudControllerTestHelper # rubocop:disable Metrics/ModuleLength
-
def test_index # :nodoc:
get :index, params: test_params
assert_response :success
@@ -11,13 +10,13 @@ def test_index # :nodoc:
end
def test_index_json # :nodoc:
- get :index, params: test_params(format: 'json')
+ get :index, params: test_params(format: "json")
assert_response :success
assert entries.present?
- assert @response.body.starts_with?('[{'), @response.body
+ assert @response.body.starts_with?("[{"), @response.body
end
- def test_index_search # rubocop:disable Metrics/AbcSize :nodoc:
+ def test_index_search # rubocop:disable Metrics/AbcSize -- :nodoc:
field = @controller.search_columns.first
val = field && test_entry[field].to_s
return if val.blank? # does not support search or no value in this field
@@ -30,7 +29,7 @@ def test_index_search # rubocop:disable Metrics/AbcSize :nodoc:
def test_index_sort_asc # :nodoc:
col = model_class.column_names.first
- get :index, params: test_params(sort: col, sort_dir: 'asc')
+ get :index, params: test_params(sort: col, sort_dir: "asc")
assert_response :success
assert entries.present?
sorted = entries.sort_by(&col.to_sym)
@@ -39,7 +38,7 @@ def test_index_sort_asc # :nodoc:
def test_index_sort_desc # :nodoc:
col = model_class.column_names.first
- get :index, params: test_params(sort: col, sort_dir: 'desc')
+ get :index, params: test_params(sort: col, sort_dir: "desc")
assert_response :success
assert entries.present?
sorted = entries.to_a.sort_by(&col.to_sym)
@@ -53,10 +52,10 @@ def test_show # :nodoc:
end
def test_show_json # :nodoc:
- get :show, params: test_params(id: test_entry.id, format: 'json')
+ get :show, params: test_params(id: test_entry.id, format: "json")
assert_response :success
assert_equal test_entry, entry
- assert @response.body.starts_with?('{')
+ assert @response.body.starts_with?("{")
end
def test_show_with_non_existing_id_raises_record_not_found # :nodoc:
@@ -83,7 +82,7 @@ def test_create # :nodoc:
def test_create_json # :nodoc:
assert_difference("#{model_class.name}.count") do
post :create, params: test_params(model_identifier => new_entry_attrs,
- format: 'json')
+ format: "json")
end
assert_response :success
assert @response.body.starts_with?('{"id":')
@@ -108,7 +107,7 @@ def test_update_json # :nodoc:
assert_no_difference("#{model_class.name}.count") do
put :update, params: test_params(id: test_entry.id,
model_identifier => edit_entry_attrs,
- format: 'json')
+ format: "json")
end
assert_response :success
assert @response.body.starts_with?('{"id":')
@@ -124,22 +123,22 @@ def test_destroy # :nodoc:
def test_destroy_json # :nodoc:
assert_difference("#{model_class.name}.count", -1) do
delete :destroy, params: test_params(id: test_entry.id,
- format: 'json')
+ format: "json")
end
assert_response :success
- assert_equal '', @response.body.strip
+ assert_equal "", @response.body.strip
end
private
def assert_redirected_to_index # :nodoc:
- assert_redirected_to test_params(action: 'index',
+ assert_redirected_to test_params(action: "index",
id: nil,
returning: true)
end
def assert_redirected_to_show(entry) # :nodoc:
- assert_redirected_to test_params(action: 'show',
+ assert_redirected_to test_params(action: "show",
id: entry.id)
end
@@ -212,5 +211,4 @@ def nesting_params
end
params
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/support/crud_test_helper.rb b/lib/generators/dry_crud/templates/test/support/crud_test_helper.rb
index 64242927..493c4c4c 100644
--- a/lib/generators/dry_crud/templates/test/support/crud_test_helper.rb
+++ b/lib/generators/dry_crud/templates/test/support/crud_test_helper.rb
@@ -8,7 +8,6 @@
# This helper is used to test the CrudController and various helpers
# without the need for an application based model.
module CrudTestHelper
-
# Controller helper methods for the tests
def model_class
@@ -16,11 +15,11 @@ def model_class
end
def controller_name
- 'crud_test_models'
+ "crud_test_models"
end
def action_name
- 'index'
+ "index"
end
def params
@@ -35,9 +34,7 @@ def sortable?(_attr)
true
end
- def h(text)
- ERB::Util.h(text)
- end
+ delegate :h, to: :'ERB::Util'
private
@@ -85,8 +82,8 @@ def create_other_crud_test_models(connection)
def create_crud_test_models_other_crud_test_models(connection)
connection.create_table :crud_test_models_other_crud_test_models,
force: true do |t|
- t.belongs_to :crud_test_model, index: { name: 'parent' }
- t.belongs_to :other_crud_test_model, index: { name: 'other' }
+ t.belongs_to :crud_test_model, index: { name: "parent" }
+ t.belongs_to :other_crud_test_model, index: { name: "other" }
end
end
@@ -116,7 +113,7 @@ def with_test_routing
with_routing do |set|
set.draw { resources :crud_test_models }
# used to define a controller in these tests
- set.default_url_options = { controller: 'crud_test_models' }
+ set.default_url_options = { controller: "crud_test_models" }
yield
end
end
@@ -181,16 +178,15 @@ def str(index)
def without_transaction
c = ActiveRecord::Base.connection
start_transaction = false
- if c.adapter_name.downcase.include?('mysql') &&
+ if c.adapter_name.downcase.include?("mysql") &&
c.open_transactions.positive?
# in transactional tests, we may simply rollback
- c.execute('ROLLBACK')
+ c.execute("ROLLBACK")
start_transaction = true
end
yield
- c.execute('BEGIN') if start_transaction
+ c.execute("BEGIN") if start_transaction
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/support/crud_test_model.rb b/lib/generators/dry_crud/templates/test/support/crud_test_model.rb
index aef52bcc..7871592b 100644
--- a/lib/generators/dry_crud/templates/test/support/crud_test_model.rb
+++ b/lib/generators/dry_crud/templates/test/support/crud_test_model.rb
@@ -1,12 +1,11 @@
# A dummy model used for general testing.
class CrudTestModel < ApplicationRecord # :nodoc:
-
- belongs_to :companion, class_name: 'CrudTestModel', optional: true
- has_and_belongs_to_many :others, class_name: 'OtherCrudTestModel'
- has_many :mores, class_name: 'OtherCrudTestModel',
+ belongs_to :companion, class_name: "CrudTestModel", optional: true
+ has_and_belongs_to_many :others, class_name: "OtherCrudTestModel"
+ has_many :mores, class_name: "OtherCrudTestModel",
foreign_key: :more_id
- has_one :comrad, class_name: 'CrudTestModel', foreign_key: :companion_id
+ has_one :comrad, class_name: "CrudTestModel", foreign_key: :companion_id
before_destroy :protect_if_companion
@@ -25,23 +24,20 @@ def chatty
def protect_if_companion
if companion.present?
- errors.add(:base, 'Cannot destroy model with companion')
+ errors.add(:base, "Cannot destroy model with companion")
throw :abort
end
end
-
end
# Second dummy model to test associations.
class OtherCrudTestModel < ApplicationRecord # :nodoc:
-
- has_and_belongs_to_many :others, class_name: 'CrudTestModel'
+ has_and_belongs_to_many :others, class_name: "CrudTestModel"
belongs_to :more,
- class_name: 'CrudTestModel',
+ class_name: "CrudTestModel",
optional: true
def to_s
name
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/support/crud_test_models_controller.rb b/lib/generators/dry_crud/templates/test/support/crud_test_models_controller.rb
index f7917f65..308e810e 100644
--- a/lib/generators/dry_crud/templates/test/support/crud_test_models_controller.rb
+++ b/lib/generators/dry_crud/templates/test/support/crud_test_models_controller.rb
@@ -1,15 +1,14 @@
# Controller for the dummy model.
class CrudTestModelsController < CrudController # :nodoc:
-
- HANDLE_PREFIX = 'handle_'.freeze
+ HANDLE_PREFIX = "handle_".freeze
self.search_columns = %i[name whatever remarks]
- self.sort_mappings = { chatty: 'length(remarks)' }
- self.default_sort = 'name'
- self.permitted_attrs = [:name, :email, :password, :whatever, :children,
+ self.sort_mappings = { chatty: "length(remarks)" }
+ self.default_sort = "name"
+ self.permitted_attrs = [ :name, :email, :password, :whatever, :children,
:companion_id, :rating, :income, :birthdate,
:gets_up_at, :last_seen, :human, :remarks,
- { other_ids: [] }]
+ { other_ids: [] } ]
before_create :possibly_redirect
before_create :handle_name
@@ -27,16 +26,16 @@ class CrudTestModelsController < CrudController # :nodoc:
def index
entries
- render plain: 'index js' if request.format.js?
+ render plain: "index js" if request.format.js?
end
def show
- render html: 'custom html' if entry.name == 'BBBBB'
+ render html: "custom html" if entry.name == "BBBBB"
end
def create
super do |_format, success|
- flash[:notice] = 'model got created' if success
+ flash[:notice] = "model got created" if success
end
end
@@ -45,9 +44,9 @@ def create
def list_entries
entries = super
if params[:filter]
- entries = entries.where(['rating < ?', 3])
+ entries = entries.where(rating: ...3)
.except(:order)
- .order('children DESC')
+ .order("children DESC")
end
entries
end
@@ -60,15 +59,15 @@ def build_entry
# custom callback
def handle_name
- if entry.name == 'illegal'
- flash[:alert] = 'illegal name'
+ if entry.name == "illegal"
+ flash[:alert] = "illegal name"
throw :abort
end
end
# callback to redirect if @should_redirect is set
def possibly_redirect
- redirect_to action: 'index' if should_redirect && !performed?
+ redirect_to action: "index" if should_redirect && !performed?
end
def set_companions
@@ -107,5 +106,4 @@ def called_callback(callback)
@called_callbacks ||= []
@called_callbacks << callback
end
-
end
diff --git a/lib/generators/dry_crud/templates/test/support/custom_assertions.rb b/lib/generators/dry_crud/templates/test/support/custom_assertions.rb
index b2ccc487..30aff1cb 100644
--- a/lib/generators/dry_crud/templates/test/support/custom_assertions.rb
+++ b/lib/generators/dry_crud/templates/test/support/custom_assertions.rb
@@ -8,9 +8,8 @@
# # inside the class definition:
# include CustomAssertions
module CustomAssertions
-
# Asserts that regexp occurs exactly expected times in string.
- def assert_count(expected, regexp, string, msg = '')
+ def assert_count(expected, regexp, string, msg = "")
actual = string.scan(regexp).size
msg = message(msg) do
"Expected #{mu_pp(regexp)} to occur #{expected} time(s), " \
@@ -21,7 +20,7 @@ def assert_count(expected, regexp, string, msg = '')
# Asserts that the given active model record is valid.
# This method used to be part of Rails but was deprecated, no idea why.
- def assert_valid(record, msg = '')
+ def assert_valid(record, msg = "")
record.valid?
msg = message(msg) do
"Expected #{mu_pp(record)} to be valid, " \
@@ -77,5 +76,4 @@ def assert_other_attrs_have_no_errors(record, *invalid_attrs)
assert invalid_attrs.include?(error.attribute), msg
end
end
-
end
diff --git a/test/templates/Gemfile.append b/test/templates/Gemfile.append
index f98afbfc..d42476d9 100644
--- a/test/templates/Gemfile.append
+++ b/test/templates/Gemfile.append
@@ -1,16 +1,14 @@
-gem 'dry_crud', path: '../..'
+gem "dry_crud", path: "../.."
-gem 'simplecov', require: false, group: :test
-
-gem 'sqlite3', '~> 1.4', platforms: :ruby
+gem "simplecov", require: false, group: :test
platforms :jruby do
- gem 'jdbc-sqlite3'
- gem 'activerecord-jdbcsqlite3-adapter'
+ gem "jdbc-sqlite3"
+ gem "activerecord-jdbcsqlite3-adapter"
end
-gem 'haml'
+gem "haml"
-gem 'kaminari'
+gem "kaminari"
-gem 'rspec-rails'
+gem "rspec-rails"
diff --git a/test/templates/app/controllers/admin/cities_controller.rb b/test/templates/app/controllers/admin/cities_controller.rb
index 49e2dcfc..aedad64f 100644
--- a/test/templates/app/controllers/admin/cities_controller.rb
+++ b/test/templates/app/controllers/admin/cities_controller.rb
@@ -3,9 +3,9 @@ module Admin
class CitiesController < TurboController
self.nesting = :admin, Country
- self.search_columns = :name, 'countries.name'
+ self.search_columns = :name, "countries.name"
- self.default_sort = 'countries.code, cities.name'
+ self.default_sort = "countries.code, cities.name"
self.permitted_attrs = %i[name person_ids]
diff --git a/test/templates/app/controllers/admin/countries_controller.rb b/test/templates/app/controllers/admin/countries_controller.rb
index 902c7036..8ff382a4 100644
--- a/test/templates/app/controllers/admin/countries_controller.rb
+++ b/test/templates/app/controllers/admin/countries_controller.rb
@@ -5,7 +5,7 @@ class CountriesController < TurboController
self.search_columns = :name, :code
- self.default_sort = 'countries.name'
+ self.default_sort = "countries.name"
self.permitted_attrs = %i[name code]
diff --git a/test/templates/app/controllers/people_controller.rb b/test/templates/app/controllers/people_controller.rb
index 902e1705..c127b346 100644
--- a/test/templates/app/controllers/people_controller.rb
+++ b/test/templates/app/controllers/people_controller.rb
@@ -1,10 +1,10 @@
# People Controller
class PeopleController < TurboController
- self.search_columns = [:name, :email, :remarks, 'cities.name']
+ self.search_columns = [ :name, :email, :remarks, "cities.name" ]
- self.default_sort = 'people.name, countries.code, cities.name'
+ self.default_sort = "people.name, countries.code, cities.name"
- self.sort_mappings = { city_id: 'cities.name' }
+ self.sort_mappings = { city_id: "cities.name" }
self.permitted_attrs = %i[name children city_id rating income
birthdate gets_up_at last_seen remarks
diff --git a/test/templates/app/controllers/vips_controller.rb b/test/templates/app/controllers/vips_controller.rb
index 56733f9c..b1438257 100644
--- a/test/templates/app/controllers/vips_controller.rb
+++ b/test/templates/app/controllers/vips_controller.rb
@@ -1,10 +1,10 @@
# List Controller for VIP people
class VipsController < ListController
- self.search_columns = [:name, :children, :rating, :remarks, 'cities.name']
+ self.search_columns = [ :name, :children, :rating, :remarks, "cities.name" ]
- self.sort_mappings = { city_id: 'cities.name' }
+ self.sort_mappings = { city_id: "cities.name" }
- self.default_sort = 'people.name, countries.code, cities.name'
+ self.default_sort = "people.name, countries.code, cities.name"
private
@@ -15,7 +15,7 @@ def model_class
end
def list_entries
- list = super.where('rating > 5').includes(city: :country)
+ list = super.where("rating > 5").includes(city: :country)
if list.respond_to?(:references)
list = list.references(:cities, :countries)
end
diff --git a/test/templates/app/helpers/people_helper.rb b/test/templates/app/helpers/people_helper.rb
index da6c00ca..6e50063e 100644
--- a/test/templates/app/helpers/people_helper.rb
+++ b/test/templates/app/helpers/people_helper.rb
@@ -7,8 +7,8 @@ def format_person_income(person)
def f(value)
case value
- when true then 'iu'
- else super(value)
+ when true then "iu"
+ else super
end
end
end
diff --git a/test/templates/app/models/city.rb b/test/templates/app/models/city.rb
index f8e2b0aa..315f14d9 100644
--- a/test/templates/app/models/city.rb
+++ b/test/templates/app/models/city.rb
@@ -7,7 +7,7 @@ class City < ApplicationRecord
before_destroy :protect_with_inhabitants
- scope :options_list, -> { includes(:country).order('cities.name') }
+ scope :options_list, -> { includes(:country).order("cities.name") }
def to_s
"#{name} (#{country.code})"
diff --git a/test/templates/app/models/person.rb b/test/templates/app/models/person.rb
index 796db013..ec5734b3 100644
--- a/test/templates/app/models/person.rb
+++ b/test/templates/app/models/person.rb
@@ -4,7 +4,7 @@ class Person < ApplicationRecord
validates :name, presence: true
- scope :list, -> { order('people.name') }
+ scope :list, -> { order("people.name") }
def to_s
name
diff --git a/test/templates/config/initializers/deprecations.rb b/test/templates/config/initializers/deprecations.rb
index 26d055d7..b665119d 100644
--- a/test/templates/config/initializers/deprecations.rb
+++ b/test/templates/config/initializers/deprecations.rb
@@ -6,4 +6,4 @@
# config.active_record.time_zone_aware_types = [:datetime]
# To silence this deprecation warning, add the following:
-Rails.application.config.active_record.time_zone_aware_types = [:datetime, :time]
+Rails.application.config.active_record.time_zone_aware_types = [ :datetime, :time ]
diff --git a/test/templates/config/routes.rb b/test/templates/config/routes.rb
index 0dbc03bd..2b285985 100644
--- a/test/templates/config/routes.rb
+++ b/test/templates/config/routes.rb
@@ -1,14 +1,13 @@
# encoding: UTF-8
TestApp::Application.routes.draw do
-
resources :people do
collection do
get :turbo
end
end
- get 'vips' => 'vips#index', :as => :vips
+ get "vips" => "vips#index", :as => :vips
namespace :admin do
resources :countries do
@@ -24,7 +23,7 @@
end
end
- root to: 'people#index'
+ root to: "people#index"
# Install the default routes as the lowest priority.
# Note: These default routes make all actions in every controller accessible
diff --git a/test/templates/db/seeds.rb b/test/templates/db/seeds.rb
index 80795a21..f12fd656 100644
--- a/test/templates/db/seeds.rb
+++ b/test/templates/db/seeds.rb
@@ -2,75 +2,75 @@
if City.count == 0
-ch = Country.create!(name: 'Switzerland', code: 'CH')
-de = Country.create!(name: 'Germany', code: 'DE')
-usa = Country.create!(name: 'USA', code: 'USA')
-gb = Country.create!(name: 'England', code: 'GB')
-jp = Country.create!(name: 'Japan', code: 'JP')
+ch = Country.create!(name: "Switzerland", code: "CH")
+de = Country.create!(name: "Germany", code: "DE")
+usa = Country.create!(name: "USA", code: "USA")
+gb = Country.create!(name: "England", code: "GB")
+jp = Country.create!(name: "Japan", code: "JP")
-be = City.create!(name: 'Bern', country: ch)
-ny = City.create!(name: 'New York', country: usa)
-sf = City.create!(name: 'San Francisco', country: usa)
-lon = City.create!(name: 'London', country: gb)
-br = City.create!(name: 'Berlin', country: de)
+be = City.create!(name: "Bern", country: ch)
+ny = City.create!(name: "New York", country: usa)
+sf = City.create!(name: "San Francisco", country: usa)
+lon = City.create!(name: "London", country: gb)
+br = City.create!(name: "Berlin", country: de)
-Person.create!(name: 'Albert Einstein',
+Person.create!(name: "Albert Einstein",
city: be,
children: 2,
rating: 9.8,
income: 84_000,
- birthdate: '1904-10-18',
- gets_up_at: '05:43',
+ birthdate: "1904-10-18",
+ gets_up_at: "05:43",
remarks: "Great physician\n Good cyclist")
-Person.create!(name: 'Adolf Ogi',
+Person.create!(name: "Adolf Ogi",
city: be,
children: 3,
rating: 4.2,
income: 264_000,
- birthdate: '1938-01-22',
- gets_up_at: '04:30',
- remarks: 'Freude herrscht!')
-Person.create!(name: 'Jay Z',
+ birthdate: "1938-01-22",
+ gets_up_at: "04:30",
+ remarks: "Freude herrscht!")
+Person.create!(name: "Jay Z",
city: ny,
children: 0,
rating: 7.2,
income: 868_345,
- birthdate: '1976-05-02',
- gets_up_at: '12:00',
- last_seen: '2011-03-10 17:29',
+ birthdate: "1976-05-02",
+ gets_up_at: "12:00",
+ last_seen: "2011-03-10 17:29",
cool: true,
remarks: "I got 99 problems\nbut you *** ain't one\nTschie")
-Person.create!(name: 'Queen Elisabeth',
+Person.create!(name: "Queen Elisabeth",
city: lon,
children: 1,
rating: 1.56,
income: 345_622,
- birthdate: '1927-08-11',
- gets_up_at: '17:12',
- remarks: '')
-Person.create!(name: 'Schopenhauer',
+ birthdate: "1927-08-11",
+ gets_up_at: "17:12",
+ remarks: "")
+Person.create!(name: "Schopenhauer",
city: br,
children: 7,
rating: 6.9,
income: 14_000,
- birthdate: '1788-10-18',
- last_seen: '1854-09-01 11:01',
- remarks: 'Neminem laede, immo omnes, quantum potes, iuva.')
-Person.create!(name: 'ZZ Top',
+ birthdate: "1788-10-18",
+ last_seen: "1854-09-01 11:01",
+ remarks: "Neminem laede, immo omnes, quantum potes, iuva.")
+Person.create!(name: "ZZ Top",
city: ny,
children: 185,
rating: 1.8,
income: 84_000,
- birthdate: '1948-03-18',
+ birthdate: "1948-03-18",
cool: true,
- remarks: 'zzz..')
-Person.create!(name: 'Andy Warhol',
+ remarks: "zzz..")
+Person.create!(name: "Andy Warhol",
city: ny,
children: 0,
rating: 7.5,
income: 123_000,
- birthdate: '1938-09-08',
- last_seen: '1984-10-10 23:39',
- remarks: 'Tomato Soup')
+ birthdate: "1938-09-08",
+ last_seen: "1984-10-10 23:39",
+ remarks: "Tomato Soup")
-end
\ No newline at end of file
+end
diff --git a/test/templates/spec/.rubocop.yml b/test/templates/spec/.rubocop.yml
deleted file mode 100644
index 23e2d80d..00000000
--- a/test/templates/spec/.rubocop.yml
+++ /dev/null
@@ -1,57 +0,0 @@
-require:
- - rubocop-rails
-
-AllCops:
- DisplayCopNames: true
- NewCops: enable
- Exclude:
- # File generated by Rails
- - spec_helper.rb
-
-Metrics/ModuleLength:
- Max: 150
-
-Metrics/MethodLength:
- Max: 25
-
-Metrics/BlockLength:
- Enabled: false
-
-Metrics/AbcSize:
- Max: 20
-
-Layout/EmptyLinesAroundClassBody:
- Enabled: false
-
-Layout/EmptyLinesAroundModuleBody:
- Enabled: false
-
-Layout/EmptyLinesAroundBlockBody:
- Enabled: false
-
-Layout/ParameterAlignment:
- Enabled: false
-
-Layout/SpaceBeforeFirstArg:
- Enabled: false
-
-Lint/Void:
- Enabled: false
-
-Rails/OutputSafety:
- Enabled: false
-
-Style/ClassVars:
- Enabled: false
-
-Style/GuardClause:
- Enabled: false
-
-Style/RegexpLiteral:
- Enabled: false
-
-Style/FrozenStringLiteralComment:
- Enabled: false
-
-Style/SafeNavigation:
- Enabled: false
diff --git a/test/templates/spec/controllers/admin/cities_controller_spec.rb b/test/templates/spec/controllers/admin/cities_controller_spec.rb
index ec177292..53d2d799 100644
--- a/test/templates/spec/controllers/admin/cities_controller_spec.rb
+++ b/test/templates/spec/controllers/admin/cities_controller_spec.rb
@@ -1,52 +1,52 @@
-require 'rails_helper'
+require "rails_helper"
describe Admin::CitiesController do
fixtures :all
- include_examples 'crud controller', {}
+ include_examples "crud controller", {}
let(:test_entry) { cities(:rj) }
- let(:test_entry_attrs) { { name: 'Rejkiavik' } }
+ let(:test_entry_attrs) { { name: "Rejkiavik" } }
alias_method :new_entry_attrs, :test_entry_attrs
alias_method :edit_entry_attrs, :test_entry_attrs
- it 'loads fixtures' do
+ it "loads fixtures" do
expect(City.count).to eq(3)
end
describe_action :get, :index do
- it 'is ordered by default scope' do
+ it "is ordered by default scope" do
expected = test_entry.country.cities.includes(:country)
- .order('countries.code, cities.name')
+ .order("countries.code, cities.name")
expected = expected.references(:countries) if expected.respond_to?(:references)
entries == expected
end
- it 'sets parents' do
- expect(controller.send(:parents)).to eq([:admin, test_entry.country])
+ it "sets parents" do
+ expect(controller.send(:parents)).to eq([ :admin, test_entry.country ])
end
- it 'sets parent variable' do
+ it "sets parent variable" do
expect(ivar(:country)).to eq(test_entry.country)
end
- it 'uses correct model_scope' do
+ it "uses correct model_scope" do
expect(controller.send(:model_scope)).to eq(test_entry.country.cities)
end
- it 'has correct path args' do
+ it "has correct path args" do
expect(controller.send(:path_args, 2)).to eq(
- [:admin, test_entry.country, 2]
+ [ :admin, test_entry.country, 2 ]
)
end
end
describe_action :get, :show, id: true do
- it 'sets parents' do
- expect(controller.send(:parents)).to eq([:admin, test_entry.country])
+ it "sets parents" do
+ expect(controller.send(:parents)).to eq([ :admin, test_entry.country ])
end
- it 'sets parent variable' do
+ it "sets parent variable" do
expect(ivar(:country)).to eq(test_entry.country)
end
end
@@ -54,21 +54,21 @@
describe_action :post, :create do
let(:params) { { model_identifier => new_entry_attrs } }
- it 'sets parent' do
+ it "sets parent" do
expect(entry.country).to eq(test_entry.country)
end
end
describe_action :delete, :destroy, id: true do
- context 'with inhabitants' do
+ context "with inhabitants" do
let(:test_entry) { cities(:ny) }
- it 'does not remove city from database', perform_request: false do
+ it "does not remove city from database", perform_request: false do
expect { perform_request }.to change { City.count }.by(0)
end
- it 'redirects to referer', perform_request: false do
- ref = @request.env['HTTP_REFERER'] =
+ it "redirects to referer", perform_request: false do
+ ref = @request.env["HTTP_REFERER"] =
admin_country_city_url(test_entry.country, test_entry)
perform_request
is_expected.to redirect_to(ref)
diff --git a/test/templates/spec/controllers/admin/countries_controller_spec.rb b/test/templates/spec/controllers/admin/countries_controller_spec.rb
index d3f48281..ac073b35 100644
--- a/test/templates/spec/controllers/admin/countries_controller_spec.rb
+++ b/test/templates/spec/controllers/admin/countries_controller_spec.rb
@@ -1,41 +1,41 @@
-require 'rails_helper'
+require "rails_helper"
describe Admin::CountriesController do
fixtures :all
- include_examples 'crud controller', skip: %w[show html plain]
+ include_examples "crud controller", skip: %w[show html plain]
let(:test_entry) { countries(:br) }
let(:test_entry_attrs) do
- { name: 'United States of America', code: 'US' }
+ { name: "United States of America", code: "US" }
end
alias_method :new_entry_attrs, :test_entry_attrs
alias_method :edit_entry_attrs, :test_entry_attrs
- it 'loads fixtures' do
+ it "loads fixtures" do
expect(Country.count).to eq(3)
end
describe_action :get, :index do
- it 'is ordered by default scope' do
+ it "is ordered by default scope" do
entries == Country.order(:name)
end
- it 'sets parents' do
- expect(controller.send(:parents)).to eq([:admin])
+ it "sets parents" do
+ expect(controller.send(:parents)).to eq([ :admin ])
end
- it 'sets nil parent' do
+ it "sets nil parent" do
expect(controller.send(:parent)).to be_nil
end
- it 'uses correct model_scope' do
+ it "uses correct model_scope" do
expect(controller.send(:model_scope)).to eq(Country.all)
end
- it 'has correct path args' do
- expect(controller.send(:path_args, 2)).to eq([:admin, 2])
+ it "has correct path args" do
+ expect(controller.send(:path_args, 2)).to eq([ :admin, 2 ])
end
end
diff --git a/test/templates/spec/controllers/people_controller_spec.rb b/test/templates/spec/controllers/people_controller_spec.rb
index 99943ecf..149d50cb 100644
--- a/test/templates/spec/controllers/people_controller_spec.rb
+++ b/test/templates/spec/controllers/people_controller_spec.rb
@@ -1,15 +1,15 @@
-require 'rails_helper'
+require "rails_helper"
describe PeopleController do
fixtures :all
render_views
- include_examples 'crud controller', {}
+ include_examples "crud controller", {}
let(:test_entry) { people(:john) }
let(:test_entry_attrs) do
- { name: 'Fischers Fritz',
+ { name: "Fischers Fritz",
children: 2,
income: 120,
city_id: cities(:rj).id }
@@ -18,60 +18,60 @@
alias_method :new_entry_attrs, :test_entry_attrs
alias_method :edit_entry_attrs, :test_entry_attrs
- it 'loads fixtures' do
+ it "loads fixtures" do
expect(Person.count).to eq(2)
end
describe_action :get, :index do
- it 'is ordered by default scope' do
+ it "is ordered by default scope" do
expected = Person.includes(city: :country)
- .order('people.name, countries.code, cities.name')
+ .order("people.name, countries.code, cities.name")
expected = expected.references(:cities, :countries) if expected.respond_to?(:references)
entries == expected
end
- it 'sets parents' do
+ it "sets parents" do
expect(controller.send(:parents)).to eq([])
end
- it 'sets nil parent' do
+ it "sets nil parent" do
expect(controller.send(:parent)).to be_nil
end
- it 'uses correct model_scope' do
+ it "uses correct model_scope" do
expect(controller.send(:model_scope)).to eq(Person.all)
end
- it 'has correct path args' do
- expect(controller.send(:path_args, 2)).to eq([2])
+ it "has correct path args" do
+ expect(controller.send(:path_args, 2)).to eq([ 2 ])
end
end
describe_action :get, :show, id: true do
- context 'turbo', format: :turbo_stream do
+ context "turbo", format: :turbo_stream do
it_is_expected_to_respond
it { expect(response.body).to match(/<turbo-stream action="update" target="content">/) }
end
end
describe_action :get, :edit, id: true do
- context 'turbo', format: :turbo_stream do
+ context "turbo", format: :turbo_stream do
it_is_expected_to_respond
it { expect(response.body).to match(/<turbo-stream action="update" target="content">/) }
end
end
describe_action :put, :update, id: true do
- context 'turbo', format: :turbo_stream do
- context 'with valid params' do
- let(:params) { { person: { name: 'New Name' } } }
+ context "turbo", format: :turbo_stream do
+ context "with valid params" do
+ let(:params) { { person: { name: "New Name" } } }
it_is_expected_to_respond
it { expect(response.body).to match(/<turbo-stream action="update" target="content">/) }
end
- context 'with invalid params' do
- let(:params) { { person: { name: ' ' } } }
+ context "with invalid params" do
+ let(:params) { { person: { name: " " } } }
it_is_expected_to_respond
it { expect(response.body).to match(/<turbo-stream action="update" target="content">/) }
diff --git a/test/templates/spec/routing/cities_routing_spec.rb b/test/templates/spec/routing/cities_routing_spec.rb
index a18e5ce5..400049c6 100644
--- a/test/templates/spec/routing/cities_routing_spec.rb
+++ b/test/templates/spec/routing/cities_routing_spec.rb
@@ -1,15 +1,15 @@
-require 'rails_helper'
+require "rails_helper"
describe Admin::CitiesController do
- it 'routes index' do
- expect(get: 'admin/countries/1/cities').to route_to(
- controller: 'admin/cities', action: 'index', country_id: '1'
+ it "routes index" do
+ expect(get: "admin/countries/1/cities").to route_to(
+ controller: "admin/cities", action: "index", country_id: "1"
)
end
- it 'routes show' do
- expect(get: 'admin/countries/2/cities/1').to route_to(
- controller: 'admin/cities', action: 'show', country_id: '2', id: '1'
+ it "routes show" do
+ expect(get: "admin/countries/2/cities/1").to route_to(
+ controller: "admin/cities", action: "show", country_id: "2", id: "1"
)
end
end
diff --git a/test/templates/spec/routing/countries_routing_spec.rb b/test/templates/spec/routing/countries_routing_spec.rb
index 67e45d90..86d0fda4 100644
--- a/test/templates/spec/routing/countries_routing_spec.rb
+++ b/test/templates/spec/routing/countries_routing_spec.rb
@@ -1,15 +1,15 @@
-require 'rails_helper'
+require "rails_helper"
describe Admin::CountriesController do
- it 'routes index' do
- expect(get: 'admin/countries').to route_to(
- controller: 'admin/countries', action: 'index'
+ it "routes index" do
+ expect(get: "admin/countries").to route_to(
+ controller: "admin/countries", action: "index"
)
end
- it 'routes show' do
- expect(get: 'admin/countries/1').to route_to(
- controller: 'admin/countries', action: 'show', id: '1'
+ it "routes show" do
+ expect(get: "admin/countries/1").to route_to(
+ controller: "admin/countries", action: "show", id: "1"
)
end
end
diff --git a/test/templates/test/.rubocop.yml b/test/templates/test/.rubocop.yml
deleted file mode 100644
index 782caf9a..00000000
--- a/test/templates/test/.rubocop.yml
+++ /dev/null
@@ -1,56 +0,0 @@
-require:
- - rubocop-rails
-
-AllCops:
- DisplayCopNames: true
- NewCops: enable
- Exclude:
- # File generated by Rails
- - test_helper.rb
-
-Metrics/BlockLength:
- Enabled: false
-
-Metrics/ClassLength:
- Max: 300
-
-Metrics/ModuleLength:
- Max: 150
-
-Metrics/MethodLength:
- Max: 25
-
-Metrics/AbcSize:
- Max: 20
- Exclude:
- - '**/*_test.rb'
-
-Layout/EmptyLinesAroundClassBody:
- Enabled: false
-
-Layout/EmptyLinesAroundModuleBody:
- Enabled: false
-
-Layout/ParameterAlignment:
- Enabled: false
-
-Layout/SpaceBeforeFirstArg:
- Enabled: false
-
-Rails/ActionControllerTestCase:
- Enabled: false
-
-Rails/OutputSafety:
- Enabled: false
-
-Style/ClassVars:
- Enabled: false
-
-Style/GuardClause:
- Enabled: false
-
-Style/RegexpLiteral:
- Enabled: false
-
-Style/FrozenStringLiteralComment:
- Enabled: false
diff --git a/test/templates/test/controllers/admin/cities_controller_test.rb b/test/templates/test/controllers/admin/cities_controller_test.rb
index e82f6359..f1068225 100644
--- a/test/templates/test/controllers/admin/cities_controller_test.rb
+++ b/test/templates/test/controllers/admin/cities_controller_test.rb
@@ -1,5 +1,5 @@
-require 'test_helper'
-require 'support/crud_controller_test_helper'
+require "test_helper"
+require "support/crud_controller_test_helper"
module Admin
# Cities Controller Test
@@ -8,35 +8,35 @@ class CitiesControllerTest < ActionController::TestCase
def test_setup
assert_equal 3, City.count
- assert_recognizes({ controller: 'admin/cities',
- action: 'index',
- country_id: '1' },
- 'admin/countries/1/cities')
- assert_recognizes({ controller: 'admin/cities',
- action: 'show',
- country_id: '2',
- id: '1' },
- 'admin/countries/2/cities/1')
+ assert_recognizes({ controller: "admin/cities",
+ action: "index",
+ country_id: "1" },
+ "admin/countries/1/cities")
+ assert_recognizes({ controller: "admin/cities",
+ action: "show",
+ country_id: "2",
+ id: "1" },
+ "admin/countries/2/cities/1")
end
def test_index
super
expected = test_entry.country.cities.includes(:country)
- .order('countries.code, cities.name')
+ .order("countries.code, cities.name")
expected = expected.references(:countries) if expected.respond_to?(:references)
assert_equal expected.to_a, entries.to_a
- assert_equal [:admin, test_entry.country], @controller.send(:parents)
+ assert_equal [ :admin, test_entry.country ], @controller.send(:parents)
assert_equal test_entry.country, @controller.send(:parent)
assert_equal test_entry.country.cities.to_a,
@controller.send(:model_scope).to_a
- assert_equal [:admin, test_entry.country, 2],
+ assert_equal [ :admin, test_entry.country, 2 ],
@controller.send(:path_args, 2)
end
def test_show
super
- assert_equal [:admin, test_entry.country], @controller.send(:parents)
+ assert_equal [ :admin, test_entry.country ], @controller.send(:parents)
assert_equal test_entry.country, @controller.send(:parent)
end
@@ -47,11 +47,11 @@ def test_create
def test_destroy_with_inhabitants
ny = cities(:ny)
- assert_no_difference('City.count') do
- @request.env['HTTP_REFERER'] = admin_country_city_url(ny.country, ny)
+ assert_no_difference("City.count") do
+ @request.env["HTTP_REFERER"] = admin_country_city_url(ny.country, ny)
delete :destroy, params: { country_id: ny.country_id, id: ny.id }
end
- assert_redirected_to [:admin, ny.country, ny]
+ assert_redirected_to [ :admin, ny.country, ny ]
assert flash[:alert].present?
end
@@ -62,7 +62,7 @@ def test_entry
end
def test_entry_attrs
- { name: 'Rejkiavik' }
+ { name: "Rejkiavik" }
end
end
end
diff --git a/test/templates/test/controllers/admin/countries_controller_test.rb b/test/templates/test/controllers/admin/countries_controller_test.rb
index 42ebb80f..f4c21459 100644
--- a/test/templates/test/controllers/admin/countries_controller_test.rb
+++ b/test/templates/test/controllers/admin/countries_controller_test.rb
@@ -1,5 +1,5 @@
-require 'test_helper'
-require 'support/crud_controller_test_helper'
+require "test_helper"
+require "support/crud_controller_test_helper"
module Admin
# Countries Controller Test
@@ -8,22 +8,22 @@ class CountriesControllerTest < ActionController::TestCase
def test_setup
assert_equal 3, Country.count
- assert_recognizes({ controller: 'admin/countries',
- action: 'index' },
- 'admin/countries')
- assert_recognizes({ controller: 'admin/countries',
- action: 'show',
- id: '1' },
- 'admin/countries/1')
+ assert_recognizes({ controller: "admin/countries",
+ action: "index" },
+ "admin/countries")
+ assert_recognizes({ controller: "admin/countries",
+ action: "show",
+ id: "1" },
+ "admin/countries/1")
end
def test_index
super
- assert_equal Country.order('name').to_a, entries
- assert_equal [:admin], @controller.send(:parents)
+ assert_equal Country.order("name").to_a, entries
+ assert_equal [ :admin ], @controller.send(:parents)
assert_nil @controller.send(:parent)
assert_equal Country.all, @controller.send(:model_scope)
- assert_equal [:admin, 2], @controller.send(:path_args, 2)
+ assert_equal [ :admin, 2 ], @controller.send(:path_args, 2)
end
def test_show
@@ -38,7 +38,7 @@ def test_entry
end
def test_entry_attrs
- { name: 'United States of America', code: 'US' }
+ { name: "United States of America", code: "US" }
end
end
end
diff --git a/test/templates/test/controllers/people_controller_test.rb b/test/templates/test/controllers/people_controller_test.rb
index 70b12d47..e2d28c5c 100644
--- a/test/templates/test/controllers/people_controller_test.rb
+++ b/test/templates/test/controllers/people_controller_test.rb
@@ -1,5 +1,5 @@
-require 'test_helper'
-require 'support/crud_controller_test_helper'
+require "test_helper"
+require "support/crud_controller_test_helper"
# People Controller Test
class PeopleControllerTest < ActionController::TestCase
@@ -7,27 +7,27 @@ class PeopleControllerTest < ActionController::TestCase
def test_setup
assert_equal 2, Person.count
- assert_recognizes({ controller: 'people',
- action: 'index' },
- '/people')
- assert_recognizes({ controller: 'people',
- action: 'show',
- id: '1' },
- '/people/1')
+ assert_recognizes({ controller: "people",
+ action: "index" },
+ "/people")
+ assert_recognizes({ controller: "people",
+ action: "show",
+ id: "1" },
+ "/people/1")
end
def test_index
super
assert_equal 2, entries.size
expected = Person.includes(city: :country)
- .order('people.name, countries.code, cities.name')
+ .order("people.name, countries.code, cities.name")
expected = expected.references(:cities, :countries) if expected.respond_to?(:references)
assert_equal expected.to_a, entries
assert_equal [], @controller.send(:parents)
assert_nil @controller.send(:parent)
assert_equal Person.all, @controller.send(:model_scope)
- assert_equal [2], @controller.send(:path_args, 2)
+ assert_equal [ 2 ], @controller.send(:path_args, 2)
end
def test_index_search
@@ -51,7 +51,7 @@ def test_update_turbo
put :update,
as: :turbo_stream,
params: { id: test_entry.id,
- person: { name: 'New Name' } }
+ person: { name: "New Name" } }
assert_response :success
assert_match(/<turbo-stream action="update" target="content">/, response.body)
end
@@ -60,7 +60,7 @@ def test_update_fail_turbo
put :update,
as: :turbo_stream,
params: { id: test_entry.id,
- person: { name: ' ' } }
+ person: { name: " " } }
assert_response :success
assert_match(/alert/, response.body)
end
@@ -72,7 +72,7 @@ def test_entry
end
def test_entry_attrs
- { name: 'Fischers Fritz',
+ { name: "Fischers Fritz",
children: 2,
income: 120,
city_id: cities(:rj).id }