Merge remote-tracking branch 'upstream/pull/5236'

[rails.git] / test / models / diary_entry_test.rb

diff --git a/test/models/diary_entry_test.rb b/test/models/diary_entry_test.rb
index 6c9a7544312aee0d4132c061c5abeea00f7d603d..eb794acec2d858495427a5d79e3404ebc87579cd 100644 (file)
--- a/test/models/diary_entry_test.rb
+++ b/test/models/diary_entry_test.rb
@@ -1,8 +1,6 @@
 require "test_helper"
 
 class DiaryEntryTest < ActiveSupport::TestCase
 require "test_helper"
 
 class DiaryEntryTest < ActiveSupport::TestCase

-  fixtures :users
-

   def setup
     # Create the default language for diary entries
     create(:language, :code => "en")
   def setup
     # Create the default language for diary entries
     create(:language, :code => "en")


@@ -10,18 +8,18 @@ class DiaryEntryTest < ActiveSupport::TestCase
 
   def test_diary_entry_validations
     diary_entry_valid({})
 
   def test_diary_entry_validations
     diary_entry_valid({})

-    diary_entry_valid({ :title => "" }, false)
-    diary_entry_valid(:title => "a" * 255)
-    diary_entry_valid({ :title => "a" * 256 }, false)
-    diary_entry_valid({ :body => "" }, false)
-    diary_entry_valid(:latitude => 90)
-    diary_entry_valid({ :latitude => 90.00001 }, false)
-    diary_entry_valid(:latitude => -90)
-    diary_entry_valid({ :latitude => -90.00001 }, false)
-    diary_entry_valid(:longitude => 180)
-    diary_entry_valid({ :longitude => 180.00001 }, false)
-    diary_entry_valid(:longitude => -180)
-    diary_entry_valid({ :longitude => -180.00001 }, false)
+    diary_entry_valid({ :title => "" }, :valid => false)
+    diary_entry_valid({ :title => "a" * 255 })
+    diary_entry_valid({ :title => "a" * 256 }, :valid => false)
+    diary_entry_valid({ :body => "" }, :valid => false)
+    diary_entry_valid({ :latitude => 90 })
+    diary_entry_valid({ :latitude => 90.00001 }, :valid => false)
+    diary_entry_valid({ :latitude => -90 })
+    diary_entry_valid({ :latitude => -90.00001 }, :valid => false)
+    diary_entry_valid({ :longitude => 180 })
+    diary_entry_valid({ :longitude => 180.00001 }, :valid => false)
+    diary_entry_valid({ :longitude => -180 })
+    diary_entry_valid({ :longitude => -180.00001 }, :valid => false)

   end
 
   def test_diary_entry_visible
   end
 
   def test_diary_entry_visible


@@ -50,8 +48,8 @@ class DiaryEntryTest < ActiveSupport::TestCase
 
   private
 
 
   private
 

-  def diary_entry_valid(attrs, result = true)
+  def diary_entry_valid(attrs, valid: true)

     entry = build(:diary_entry, attrs)
     entry = build(:diary_entry, attrs)

-    assert_equal result, entry.valid?, "Expected #{attrs.inspect} to be #{result}"
+    assert_equal valid, entry.valid?, "Expected #{attrs.inspect} to be #{valid}"

   end
 end
   end
 end