Download as
Browse Code »

Commit 4ad3a6f3b56b0895a29e9e963bcf86036c56f942

Authored by Leandro Santos
1 parent 8c31af4c
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

refactoring person custom field tests

Showing 1 changed file with 6 additions and 7 deletions   Show diff stats
test/unit/environment_test.rb
  Diff comments   View file @4ad3a6f
... ... @@ -761,11 +761,9 @@ class EnvironmentTest < ActiveSupport::TestCase
761 761 env = Environment.new
762 762 env.custom_person_fields = {'cell_phone' => {'required' => 'true', 'active' => '', 'signup' => ''}, 'comercial_phone'=> {'required' => '', 'active' => 'true', 'signup' => '' }, 'description' => {'required' => '', 'active' => '', 'signup' => 'true'}}
763 763  
764   - assert_equal({'cell_phone' => {'required' => 'true', 'active' => 'true', 'signup' => 'true'}, 'comercial_phone'=> {'required' => '', 'active' => 'true', 'signup' => '' }, 'description' => {'required' => '', 'active' => 'true', 'signup' => 'true'}}, env.custom_person_fields)
765   - end
766   -
767   - should 'have no custom_person_fields by default' do
768   - assert_equal({}, Environment.new.custom_person_fields)
  764 + assert_equal({'required' => 'true', 'active' => 'true', 'signup' => 'true'}, env.custom_person_fields['cell_phone'])
  765 + assert_equal({'required' => '', 'active' => 'true', 'signup' => '' }, env.custom_person_fields['comercial_phone'])
  766 + assert_equal({'required' => '', 'active' => 'true', 'signup' => 'true'}, env.custom_person_fields['description'])
769 767 end
770 768  
771 769 should 'not set in custom_person_fields if not in person.fields' do
... ... @@ -773,7 +771,7 @@ class EnvironmentTest < ActiveSupport::TestCase
773 771 Person.stubs(:fields).returns(['cell_phone', 'comercial_phone'])
774 772  
775 773 env.custom_person_fields = { 'birth_date' => {'required' => 'true', 'active' => 'true'}, 'cell_phone' => {'required' => 'true', 'active' => 'true'}}
776   - assert_equal({'cell_phone' => {'required' => 'true','signup' => 'true', 'active' => 'true'}}, env.custom_person_fields)
  774 + assert_equal({'required' => 'true','signup' => 'true', 'active' => 'true'}, env.custom_person_fields['cell_phone'])
777 775 assert ! env.custom_person_fields.keys.include?('birth_date')
778 776 end
779 777  
... ... @@ -782,7 +780,8 @@ class EnvironmentTest < ActiveSupport::TestCase
782 780 Person.stubs(:fields).returns(['cell_phone', 'schooling'])
783 781  
784 782 env.custom_person_fields = { 'schooling' => {'required' => 'true', 'active' => 'true'}}
785   - assert_equal({'schooling' => {'required' => 'true', 'signup' => 'true', 'active' => 'true'}, 'schooling_status' => {'required' => 'true', 'signup' => 'true', 'active' => 'true'}}, env.custom_person_fields)
  783 + assert_equal({'required' => 'true', 'signup' => 'true', 'active' => 'true'}, env.custom_person_fields['schooling'])
  784 + assert_equal({'required' => 'true', 'signup' => 'true', 'active' => 'true'}, env.custom_person_fields['schooling_status'])
786 785 assert ! env.custom_person_fields.keys.include?('birth_date')
787 786 end
788 787  
... ...