From 156e3078e205dcbf849590b343e1ae08f8f4d4d8 2015-06-13 11:09:30
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: 2015-06-13 11:09:30
Subject: [PATCH] Adds test for when filtering on invalid issue ids (#6118).

git-svn-id: http://svn.redmine.org/redmine/trunk@14305 e93f8b46-1217-0410-a6f0-8f06a7374b81

---

diff --git a/test/unit/query_test.rb b/test/unit/query_test.rb
index 64cb8d1..1907cd9 100644
--- a/test/unit/query_test.rb
+++ b/test/unit/query_test.rb
@@ -898,6 +898,15 @@ class QueryTest < ActiveSupport::TestCase
     assert_equal [parent.id], find_issues_with_query(query).map(&:id).sort
   end
 
+  def test_filter_on_invalid_parent_should_return_no_results
+    query = IssueQuery.new(:name => '_')
+    query.filters = {"parent_id" => {:operator => '=', :values => '99999999999'}}
+    assert_equal [], find_issues_with_query(query).map(&:id).sort
+
+    query.filters = {"parent_id" => {:operator => '~', :values => '99999999999'}}
+    assert_equal [], find_issues_with_query(query)
+  end
+
   def test_filter_on_child
     Issue.delete_all
     parent = Issue.generate_with_descendants!
@@ -919,6 +928,15 @@ class QueryTest < ActiveSupport::TestCase
     assert_equal [grandchild, leaf].map(&:id).sort, find_issues_with_query(query).map(&:id).sort
   end
 
+  def test_filter_on_invalid_child_should_return_no_results
+    query = IssueQuery.new(:name => '_')
+    query.filters = {"child_id" => {:operator => '=', :values =>  '99999999999'}}
+    assert_equal [], find_issues_with_query(query)
+
+    query.filters = {"child_id" => {:operator => '~', :values =>  '99999999999'}}
+    assert_equal [].map(&:id).sort, find_issues_with_query(query)
+  end
+
   def test_statement_should_be_nil_with_no_filters
     q = IssueQuery.new(:name => '_')
     q.filters = {}