Actions
Defect #29708
closedWrong use of refute_includes in tests
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Description
The refute_includes
assertion expects the collection
first and the object
to test for after. The test in source:/trunk/test/functional/queries_controller_test.rb@17558#L627 uses the object
then collection
form though.
It seems most of Redmine tests use source:/trunk/test/test_helper.rb@17558#L205 assert_not_include
which expects the object
then collection
form. The following patch corrects this problem:
--- a/test/functional/queries_controller_test.rb
+++ b/test/functional/queries_controller_test.rb
@@ -645,7 +645,7 @@ class QueriesControllerTest < Redmine::ControllerTest
# response includes systemwide visible version
assert_include ["OnlineStore - Systemwide visible version", "7", "open"], json
# response doesn't include non visible version
- refute_includes ["Private child of eCookbook - Private Version of public subproject", "6", "open"], json
+ assert_not_include ["Private child of eCookbook - Private Version of public subproject", "6", "open"], json
end
def test_subproject_filter_time_entries_with_project_id_should_return_filter_values
Actions