jro_apps/redmine Commit - r12296:907663f83c3b

Rails4: replace deprecated find_all_by_* at UserTest...

Toshi MARUYAMA -

r12296:907663f83c3b

parent child

Context file:

r12296:907663f83c3b

test/unit/user_test.rb +5 -5

               end
               def test_destroy_should_delete_members_and_roles
-                members = Member.find_all_by_user_id(2)
+                members = Member.where(:user_id => 2)
-                ms = members.size
+                ms = members.count
                 rs = members.collect(&:roles).flatten.size
+                assert ms > 0
+                assert rs > 0
                 assert_difference 'Member.count', - ms do
                   assert_difference 'MemberRole.count', - rs do
                     User.find(2).destroy
                   end
                 end
                 assert_nil User.find_by_id(2)
-                assert Member.find_all_by_user_id(2).empty?
+                assert_equal 0, Member.where(:user_id => 2).count
               end
               def test_destroy_should_update_attachments

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages