Replace friends fixture with a factory
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 5 Oct 2016 12:18:45 +0000 (13:18 +0100)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 5 Oct 2016 12:18:45 +0000 (13:18 +0100)
test/controllers/changeset_controller_test.rb
test/controllers/diary_entry_controller_test.rb
test/controllers/user_controller_test.rb
test/factories/friends.rb [new file with mode: 0644]
test/fixtures/friends.yml [deleted file]
test/models/friend_test.rb
test/models/user_test.rb

index e1960d4e8882492f9c3869bc43ed1549f67e590a..c996bbaab3c2297e8effe87b41158b58b1610721 100644 (file)
@@ -3,7 +3,7 @@ require "changeset_controller"
 
 class ChangesetControllerTest < ActionController::TestCase
   api_fixtures
-  fixtures :friends, :changeset_comments, :changesets_subscribers
+  fixtures :changeset_comments, :changesets_subscribers
 
   ##
   # test all routes which lead to this controller
index 6e07a409102a0c254bd36bb3c3c5210b62b94cf2..813b50b2f3be35fb35d2dccecabde2ca15c21fdb 100644 (file)
@@ -1,7 +1,7 @@
 require "test_helper"
 
 class DiaryEntryControllerTest < ActionController::TestCase
-  fixtures :users, :user_roles, :languages, :friends
+  fixtures :users, :user_roles, :languages
 
   include ActionView::Helpers::NumberHelper
 
@@ -419,7 +419,8 @@ class DiaryEntryControllerTest < ActionController::TestCase
   end
 
   def test_list_friends
-    diary_entry = create(:diary_entry, :user_id => friends(:normal_user_with_second_user).friend_user_id)
+    friend = create(:friend, :user_id => users(:normal_user).id)
+    diary_entry = create(:diary_entry, :user_id => friend.friend_user_id)
     _other_entry = create(:diary_entry, :user_id => users(:second_public_user).id)
 
     # Try a list of diary entries for your friends when not logged in
index ede841032ef20072514bde568a1f1220f62b228d..6a5fca78563af0077ae27c80c2dceb33032dbcbf 100644 (file)
@@ -2,7 +2,7 @@ require "test_helper"
 
 class UserControllerTest < ActionController::TestCase
   api_fixtures
-  fixtures :messages, :friends
+  fixtures :messages
 
   ##
   # test all routes which lead to this controller
@@ -1191,6 +1191,7 @@ class UserControllerTest < ActionController::TestCase
     # Get users to work with
     user = users(:normal_user)
     friend = users(:public_user)
+    create(:friend, :user_id => user.id, :friend_user_id => friend.id)
 
     # Check that the users are friends
     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
@@ -1231,6 +1232,7 @@ class UserControllerTest < ActionController::TestCase
     # Get users to work with
     user = users(:normal_user)
     friend = users(:public_user)
+    create(:friend, :user_id => user.id, :friend_user_id => friend.id)
 
     # Check that the users are friends
     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
diff --git a/test/factories/friends.rb b/test/factories/friends.rb
new file mode 100644 (file)
index 0000000..46b14a3
--- /dev/null
@@ -0,0 +1,7 @@
+FactoryGirl.define do
+  factory :friend do
+    # Fixme requires User Factory
+    user_id 1
+    friend_user_id 2
+  end
+end
diff --git a/test/fixtures/friends.yml b/test/fixtures/friends.yml
deleted file mode 100644 (file)
index 782f1e3..0000000
+++ /dev/null
@@ -1,4 +0,0 @@
-normal_user_with_second_user:
-  id: 1
-  user_id: 1
-  friend_user_id: 2
index a13f67470fd1c409f6cbabfec16011477e262934..08eeeeb17a3b3c7839245431a021469d1c7af962 100644 (file)
@@ -2,9 +2,9 @@ require "test_helper"
 
 class FriendTest < ActiveSupport::TestCase
   api_fixtures
-  fixtures :friends
 
   def test_friend_count
+    create(:friend)
     assert_equal 1, Friend.count
   end
 end
index f9d94cc17cf1fabc9e386252c445bcd0bbe53ab4..5de5db326a34033a4ddda42184456c0fb013b298 100644 (file)
@@ -5,7 +5,7 @@ class UserTest < ActiveSupport::TestCase
   include Rails::Dom::Testing::Assertions::SelectorAssertions
 
   api_fixtures
-  fixtures :friends, :languages, :user_roles
+  fixtures :languages, :user_roles
 
   def test_invalid_with_empty_attributes
     user = User.new
@@ -106,6 +106,7 @@ class UserTest < ActiveSupport::TestCase
   end
 
   def test_friend_with
+    create(:friend, :user_id => users(:normal_user).id, :friend_user_id => users(:public_user).id)
     assert users(:normal_user).is_friends_with?(users(:public_user))
     assert !users(:normal_user).is_friends_with?(users(:inactive_user))
     assert !users(:public_user).is_friends_with?(users(:normal_user))
@@ -129,14 +130,11 @@ class UserTest < ActiveSupport::TestCase
 
   def test_friends_with
     # normal user is a friend of second user
-    # it should be a one way friend accossitation
-    assert_equal 1, Friend.count
+    # it should be a one way friend associatation
     norm = users(:normal_user)
     sec = users(:public_user)
-    # friend = Friend.new
-    # friend.befriender = norm
-    # friend.befriendee = sec
-    # friend.save
+    create(:friend, :user_id => norm.id, :friend_user_id => sec.id)
+    assert_equal 1, Friend.count
     assert_equal [sec], norm.friend_users
     assert_equal 1, norm.friend_users.size
     assert_equal 1, Friend.count