You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2005/08/26 18:44:56 UTC
[Bug 4559] New: create a message parser test suite
http://bugzilla.spamassassin.org/show_bug.cgi?id=4559
Summary: create a message parser test suite
Product: Spamassassin
Version: 3.1.0
Platform: Other
OS/Version: other
Status: NEW
Severity: enhancement
Priority: P5
Component: Regression Tests
AssignedTo: dev@spamassassin.apache.org
ReportedBy: jm@jmason.org
Every now and again, we come up against bugs in our message parser (MIME, HTML,
headers, base64/qp decoding, etc. etc.) We fix them, but occasionally there's
regressions. Funnily enough, that's why regression test suites were invented! ;)
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.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 4559] create a message parser test suite
Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4559
jm@jmason.org changed:
What |Removed |Added
----------------------------------------------------------------------------
OtherBugsDependingO| |4560
nThis| |
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 4559] create a message parser test suite
Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4559
------- Additional Comments From felicity@apache.org 2005-08-26 10:58 -------
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.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 4559] create a message parser test suite
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4559
felicity@apache.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|Undefined |Future
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.