Commit 8c9cd344 authored by Max Kanat-Alexander's avatar Max Kanat-Alexander

Fix search.t now that negative keyword searches are working.

parent 133243ac
...@@ -211,9 +211,8 @@ use constant ATTACHMENT_BOOLEANS_CONTAINS_BROKEN => ( ...@@ -211,9 +211,8 @@ use constant ATTACHMENT_BOOLEANS_CONTAINS_BROKEN => (
# bug_file_loc can be NULL, so it gets missed by the normal # bug_file_loc can be NULL, so it gets missed by the normal
# notequals search. # notequals search.
# #
# keywords & longdescs "notequals" match if *any* of the values # longdescs "notequals" matches if *any* of the values
# are not equal to the string provided. Also, keywords fails to match # are not equal to the string provided.
# if there are no keywords on the bug.
# #
# attachments.* notequals doesn't find bugs that lack attachments. # attachments.* notequals doesn't find bugs that lack attachments.
# #
...@@ -236,7 +235,6 @@ use constant NEGATIVE_BROKEN => ( ...@@ -236,7 +235,6 @@ use constant NEGATIVE_BROKEN => (
cc => { contains => [1,5] }, cc => { contains => [1,5] },
deadline => { contains => [5] }, deadline => { contains => [5] },
dependson => { contains => [2,4,5] }, dependson => { contains => [2,4,5] },
keywords => { contains => [1,5] },
longdesc => { contains => [1] }, longdesc => { contains => [1] },
'longdescs.isprivate' => { contains => [1] }, 'longdescs.isprivate' => { contains => [1] },
percentage_complete => { contains => [1] }, percentage_complete => { contains => [1] },
...@@ -288,7 +286,7 @@ use constant ALLWORDS_BROKEN => ( ...@@ -288,7 +286,7 @@ use constant ALLWORDS_BROKEN => (
# nowords and nowordssubstr have these broken tests in common. # nowords and nowordssubstr have these broken tests in common.
# #
# flagtypes.name doesn't match bugs without flags. # flagtypes.name doesn't match bugs without flags.
# cc, keywords, longdescs.isprivate, and bug_group actually work properly in # cc, longdescs.isprivate, and bug_group actually work properly in
# terms of excluding bug 1 (since we exclude all values in the search, # terms of excluding bug 1 (since we exclude all values in the search,
# on our test), but still fail at including bug 5. # on our test), but still fail at including bug 5.
# The longdesc* and work_time fields, coincidentally, work completely # The longdesc* and work_time fields, coincidentally, work completely
...@@ -298,7 +296,6 @@ use constant NOWORDS_BROKEN => ( ...@@ -298,7 +296,6 @@ use constant NOWORDS_BROKEN => (
'flagtypes.name' => { contains => [5] }, 'flagtypes.name' => { contains => [5] },
bug_group => { contains => [5] }, bug_group => { contains => [5] },
cc => { contains => [5] }, cc => { contains => [5] },
keywords => { contains => [5] },
longdesc => {}, longdesc => {},
work_time => {}, work_time => {},
'longdescs.isprivate' => {}, 'longdescs.isprivate' => {},
...@@ -951,7 +948,8 @@ use constant INJECTION_BROKEN_FIELD => { ...@@ -951,7 +948,8 @@ use constant INJECTION_BROKEN_FIELD => {
operator_ok => [qw(allwordssubstr anywordssubstr casesubstring operator_ok => [qw(allwordssubstr anywordssubstr casesubstring
changedfrom changedto greaterthan greaterthaneq changedfrom changedto greaterthan greaterthaneq
lessthan lessthaneq notregexp notsubstring lessthan lessthaneq notregexp notsubstring
nowordssubstr regexp substring)] nowordssubstr regexp substring anywords
notequals nowords)]
}, },
}; };
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment