diff options
| author | Rafael França <rafaelmfranca@gmail.com> | 2019-04-15 18:11:55 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-04-15 18:11:55 -0400 |
| commit | fbb8b68403ff8663a561b20746e9da7719eeb83e (patch) | |
| tree | 37dbbe6f90573b0b1b443a228b70abcdb2b7eea1 | |
| parent | ddf471a163e96e52e1a1cc507bde5a0cfe9f643f (diff) | |
| parent | 76c1528c5fd372af080081873b1ba02582c51f25 (diff) | |
| download | rails-fbb8b68403ff8663a561b20746e9da7719eeb83e.tar.gz rails-fbb8b68403ff8663a561b20746e9da7719eeb83e.tar.bz2 rails-fbb8b68403ff8663a561b20746e9da7719eeb83e.zip | |
Merge pull request #35977 from prathamesh-sonpatki/rm-required-in-generators
Remove `required: true` from the model generator template
| -rw-r--r-- | activerecord/lib/rails/generators/active_record/model/templates/model.rb.tt | 2 | ||||
| -rw-r--r-- | railties/test/generators/model_generator_test.rb | 15 |
2 files changed, 3 insertions, 14 deletions
diff --git a/activerecord/lib/rails/generators/active_record/model/templates/model.rb.tt b/activerecord/lib/rails/generators/active_record/model/templates/model.rb.tt index c1c03e2762..77b9ea1c86 100644 --- a/activerecord/lib/rails/generators/active_record/model/templates/model.rb.tt +++ b/activerecord/lib/rails/generators/active_record/model/templates/model.rb.tt @@ -1,7 +1,7 @@ <% module_namespacing do -%> class <%= class_name %> < <%= parent_class_name.classify %> <% attributes.select(&:reference?).each do |attribute| -%> - belongs_to :<%= attribute.name %><%= ', polymorphic: true' if attribute.polymorphic? %><%= ', required: true' if attribute.required? %> + belongs_to :<%= attribute.name %><%= ', polymorphic: true' if attribute.polymorphic? %> <% end -%> <% attributes.select(&:rich_text?).each do |attribute| -%> has_rich_text :<%= attribute.name %> diff --git a/railties/test/generators/model_generator_test.rb b/railties/test/generators/model_generator_test.rb index c97cd17ec6..1b86e509ee 100644 --- a/railties/test/generators/model_generator_test.rb +++ b/railties/test/generators/model_generator_test.rb @@ -414,23 +414,12 @@ class ModelGeneratorTest < Rails::Generators::TestCase end end - def test_required_belongs_to_adds_required_association - run_generator ["account", "supplier:references{required}"] - - expected_file = <<~FILE - class Account < ApplicationRecord - belongs_to :supplier, required: true - end - FILE - assert_file "app/models/account.rb", expected_file - end - def test_required_polymorphic_belongs_to_generages_correct_model run_generator ["account", "supplier:references{required,polymorphic}"] expected_file = <<~FILE class Account < ApplicationRecord - belongs_to :supplier, polymorphic: true, required: true + belongs_to :supplier, polymorphic: true end FILE assert_file "app/models/account.rb", expected_file @@ -441,7 +430,7 @@ class ModelGeneratorTest < Rails::Generators::TestCase expected_file = <<~FILE class Account < ApplicationRecord - belongs_to :supplier, polymorphic: true, required: true + belongs_to :supplier, polymorphic: true end FILE assert_file "app/models/account.rb", expected_file |