|
SA Bugzilla – Full Text Bug Listing |
Summary: | create a message parser test suite | ||
---|---|---|---|
Product: | Spamassassin | Reporter: | Justin Mason <jm> |
Component: | Regression Tests | Assignee: | SpamAssassin Developer Mailing List <dev> |
Status: | RESOLVED WORKSFORME | ||
Severity: | enhancement | CC: | apache |
Priority: | P5 | ||
Version: | 3.1.0 | ||
Target Milestone: | Future | ||
Hardware: | Other | ||
OS: | other | ||
Whiteboard: | |||
Bug Depends on: | |||
Bug Blocks: | 4560 |
Description
Justin Mason
2005-08-26 09:44:56 UTC
Subject: Re: New: create a message parser test suite On Fri, Aug 26, 2005 at 09:44:56AM -0700, bugzilla-daemon@bugzilla.spamassassin.org wrote: > So this bug is a reminder that we need to implement a regression test suite for > our message parser, something like "t/messageparser.t". > > I envisage it as using a vast collection of message files, something like a > mass-check corpus. I don't think it's necessary to distribute these message > files, they could be in SVN only, in order not to bloat the distribution tarball > with unnecessary tests. It's worth noting that we already have t/mimeparse.t to cover some of these, but not all obviously. I don't think there's really an issue distributing the messages, I actually think the rule can just generate them and throw through parse() to get the output. I've typically been doing some form of "parse this, then verify the correct internal part layout and sha1sum outputs", though there are likely other/better ways to do it. Closing ancient stale bug. |