You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by Theo Van Dinter <fe...@apache.org> on 2006/01/07 07:20:04 UTC

Re: svn commit: r366606 - /spamassassin/rules/trunk/sandbox/felicity/70_other.cf

On Sat, Jan 07, 2006 at 12:08:40AM -0000, jm@apache.org wrote:
> comment failing rule test
> +# TODO: these are failing, http://buildbot.spamassassin.org:8010/t-sol10-561/builds/17/test/0
> +# test T_INVALID_DATE ok    Sat, 31 Dec 2005 23:59:60 -0500
> +# test T_INVALID_DATE fail  Sat, 31 Dec 2005 24:00:00 -0500

Yeah.  I had some time to look into this today.  Apparently somewhere in the
code the name changes from T_INVALID_DATE to TEST_INVALID_DATE:

sub TEST_INVALID_DATE_head_test {
[...]
$self->got_hit (q#TEST_INVALID_DATE#, q{});
[...]

and then all hell breaks loose (rule_tests.t specifically disables everything
and enables the T_* name, so the rule never actually runs, etc.)  I haven't
been able to track down where that's happening yet.

-- 
Randomly Generated Tagline:
<This Tagline has been removed by U.S. Customs>

Re: svn commit: r366606 - /spamassassin/rules/trunk/sandbox/felicity/70_other.cf

Posted by Theo Van Dinter <fe...@apache.org>.
On Sat, Jan 07, 2006 at 01:20:04AM -0500, Theo Van Dinter wrote:
> Yeah.  I had some time to look into this today.  Apparently somewhere in the
> code the name changes from T_INVALID_DATE to TEST_INVALID_DATE:

Hrm.  Never mind.  I was thrown off by the fact that there *is*
a random TEST_INVALID_DATE being created for some reason, but
T_INVALID_DATE_head_test exists too, so that's not it.  Still digging.
<sigh>

-- 
Randomly Generated Tagline:
"Always bear in mind that your own resolution to succeed is more important
 than any other."                        - Abraham Lincoln