Commit a0884dec6aead874b8938f58bb11780c1c19b2fb
Committed by
Rafael Manzo
1 parent
fac2efa5
Exists in
master
and in
28 other branches
[Mezuro] Fixed 'get module result root when nil is given' test
Showing
1 changed file
with
38 additions
and
16 deletions
Show diff stats
plugins/mezuro/test/unit/mezuro_plugin/project_content_test.rb
| @@ -44,29 +44,25 @@ class ProjectContentTest < ActiveSupport::TestCase | @@ -44,29 +44,25 @@ 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 | - Kalibro::Client::ProjectClient.expects(:project).with(@content.name).returns(@project) | ||
| 48 | - | ||
| 49 | - project_result = mock | ||
| 50 | - project_result.expects(:date).returns('12/04/2012') | ||
| 51 | - Kalibro::Client::ProjectResultClient.expects(:last_result).with(@content.name).returns(project_result) | ||
| 52 | - | 47 | + mock_project_client |
| 48 | + project_result = mock_project_result_client | ||
| 53 | module_name = 'My module name' | 49 | module_name = 'My module name' |
| 54 | module_result_client = mock | 50 | module_result_client = mock |
| 55 | module_result = Kalibro::Entities::ModuleResult.new | 51 | 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) | 52 | + @content.expects(:module_result_client).returns(module_result_client) |
| 53 | + module_result_client.expects(:module_result).with(@project.name, module_name, project_result.date). | ||
| 54 | +returns(module_result) | ||
| 58 | assert_equal module_result, @content.module_result(module_name) | 55 | assert_equal module_result, @content.module_result(module_name) |
| 59 | end | 56 | end |
| 60 | 57 | ||
| 61 | should 'get module result root when nil is given' do | 58 | should 'get module result root when nil is given' do |
| 62 | - module_result = mock | ||
| 63 | - module_result_client = mock | ||
| 64 | - project_result = mock | ||
| 65 | - @content.expects(:project_result).returns(project_result) | ||
| 66 | - project_result.expects(:date).returns('12/04/2012') | ||
| 67 | - @content.expects(:module_result_client).returns(module_result_client) | ||
| 68 | - module_result_client.expects(:module_result).with(@project.name, @project.name, '12/04/2012'). | ||
| 69 | - returns(module_result) | 59 | + mock_project_client |
| 60 | + project_result = mock_project_result_client | ||
| 61 | + module_result_client = mock | ||
| 62 | + module_result = Kalibro::Entities::ModuleResult.new | ||
| 63 | + @content.expects(:module_result_client).returns(module_result_client) | ||
| 64 | + module_result_client.expects(:module_result).with(@project.name, @project.name, project_result.date). | ||
| 65 | +returns(module_result) | ||
| 70 | assert_equal module_result, @content.module_result(nil) | 66 | assert_equal module_result, @content.module_result(nil) |
| 71 | end | 67 | end |
| 72 | 68 | ||
| @@ -94,4 +90,30 @@ class ProjectContentTest < ActiveSupport::TestCase | @@ -94,4 +90,30 @@ class ProjectContentTest < ActiveSupport::TestCase | ||
| 94 | @content.send :remove_project_from_service | 90 | @content.send :remove_project_from_service |
| 95 | end | 91 | end |
| 96 | 92 | ||
| 93 | + private | ||
| 94 | + def mock_project_client | ||
| 95 | + Kalibro::Client::ProjectClient.expects(:project).with(@content.name).returns(@project) | ||
| 96 | + end | ||
| 97 | + | ||
| 98 | + def mock_project_result_client | ||
| 99 | + project_result = ProjectResultFixtures.qt_calculator | ||
| 100 | + Kalibro::Client::ProjectResultClient.expects(:last_result).with(@content.name).returns(project_result) | ||
| 101 | + project_result | ||
| 102 | + end | ||
| 103 | + | ||
| 97 | end | 104 | end |
| 105 | + | ||
| 106 | + | ||
| 107 | + | ||
| 108 | + | ||
| 109 | + | ||
| 110 | + | ||
| 111 | + | ||
| 112 | + | ||
| 113 | + | ||
| 114 | + | ||
| 115 | + | ||
| 116 | + | ||
| 117 | + | ||
| 118 | + | ||
| 119 | + |