237778 - update filter list in t/004template

Status: RESOLVED FIXED

(Bugzilla :: Testing Suite, defect)

Description

[Vlad Dascalu]

We should probably add the ics filter to the testing suite. It's already in 008
but it's missing from 004.

Comment 1

[Vlad Dascalu]

Attachment: V1: adds the ics filter in the 004filter test

Comment 2

[Christian Reis]

Vlad, help me out here and find out if the format here has to be compatible with
the Template.pm format (i.e., filters taking arguments need to have be function
generators).

Comment 3

[Vlad Dascalu]

Gerv, William: any know-how regarding the comment above?

Comment 4

[William Jon McCann]

It should probably look like the bug_link line.

            ics => [ sub { return sub { return $_; } }, 1] ,

Comment 5

[Christian Reis]

If you want to fix up the closed/obsolete/inactive filters we added yesterday
too I'd be real happy about it.

Comment 6

[Vlad Dascalu]

Attachment: Version 2

Sorry for the delay.

Comment 7

[Christian Reis]

Comment on attachment 144471 [details] [diff] [review]
Version 2

Looks good, but I'm wondering: when are these used if the tests are running
fine now?

Comment 8

[Vlad Dascalu]

Checking in t/004template.t;
/cvsroot/mozilla/webtools/bugzilla/t/004template.t,v  <--  004template.t
new revision: 1.32; previous revision: 1.31
done