Download as
Browse Code »

Commit 606ef8eb43c760557326d119c4e51ea260e0f36d

Authored by Antonio Terceiro
1 parent a4675300
Exists in staging and in 42 other branches all_pending_tasks_api, api-articles-period, api_roles, caching-rails4, captcha_serpro_plugin, comments_permissions, content-manager-hostspot, elasticsearch, elasticsearch_api, elasticsearch_categories, elasticsearch_filter, elasticsearch_sort, elasticsearch_to_merge, elasticsearch_view, environment-exposes-api, export-comment-api, export-comment-paragraph, export_data, external_followers, federation-webfinger, federation_followers, federation_followers_backend, federation_oauth_provider, federation_webfinger, fix_event_date_issue, fix_notification_email, fix_string_downcase_and_upcase, follower_permition, json_cookie_serializer, login-captcha, master, master_profile_followers, oauth_external_login, oauth_login, private-scraps, private-scraps-rebase, production, production-vendorized, profile_api_improvements, tasks_keep_filter_params, user_mention, webfinger_server

ActionItem618: fixing tests

Showing 2 changed files with 4 additions and 0 deletions   Show diff stats
test/functional/mailconf_controller_test.rb
  Diff comments   View file @606ef8e
@@ -24,6 +24,7 @@ class MailconfControllerTest < Test::Unit::TestCase @@ -24,6 +24,7 @@ class MailconfControllerTest < Test::Unit::TestCase
24 24
25 should 'not be used by organizations' do 25 should 'not be used by organizations' do
26 org = Organization.create!(:name => 'testorg', :identifier => 'testorg') 26 org = Organization.create!(:name => 'testorg', :identifier => 'testorg')
  27 + login_as('ze')
27 get :index, :profile => 'testorg' 28 get :index, :profile => 'testorg'
28 assert_response 403 29 assert_response 403
29 end 30 end
test/functional/my_profile_controller_test.rb
  Diff comments   View file @606ef8e
@@ -34,6 +34,7 @@ class MyProfileControllerTest < Test::Unit::TestCase @@ -34,6 +34,7 @@ class MyProfileControllerTest < Test::Unit::TestCase
34 def test_should_allow_person 34 def test_should_allow_person
35 @controller = OnlyForPersonTestController.new 35 @controller = OnlyForPersonTestController.new
36 person = create_user('random_joe') 36 person = create_user('random_joe')
  37 + login_as('random_joe')
37 38
38 get :index, :profile => 'random_joe' 39 get :index, :profile => 'random_joe'
39 assert_response :success 40 assert_response :success
@@ -42,6 +43,8 @@ class MyProfileControllerTest < Test::Unit::TestCase @@ -42,6 +43,8 @@ class MyProfileControllerTest < Test::Unit::TestCase
42 def test_should_not_allow_bare_profile 43 def test_should_not_allow_bare_profile
43 @controller = OnlyForPersonTestController.new 44 @controller = OnlyForPersonTestController.new
44 org = Organization.create!(:identifier => 'hacking_institute', :name => 'Hacking Institute') 45 org = Organization.create!(:identifier => 'hacking_institute', :name => 'Hacking Institute')
  46 + create_user('random_joe')
  47 + login_as('random_joe')
45 48
46 get :index, :profile => 'hacking_institute' 49 get :index, :profile => 'hacking_institute'
47 assert_response 403 # forbidden 50 assert_response 403 # forbidden