601364 - search.t should also test the basic standard search parameters

Status: RESOLVED FIXED

(Bugzilla :: Testing Suite, enhancement)

Description

[Max Kanat-Alexander]

Right now, we only test things via the boolean charts, but we should also make sure that the standard search parameters (and particularly the special ones, like chfield) continue to work exactly as they always have (or better).

Comment 1

[Max Kanat-Alexander]

Okay, I'm going to focus and limit this first bug to just testing the simple standard search parameters, the basic field=value ones.

Comment 2

[Max Kanat-Alexander]

Attachment: v1

Comment 3

[Max Kanat-Alexander]

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
modified xt/lib/Bugzilla/Test/Search.pm
modified xt/lib/Bugzilla/Test/Search/FieldTest.pm                              
added xt/lib/Bugzilla/Test/Search/FieldTestNormal.pm
modified xt/lib/Bugzilla/Test/Search/OperatorTest.pm
Committed revision 7498.

Comment 4

[Max Kanat-Alexander]

Oh, wait, there's a bug in this patch that is preventing it from actually doing the right thing.

Comment 5

[Max Kanat-Alexander]

Attachment: Fix

This fixes the search test to actually work. (The problem before was that it was re-using the existing search_params from $field_test.)

Comment 6

[Max Kanat-Alexander]

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
modified xt/lib/Bugzilla/Test/Search/FieldTestNormal.pm
Committed revision 7501.