diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2018-11-08 10:42:01 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2018-11-08 10:42:01 +0900 |
commit | 6ee0d8f75dd9608e305491b54e6962f4ccba60a5 (patch) | |
tree | 9d352e74091acd7a16d78276b98ce0e276f78887 /activerecord/test | |
parent | 5df4efd2fd08dfdf8583bcb6a4322aa5997c18e1 (diff) | |
parent | 25f1cbdecd4f506b46a6c476f18074dafdb581f2 (diff) | |
download | rails-6ee0d8f75dd9608e305491b54e6962f4ccba60a5.tar.gz rails-6ee0d8f75dd9608e305491b54e6962f4ccba60a5.tar.bz2 rails-6ee0d8f75dd9608e305491b54e6962f4ccba60a5.zip |
Merge pull request #34385 from christophemaximin/guard-enum-empty-keys
Guard Enums against definitions with blank label names
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/enum_test.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/activerecord/test/cases/enum_test.rb b/activerecord/test/cases/enum_test.rb index b4593ccdf2..867ce7082b 100644 --- a/activerecord/test/cases/enum_test.rb +++ b/activerecord/test/cases/enum_test.rb @@ -274,6 +274,24 @@ class EnumTest < ActiveRecord::TestCase end assert_match(/must be either a hash, an array of symbols, or an array of strings./, e.message) + + e = assert_raises(ArgumentError) do + Class.new(ActiveRecord::Base) do + self.table_name = "books" + enum status: { "" => 1, "active" => 2 } + end + end + + assert_match(/Enum label name must not be blank/, e.message) + + e = assert_raises(ArgumentError) do + Class.new(ActiveRecord::Base) do + self.table_name = "books" + enum status: ["active", ""] + end + end + + assert_match(/Enum label name must not be blank/, e.message) end test "reserved enum names" do |