Download as
Browse Code »

Commit fac2efa57992551b80171f828604db7434e9771e

Authored by Alessandro Palmeira + Caio Salgado
Committed by Rafael Manzo
1 parent 3e12b2c0
Exists in master and in 28 other branches 1.2+SPB2, 2_split_by_role, add_fields_in_api, api_fixies, api_pagination, api_private_token, api_tasks, article_hit_count, blog_page_bug, checkbox_to_user_can_edit_page, communities_ratings, communities_ratings_moderation, community_ratings_rebase, fix_cloned_article_parent, fix_gallery_image_url, fix_javascript_categories, fix_optional_fields, fix_suggest_article, fix_user_email_validation, highlight_block_fix, notification_plugin, notification_plugin_rebase, script_to_update, split_by_role, stable, temp_ratings, tests, tests_fixies

[Mezuro] Fixed 'get module result from service' test

Showing 2 changed files with 10 additions and 12 deletions   Show diff stats
plugins/mezuro/lib/mezuro_plugin/project_content.rb
  Diff comments   View file @fac2efa
@@ -20,10 +20,6 @@ class MezuroPlugin::ProjectContent < Article @@ -20,10 +20,6 @@ class MezuroPlugin::ProjectContent < Article
20 def project 20 def project
21 @project ||= Kalibro::Client::ProjectClient.project(name) 21 @project ||= Kalibro::Client::ProjectClient.project(name)
22 end 22 end
23 -  
24 - def project  
25 - @project ||= Kalibro::Client::ProjectClient.project(name)  
26 - end  
27 23
28 def project_result 24 def project_result
29 @project_result ||= Kalibro::Client::ProjectResultClient.last_result(name) 25 @project_result ||= Kalibro::Client::ProjectResultClient.last_result(name)
plugins/mezuro/test/unit/mezuro_plugin/project_content_test.rb
  Diff comments   View file @fac2efa
@@ -44,15 +44,17 @@ class ProjectContentTest < ActiveSupport::TestCase @@ -44,15 +44,17 @@ class ProjectContentTest < ActiveSupport::TestCase
44 end 44 end
45 45
46 should 'get module result from service' do 46 should 'get module result from service' do
47 - module_name = 'My module name'  
48 - module_result = mock  
49 - module_result_client = mock  
50 - project_result = mock  
51 - @content.expects(:project_result).returns(project_result) 47 + Kalibro::Client::ProjectClient.expects(:project).with(@content.name).returns(@project)
  48 +
  49 + project_result = mock
52 project_result.expects(:date).returns('12/04/2012') 50 project_result.expects(:date).returns('12/04/2012')
53 - @content.expects(:module_result_client).returns(module_result_client)  
54 - module_result_client.expects(:module_result).with(@project.name, module_name, '12/04/2012').  
55 - returns(module_result) 51 + Kalibro::Client::ProjectResultClient.expects(:last_result).with(@content.name).returns(project_result)
  52 +
  53 + module_name = 'My module name'
  54 + module_result_client = mock
  55 + module_result = Kalibro::Entities::ModuleResult.new
  56 + Kalibro::Client::ModuleResultClient.expects(:new).returns(module_result_client)
  57 + module_result_client.expects(:module_result).with(@project.name, module_name, '12/04/2012').returns(module_result)
56 assert_equal module_result, @content.module_result(module_name) 58 assert_equal module_result, @content.module_result(module_name)
57 end 59 end
58 60