From 4c208254573c3428d82bd8744860bd86e1c78acb Mon Sep 17 00:00:00 2001 From: Xavier Noria Date: Sat, 6 Aug 2016 18:38:23 +0200 Subject: applies new string literal convention in activemodel/test The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. --- activemodel/test/cases/errors_test.rb | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) (limited to 'activemodel/test/cases/errors_test.rb') diff --git a/activemodel/test/cases/errors_test.rb b/activemodel/test/cases/errors_test.rb index 13fa5c76bc..da99a0eca0 100644 --- a/activemodel/test/cases/errors_test.rb +++ b/activemodel/test/cases/errors_test.rb @@ -29,45 +29,45 @@ class ErrorsTest < ActiveModel::TestCase def test_delete errors = ActiveModel::Errors.new(self) - errors[:foo] << 'omg' + errors[:foo] << "omg" errors.delete(:foo) assert_empty errors[:foo] end def test_include? errors = ActiveModel::Errors.new(self) - errors[:foo] << 'omg' - assert errors.include?(:foo), 'errors should include :foo' + errors[:foo] << "omg" + assert errors.include?(:foo), "errors should include :foo" end def test_dup errors = ActiveModel::Errors.new(self) - errors[:foo] << 'bar' + errors[:foo] << "bar" errors_dup = errors.dup - errors_dup[:bar] << 'omg' + errors_dup[:bar] << "omg" assert_not_same errors_dup.messages, errors.messages end def test_has_key? errors = ActiveModel::Errors.new(self) - errors[:foo] << 'omg' - assert_equal true, errors.has_key?(:foo), 'errors should have key :foo' + errors[:foo] << "omg" + assert_equal true, errors.has_key?(:foo), "errors should have key :foo" end def test_has_no_key errors = ActiveModel::Errors.new(self) - assert_equal false, errors.has_key?(:name), 'errors should not have key :name' + assert_equal false, errors.has_key?(:name), "errors should not have key :name" end def test_key? errors = ActiveModel::Errors.new(self) - errors[:foo] << 'omg' - assert_equal true, errors.key?(:foo), 'errors should have key :foo' + errors[:foo] << "omg" + assert_equal true, errors.key?(:foo), "errors should have key :foo" end def test_no_key errors = ActiveModel::Errors.new(self) - assert_equal false, errors.key?(:name), 'errors should not have key :name' + assert_equal false, errors.key?(:name), "errors should not have key :name" end test "clear errors" do @@ -145,7 +145,7 @@ class ErrorsTest < ActiveModel::TestCase test "assign error" do person = Person.new assert_deprecated do - person.errors[:name] = 'should not be nil' + person.errors[:name] = "should not be nil" end assert_equal ["should not be nil"], person.errors[:name] end @@ -331,16 +331,16 @@ class ErrorsTest < ActiveModel::TestCase test "add_on_empty generates message with custom default message" do person = Person.new - assert_called_with(person.errors, :generate_message, [:name, :empty, { message: 'custom' }]) do + assert_called_with(person.errors, :generate_message, [:name, :empty, { message: "custom" }]) do assert_deprecated do - person.errors.add_on_empty :name, message: 'custom' + person.errors.add_on_empty :name, message: "custom" end end end test "add_on_empty generates message with empty string value" do person = Person.new - person.name = '' + person.name = "" assert_called_with(person.errors, :generate_message, [:name, :empty, {}]) do assert_deprecated do person.errors.add_on_empty :name @@ -369,9 +369,9 @@ class ErrorsTest < ActiveModel::TestCase test "add_on_blank generates message with custom default message" do person = Person.new - assert_called_with(person.errors, :generate_message, [:name, :blank, { message: 'custom' }]) do + assert_called_with(person.errors, :generate_message, [:name, :blank, { message: "custom" }]) do assert_deprecated do - person.errors.add_on_blank :name, message: 'custom' + person.errors.add_on_blank :name, message: "custom" end end end -- cgit v1.2.3