Commit 08b04c72be930c1d3f16660aae8851baa3e5a12c
Committed by
Rafael Manzo
1 parent
4107a13b
Exists in
master
and in
29 other branches
[Mezuro] Fixed 'get module result' tests
Showing
2 changed files
with
25 additions
and
15 deletions
Show diff stats
plugins/mezuro/test/functional/echo_port_test.rb
plugins/mezuro/test/functional/mezuro_plugin_profile_controller_test.rb
... | ... | @@ -72,28 +72,25 @@ class MezuroPluginProfileControllerTest < ActionController::TestCase |
72 | 72 | assert_response 200 |
73 | 73 | end |
74 | 74 | |
75 | - should 'get module result' do | |
75 | + should 'get module result without date' do | |
76 | 76 | create_project_content |
77 | - module_result_client = mock | |
77 | + mock_module_result | |
78 | 78 | Kalibro::Client::ProjectResultClient.expects(:last_result).with(@name).returns(@project_result) |
79 | - Kalibro::Client::ModuleResultClient.expects(:new).returns(module_result_client) | |
80 | - module_result_client.expects(:module_result).with(@name, @name, @project_result.date).returns(@module_result) | |
79 | + Kalibro::Client::ProjectClient.expects(:project).with(@name).returns(@project) | |
80 | + | |
81 | 81 | get :module_result, :profile => @profile.identifier, :id => @content.id, :module_name => @name |
82 | 82 | assert_response 200 |
83 | 83 | assert_select('h5', 'Metric results for: Qt-Calculator (APPLICATION)') |
84 | 84 | end |
85 | 85 | |
86 | 86 | should 'get module result from a specific date' do |
87 | - create_project_content | |
88 | - client = mock | |
89 | - module_result_client = mock | |
90 | - Kalibro::Client::ModuleResultClient.expects(:new).returns(module_result_client) | |
91 | - module_result_client.expects(:module_result).with(@name, @name, @project_result.date).returns(@module_result) | |
92 | - Kalibro::Client::ProjectResultClient.expects(:new).returns(client) | |
93 | - client.expects(:has_results_before).returns(true) | |
94 | - client.expects(:last_result_before).returns(@project_result) | |
95 | - get :module_result, :profile => @profile.identifier, :id => @content.id, :date => "2012-04-13T20:39:41+04:00", :module_name => @name | |
96 | - assert_response 200 | |
87 | + create_project_content | |
88 | + Kalibro::Client::ProjectClient.expects(:project).with(@name).returns(@project) | |
89 | + mock_module_result | |
90 | + mock_project_result | |
91 | + | |
92 | + get :module_result, :profile => @profile.identifier, :id => @content.id, :date => @project_result.date, :module_name => @name | |
93 | + assert_response 200 | |
97 | 94 | end |
98 | 95 | |
99 | 96 | should 'get project tree' do |
... | ... | @@ -121,5 +118,18 @@ class MezuroPluginProfileControllerTest < ActionController::TestCase |
121 | 118 | @content.expects(:send_project_to_service).returns(nil) |
122 | 119 | @content.save |
123 | 120 | end |
121 | + | |
122 | + def mock_project_result | |
123 | + project_result_client = mock | |
124 | + Kalibro::Client::ProjectResultClient.expects(:new).returns(project_result_client) | |
125 | + project_result_client.expects(:has_results_before).returns(true) | |
126 | + project_result_client.expects(:last_result_before).returns(@project_result) | |
127 | + end | |
128 | + | |
129 | + def mock_module_result | |
130 | + module_result_client = mock | |
131 | + Kalibro::Client::ModuleResultClient.expects(:new).returns(module_result_client) | |
132 | + module_result_client.expects(:module_result).with(@name, @name, @project_result.date).returns(@module_result) | |
133 | + end | |
124 | 134 | |
125 | 135 | end | ... | ... |