From 38230573e91978fcf0b5a768ca7aa9ec121ab1cc Mon Sep 17 00:00:00 2001 From: Max Kanat-Alexander Date: Sat, 2 Oct 2010 12:45:23 -0700 Subject: Change the way that the xt/search.t injection tests look for known broken tests, so that the Pg tests properly understand that they can't test creation_ts changedafter or changedbefore, *ever*. https://bugzilla.mozilla.org/show_bug.cgi?id=600496 --- xt/lib/Bugzilla/Test/Search/InjectionTest.pm | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'xt/lib/Bugzilla/Test/Search') diff --git a/xt/lib/Bugzilla/Test/Search/InjectionTest.pm b/xt/lib/Bugzilla/Test/Search/InjectionTest.pm index 1ee83c57c..1bd9fd82c 100644 --- a/xt/lib/Bugzilla/Test/Search/InjectionTest.pm +++ b/xt/lib/Bugzilla/Test/Search/InjectionTest.pm @@ -37,15 +37,14 @@ sub _known_broken { # We don't want to auto-vivify $operator_broken and thus make it true. my @field_ok = $operator_broken ? @{ $operator_broken->{field_ok} || [] } : (); - - return {} if grep { $_ eq $self->field } @field_ok; + $operator_broken = undef if grep { $_ eq $self->field } @field_ok; my $field_broken = INJECTION_BROKEN_FIELD->{$self->field} || INJECTION_BROKEN_FIELD->{$self->field_object->type}; # We don't want to auto-vivify $field_broken and thus make it true. my @operator_ok = $field_broken ? @{ $field_broken->{operator_ok} || [] } : (); - return {} if grep { $_ eq $self->operator } @operator_ok; + $field_broken = undef if grep { $_ eq $self->operator } @operator_ok; return $operator_broken || $field_broken || {}; } -- cgit v1.2.3-24-g4f1b