User factory associations for all factories.
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 22 Feb 2017 18:51:45 +0000 (18:51 +0000)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 22 Feb 2017 18:51:45 +0000 (18:51 +0000)
test/controllers/user_controller_test.rb
test/factories/changeset_comments.rb
test/factories/diary_comments.rb
test/factories/diary_entries.rb
test/factories/friends.rb
test/factories/messages.rb
test/factories/traces.rb
test/factories/user_blocks.rb
test/factories/user_preferences.rb
test/models/user_preference_test.rb

index 686a860705802f4ba1aa325281243f2b6e6e18ca..87fb99a6f832b21b0e508b549a25fe72a534bd89 100644 (file)
@@ -1045,6 +1045,7 @@ class UserControllerTest < ActionController::TestCase
 
   def test_api_details
     create(:message, :read, :recipient => users(:normal_user))
+    create(:message, :sender => users(:normal_user))
 
     # check that nothing is returned when not logged in
     get :api_details
index 5fb2621843ee0925cceee004fd496f07768e9cea..d12c1b653c391f2c8d94ccd597ad38d27ae7980e 100644 (file)
@@ -6,7 +6,6 @@ FactoryGirl.define do
     # FIXME: needs changeset factory
     changeset_id 3
 
-    # FIXME: needs user factory
-    author_id 1
+    association :author, :factory => :user
   end
 end
index 810bd2f7a0b8bf84c3aea4dd18dae80b644acd26..2d54f273e6f0e48f4326df324fadd03ffe05f24e 100644 (file)
@@ -3,8 +3,6 @@ FactoryGirl.define do
     sequence(:body) { |n| "This is diary comment #{n}" }
 
     diary_entry
-
-    # Fixme requires User Factory
-    user_id 1
+    user
   end
 end
index d00a97dc5389074dcf849b657aec0d0a1e60ba11..666a37ee8e6247c5d8dea8152a9e626a606760d8 100644 (file)
@@ -3,7 +3,6 @@ FactoryGirl.define do
     sequence(:title) { |n| "Diary entry #{n}" }
     sequence(:body) { |n| "This is diary entry #{n}" }
 
-    # Fixme requires User Factory
-    user_id 1
+    user
   end
 end
index 46b14a39daea3860cd2de3cebbe80f6255b7650a..749846edbe0200f2ef087613cf0eda7905def1db 100644 (file)
@@ -1,7 +1,6 @@
 FactoryGirl.define do
   factory :friend do
-    # Fixme requires User Factory
-    user_id 1
-    friend_user_id 2
+    association :befriender, :factory => :user
+    association :befriendee, :factory => :user
   end
 end
index 75e65cdbf93c270d11dfd721e0cc571b47250fce..9d9c30a48f46c7fe3de0972c9b8875d9f151ba06 100644 (file)
@@ -4,11 +4,8 @@ FactoryGirl.define do
     sequence(:body) { |n| "Body text for message #{n}" }
     sent_on Time.now
 
-    # FIXME: needs user factory
-    from_user_id 1
-
-    # FIXME: needs user factory
-    to_user_id 2
+    association :sender, :factory => :user
+    association :recipient, :factory => :user
 
     trait :unread do
       message_read false
index 66f810468e20a1c81c8113bf1c92fa454c490044..76fd90590e6cbb782ffd9f55df0fe183b31475ea 100644 (file)
@@ -3,8 +3,7 @@ FactoryGirl.define do
     sequence(:name) { |n| "Trace #{n}.gpx" }
     sequence(:description) { |n| "This is trace #{n}" }
 
-    # Fixme requires User Factory
-    user_id 1
+    user
 
     timestamp Time.now
     inserted true
index b73e599b864757b370adda699951a4fc221a5a43..8f7edaf6e2f4444fda84ca7b1b4fb3fdd04a05d6 100644 (file)
@@ -3,11 +3,8 @@ FactoryGirl.define do
     sequence(:reason) { |n| "User Block #{n}" }
     ends_at Time.now + 1.day
 
-    # FIXME: requires User factory
-    user_id 13
-
-    # FIXME: requires User factory
-    creator_id 15
+    user
+    association :creator, :factory => :moderator_user
 
     trait :needs_view do
       needs_view true
index 552cfcd75e5c922dbc93019ded21cf9460e9e036..e6f0d6b874a917aa659c86a18980cd611b5b48c6 100644 (file)
@@ -3,7 +3,6 @@ FactoryGirl.define do
     sequence(:k) { |n| "Key #{n}" }
     sequence(:v) { |n| "Value #{n}" }
 
-    # FIXME: needs user factory
-    user_id 1
+    user
   end
 end
index c61bd607dcce10e92e5ab7c85ccb38cf22cf84d4..6f09d21c44f52b73a1e6408f1b6b0e0bd2ab008a 100644 (file)
@@ -7,7 +7,7 @@ class UserPreferenceTest < ActiveSupport::TestCase
   def test_add_duplicate_preference
     up = create(:user_preference)
     new_up = UserPreference.new
-    new_up.user = users(:normal_user)
+    new_up.user = up.user
     new_up.k = up.k
     new_up.v = "some other value"
     assert_not_equal new_up.v, up.v