Commit 1e9603c0c967ccbe3ea534e394052c4e2c291c66
1 parent
5c97f086
Exists in
master
and in
29 other branches
person: fix tests
(ActionItem3039)
Showing
1 changed file
with
3 additions
and
4 deletions
Show diff stats
test/unit/person_test.rb
@@ -404,13 +404,12 @@ class PersonTest < ActiveSupport::TestCase | @@ -404,13 +404,12 @@ class PersonTest < ActiveSupport::TestCase | ||
404 | 404 | ||
405 | should 'not allow simple member to view group pending tasks' do | 405 | should 'not allow simple member to view group pending tasks' do |
406 | community = fast_create(Community) | 406 | community = fast_create(Community) |
407 | + admin = fast_create(Person) | ||
408 | + community.add_member(admin) | ||
407 | member = fast_create(Person) | 409 | member = fast_create(Person) |
408 | community.add_member(member) | 410 | community.add_member(member) |
409 | - | ||
410 | community.tasks << Task.new | 411 | community.tasks << Task.new |
411 | 412 | ||
412 | - person = fast_create(Person) | ||
413 | - community.add_member(person) | ||
414 | 413 | ||
415 | assert_not_includes Person.with_pending_tasks, person | 414 | assert_not_includes Person.with_pending_tasks, person |
416 | end | 415 | end |
@@ -1367,7 +1366,7 @@ class PersonTest < ActiveSupport::TestCase | @@ -1367,7 +1366,7 @@ class PersonTest < ActiveSupport::TestCase | ||
1367 | u = create_user('user'+i.to_s) | 1366 | u = create_user('user'+i.to_s) |
1368 | u.activate | 1367 | u.activate |
1369 | } | 1368 | } |
1370 | - assert_equal deactivated, Person.deactivated | 1369 | + assert_equivalent deactivated, Person.deactivated |
1371 | end | 1370 | end |
1372 | 1371 | ||
1373 | should 'be able to retrieve memberships by role person has' do | 1372 | should 'be able to retrieve memberships by role person has' do |