Download as
Browse Code »

Commit 7542bd256d4452ddba275496e0be589df319abc5

Authored by Carlos Morais + Paulo Meirelles
Committed by Carlos Morais
1 parent aec7550d
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

[mezuro] Removing test for nil module_result name 

Should never invoke projcet_content.module_result with nil
Showing 2 changed files with 0 additions and 8 deletions   Show diff stats
plugins/mezuro/lib/mezuro_plugin/project_content.rb
  Diff comments   View file @7542bd2
@@ -26,7 +26,6 @@ class MezuroPlugin::ProjectContent < Article @@ -26,7 +26,6 @@ class MezuroPlugin::ProjectContent < Article
26 end 26 end
27 27
28 def module_result(module_name) 28 def module_result(module_name)
29 - module_name = project.name if module_name.nil?  
30 @module_client ||= Kalibro::Client::ModuleResultClient.module_result(self, module_name) 29 @module_client ||= Kalibro::Client::ModuleResultClient.module_result(self, module_name)
31 end 30 end
32 31
plugins/mezuro/test/unit/mezuro_plugin/project_content_test.rb
  Diff comments   View file @7542bd2
@@ -51,13 +51,6 @@ class ProjectContentTest < ActiveSupport::TestCase @@ -51,13 +51,6 @@ class ProjectContentTest < ActiveSupport::TestCase
51 assert_equal module_result, @content.module_result(module_name) 51 assert_equal module_result, @content.module_result(module_name)
52 end 52 end
53 53
54 - should 'get module result root when nil is given' do  
55 - module_result = mock  
56 - Kalibro::Client::ModuleResultClient.expects(:module_result).with(@content, @project.name).  
57 - returns(module_result)  
58 - assert_equal module_result, @content.module_result(nil)  
59 - end  
60 -  
61 should 'get module result root when project name is give' do 54 should 'get module result root when project name is give' do
62 module_result = mock 55 module_result = mock
63 Kalibro::Client::ModuleResultClient.expects(:module_result).with(@content, @project.name). 56 Kalibro::Client::ModuleResultClient.expects(:module_result).with(@content, @project.name).