Commit c4b1d0eb114342c7a7fdf666d0236b33b438d7d3
1 parent
f7096d22
Exists in
master
and in
29 other branches
Removing reload and recalling object on tests
Showing
1 changed file
with
3 additions
and
6 deletions
Show diff stats
test/unit/environment_test.rb
| @@ -375,23 +375,20 @@ class EnvironmentTest < Test::Unit::TestCase | @@ -375,23 +375,20 @@ class EnvironmentTest < Test::Unit::TestCase | ||
| 375 | env = Environment.default | 375 | env = Environment.default |
| 376 | user = create_user('testuser').person | 376 | user = create_user('testuser').person |
| 377 | env.add_admin(user) | 377 | env.add_admin(user) |
| 378 | - env.reload | ||
| 379 | 378 | ||
| 380 | - assert_includes env.admins, user | 379 | + assert_includes Environment.default.admins, user |
| 381 | end | 380 | end |
| 382 | 381 | ||
| 383 | should 'be able to remove admins easily' do | 382 | should 'be able to remove admins easily' do |
| 384 | env = Environment.default | 383 | env = Environment.default |
| 385 | user = create_user('testuser').person | 384 | user = create_user('testuser').person |
| 386 | env.affiliate(user, Environment::Roles.admin(env.id)) | 385 | env.affiliate(user, Environment::Roles.admin(env.id)) |
| 387 | - env.reload | ||
| 388 | - assert_includes env.admins, user | 386 | + assert_includes Environment.default.admins, user |
| 389 | 387 | ||
| 390 | env.remove_admin(user) | 388 | env.remove_admin(user) |
| 391 | - assert_not_includes env.admins, user | 389 | + assert_not_includes Environment.default.admins, user |
| 392 | end | 390 | end |
| 393 | 391 | ||
| 394 | - | ||
| 395 | should 'have products through enterprises' do | 392 | should 'have products through enterprises' do |
| 396 | product_category = fast_create(ProductCategory, :name => 'Products', :environment_id => Environment.default.id) | 393 | product_category = fast_create(ProductCategory, :name => 'Products', :environment_id => Environment.default.id) |
| 397 | env = Environment.default | 394 | env = Environment.default |