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

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

diff --git a/test/models/user_preference_test.rb b/test/models/user_preference_test.rb
index 6f09d21c44f52b73a1e6408f1b6b0e0bd2ab008a..d6f1dcce5698acaed266f2c3da979e2dc0179c44 100644 (file)
--- a/test/models/user_preference_test.rb
+++ b/test/models/user_preference_test.rb
@@ -1,8 +1,6 @@
 require "test_helper"
 
 class UserPreferenceTest < ActiveSupport::TestCase
 require "test_helper"
 
 class UserPreferenceTest < ActiveSupport::TestCase

-  api_fixtures
-

   # Checks that you cannot add a new preference, that is a duplicate
   def test_add_duplicate_preference
     up = create(:user_preference)
   # Checks that you cannot add a new preference, that is a duplicate
   def test_add_duplicate_preference
     up = create(:user_preference)


@@ -17,9 +15,9 @@ class UserPreferenceTest < ActiveSupport::TestCase
   def test_check_valid_length
     key = "k"
     val = "v"
   def test_check_valid_length
     key = "k"
     val = "v"

-    (1..255).each do |i|
+    [1, 255].each do |i|

       up = UserPreference.new
       up = UserPreference.new

-      up.user = users(:normal_user)
+      up.user = create(:user)

       up.k = key * i
       up.v = val * i
       assert up.valid?
       up.k = key * i
       up.v = val * i
       assert up.valid?


@@ -35,10 +33,10 @@ class UserPreferenceTest < ActiveSupport::TestCase
     val = "v"
     [0, 256].each do |i|
       up = UserPreference.new
     val = "v"
     [0, 256].each do |i|
       up = UserPreference.new

-      up.user = users(:normal_user)
+      up.user = create(:user)

       up.k = key * i
       up.v = val * i
       up.k = key * i
       up.v = val * i

-      assert_equal false, up.valid?
+      assert_not up.valid?

       assert_raise(ActiveRecord::RecordInvalid) { up.save! }
     end
   end
       assert_raise(ActiveRecord::RecordInvalid) { up.save! }
     end
   end