Commit 6d57ec81ebad14ad7c65bf161fccba93cd557a9d
1 parent
71c5bf6c
Exists in
master
and in
29 other branches
rails3: fixing Qualifier/Certifier
Showing
3 changed files
with
12 additions
and
14 deletions
Show diff stats
app/models/certifier.rb
app/models/qualifier.rb
test/unit/qualifier_certifier_test.rb
@@ -2,24 +2,18 @@ require File.dirname(__FILE__) + '/../test_helper' | @@ -2,24 +2,18 @@ require File.dirname(__FILE__) + '/../test_helper' | ||
2 | 2 | ||
3 | class QualifierCertifierTest < ActiveSupport::TestCase | 3 | class QualifierCertifierTest < ActiveSupport::TestCase |
4 | 4 | ||
5 | - should 'qualifier has many certifiers' do | 5 | + should 'connect certifiers and qualifiers' do |
6 | env_one = fast_create(Environment) | 6 | env_one = fast_create(Environment) |
7 | - qualifier = Qualifier.create(:name => 'Qualifier', :environment => env_one) | ||
8 | - certifier = Certifier.create(:name => 'Certifier', :environment => env_one) | 7 | + qualifier = env_one.qualifiers.create(:name => 'Qualifier') |
8 | + certifier = env_one.certifiers.create(:name => 'Certifier') | ||
9 | 9 | ||
10 | - QualifierCertifier.create(:qualifier => qualifier, :certifier => certifier) | ||
11 | - | ||
12 | - assert_includes qualifier.certifiers, certifier | ||
13 | - end | ||
14 | - | ||
15 | - should 'certifier has many qualifiers' do | ||
16 | - env_one = fast_create(Environment) | ||
17 | - qualifier = Qualifier.create(:name => 'Qualifier', :environment => env_one) | ||
18 | - certifier = Certifier.create(:name => 'Certifier', :environment => env_one) | ||
19 | - | ||
20 | - QualifierCertifier.create(:qualifier => qualifier, :certifier => certifier) | 10 | + QualifierCertifier.new.tap do |qc| |
11 | + qc.qualifier = qualifier | ||
12 | + qc.certifier = certifier | ||
13 | + end.save! | ||
21 | 14 | ||
22 | assert_includes certifier.qualifiers, qualifier | 15 | assert_includes certifier.qualifiers, qualifier |
16 | + assert_includes qualifier.certifiers, certifier | ||
23 | end | 17 | end |
24 | 18 | ||
25 | end | 19 | end |