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)
-                 ms = members.size
+                 members = Member.where(:user_id => 2)
+                 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