Download as
Browse Code »

Commit ff2a47e2fd60873422bbe7c7ed74595fb8353332

Authored by Leandro Santos
1 parent 05a4e705
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

Rails3: Fix Qualifier unit tests

Showing 2 changed files with 6 additions and 3 deletions   Show diff stats
app/models/product_qualifier.rb
  Diff comments   View file @ff2a47e
1 class ProductQualifier < ActiveRecord::Base 1 class ProductQualifier < ActiveRecord::Base
  2 +
  3 + attr_accessible :qualifier, :product, :certifier
  4 +
2 belongs_to :qualifier 5 belongs_to :qualifier
3 belongs_to :product 6 belongs_to :product
4 belongs_to :certifier 7 belongs_to :certifier
test/unit/qualifier_test.rb
  Diff comments   View file @ff2a47e
@@ -13,10 +13,10 @@ class QualifierTest &lt; ActiveSupport::TestCase @@ -13,10 +13,10 @@ class QualifierTest &lt; ActiveSupport::TestCase
13 13
14 should 'belongs to environment' do 14 should 'belongs to environment' do
15 env_one = fast_create(Environment) 15 env_one = fast_create(Environment)
16 - qualifier_from_env_one = Qualifier.create(:name => 'Qualifier from environment one', :environment => env_one) 16 + qualifier_from_env_one = env_one.qualifiers.create(:name => 'Qualifier from environment one')
17 17
18 env_two = fast_create(Environment) 18 env_two = fast_create(Environment)
19 - qualifier_from_env_two = Qualifier.create(:name => 'Qualifier from environment two', :environment => env_two) 19 + qualifier_from_env_two = env_two.qualifiers.create(:name => 'Qualifier from environment two')
20 20
21 assert_includes env_one.qualifiers, qualifier_from_env_one 21 assert_includes env_one.qualifiers, qualifier_from_env_one
22 assert_not_includes env_one.qualifiers, qualifier_from_env_two 22 assert_not_includes env_one.qualifiers, qualifier_from_env_two
@@ -24,7 +24,7 @@ class QualifierTest &lt; ActiveSupport::TestCase @@ -24,7 +24,7 @@ class QualifierTest &lt; ActiveSupport::TestCase
24 24
25 should 'name is mandatory' do 25 should 'name is mandatory' do
26 env_one = fast_create(Environment) 26 env_one = fast_create(Environment)
27 - qualifier = Qualifier.new(:environment => env_one) 27 + qualifier = env_one.qualifiers.build
28 assert !qualifier.valid? 28 assert !qualifier.valid?
29 29
30 qualifier.name = 'Qualifier name' 30 qualifier.name = 'Qualifier name'