002goodperl.t adds an extra test for every additional Throw*Error violation in the same file

RESOLVED FIXED in Bugzilla 2.18

Status

()

RESOLVED FIXED
15 years ago
6 years ago

People

(Reporter: justdave, Assigned: Wurblzap)

Tracking

2.17.4
Bugzilla 2.18
Bug Flags:
approval +
approval2.18 +

Details

Attachments

(1 attachment, 1 obsolete attachment)

# Looks like you planned 240 tests but ran 8 extra.
dubious
        Test returned status 19 (wstat 4864, 0x1300)
DIED. FAILED tests 199-206, 220-222
        Failed 11/240 tests, 95.42% okay

example:
#     Failed test (t/002goodperl.t at line 113)
not ok 199 - post_bug.cgi has a Throw*Error call on line 181 
#                       which doesn't use a tag --ERROR
#     Failed test (t/002goodperl.t at line 113)
not ok 200 - post_bug.cgi has a Throw*Error call on line 612 
#                       which doesn't use a tag --ERROR
#     Failed test (t/002goodperl.t at line 113)
not ok 201 - post_bug.cgi has a Throw*Error call on line 619 
#                       which doesn't use a tag --ERROR
#     Failed test (t/002goodperl.t at line 113)
not ok 202 - process_bug.cgi has a Throw*Error call on line 1230 
#                       which doesn't use a tag --ERROR
#     Failed test (t/002goodperl.t at line 113)
not ok 203 - process_bug.cgi has a Throw*Error call on line 1243 
#                       which doesn't use a tag --ERROR
#     Failed test (t/002goodperl.t at line 113)
not ok 204 - process_bug.cgi has a Throw*Error call on line 1253 
#                       which doesn't use a tag --ERROR
#     Failed test (t/002goodperl.t at line 113)
not ok 205 - process_bug.cgi has a Throw*Error call on line 1257 
#                       which doesn't use a tag --ERROR
#     Failed test (t/002goodperl.t at line 113)
not ok 206 - process_bug.cgi has a Throw*Error call on line 1598 
#                       which doesn't use a tag --ERROR

Note this is running the test on Zippy's code, not mainstream Bugzilla, so these
aren't errors we need to fix, just the test is broken because it reports a new
test number for each error instead of collecting all the errors from the same
file and reporting them as one test.

Comment 1

15 years ago
Created attachment 130005 [details] [diff] [review]
Patch v1

We really should use File::Find and do this right, but this should fix it until
we go even deeper in the directory structure (at which point we've gone insane
;)

Comment 2

15 years ago
Comment on attachment 130005 [details] [diff] [review]
Patch v1

Gaah, attached this to the wrong bug :)
Attachment #130005 - Attachment is obsolete: true
(Assignee)

Comment 3

14 years ago
Created attachment 157136 [details] [diff] [review]
Patch

One-liner.
(Assignee)

Updated

14 years ago
Attachment #157136 - Flags: review?

Comment 4

14 years ago
Comment on attachment 157136 [details] [diff] [review]
Patch

Doh :-)
Attachment #157136 - Flags: review? → review+

Updated

14 years ago
Assignee: zach → marcschum
Flags: approval?
Flags: approval?
Flags: approval2.18+
Flags: approval+
Target Milestone: --- → Bugzilla 2.18

Comment 5

14 years ago
Checking in t/002goodperl.t;
/cvsroot/mozilla/webtools/bugzilla/t/002goodperl.t,v  <--  002goodperl.t
new revision: 1.14; previous revision: 1.13
done
Checking in t/002goodperl.t;
/cvsroot/mozilla/webtools/bugzilla/t/002goodperl.t,v  <--  002goodperl.t
new revision: 1.13.2.1; previous revision: 1.13
done
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.