From 3da1e1c91effff02857badfe3a472a49aba9d60d Mon Sep 17 00:00:00 2001 From: Brian Cully Date: Sun, 19 Jun 2022 20:12:04 -0400 Subject: Fix off-by-one error in subject and remaining count header. --- t/fixtures/bar.expected | 2 +- t/fixtures/foo.expected | 2 +- t/fixtures/foo2.expected | 2 +- t/fixtures/multiple.expected | 2 +- t/fixtures/nosubj.expected | 2 +- t/lib.t | 12 ++++++------ 6 files changed, 11 insertions(+), 11 deletions(-) (limited to 't') diff --git a/t/fixtures/bar.expected b/t/fixtures/bar.expected index e554c65..f351443 100644 --- a/t/fixtures/bar.expected +++ b/t/fixtures/bar.expected @@ -6,7 +6,7 @@ Received: from mta.example.com Received: by mta.example.com (Postfix, from userid 1001) id 844956F1EE; Thu, 20 Nov 2014 17:17:18 -0500 (EST) To: bar@spamcat.example.com -Subject: test - [21/20] +Subject: test - [20/20] Message-Id: Date: Thu, 20 Nov 2014 17:17:18 -0500 (EST) From: sender@example.com (Spamcat Sender) diff --git a/t/fixtures/foo.expected b/t/fixtures/foo.expected index 7516338..cc0ef3b 100644 --- a/t/fixtures/foo.expected +++ b/t/fixtures/foo.expected @@ -10,6 +10,6 @@ Subject: test - [3/20] Message-Id: Date: Thu, 20 Nov 2014 17:17:18 -0500 (EST) From: sender@example.com (Spamcat Sender) -X-SpamCat-Remaining: 18 +X-SpamCat-Remaining: 17 Sample email. diff --git a/t/fixtures/foo2.expected b/t/fixtures/foo2.expected index ed86987..da27a77 100644 --- a/t/fixtures/foo2.expected +++ b/t/fixtures/foo2.expected @@ -10,6 +10,6 @@ Subject: test - [4/20] Message-Id: Date: Thu, 20 Nov 2014 17:17:18 -0500 (EST) From: sender@example.com (Spamcat Sender) -X-SpamCat-Remaining: 17 +X-SpamCat-Remaining: 16 Sample email. diff --git a/t/fixtures/multiple.expected b/t/fixtures/multiple.expected index c867cc4..66e4fa6 100644 --- a/t/fixtures/multiple.expected +++ b/t/fixtures/multiple.expected @@ -11,6 +11,6 @@ Subject: test - [1/20] Message-Id: Date: Thu, 20 Nov 2014 17:17:18 -0500 (EST) From: sender@example.com (Spamcat Sender) -X-SpamCat-Remaining: 20 +X-SpamCat-Remaining: 19 Sample email. diff --git a/t/fixtures/nosubj.expected b/t/fixtures/nosubj.expected index 440a862..cf017ec 100644 --- a/t/fixtures/nosubj.expected +++ b/t/fixtures/nosubj.expected @@ -10,6 +10,6 @@ Message-Id: Date: Thu, 20 Nov 2014 17:17:18 -0500 (EST) From: sender@example.com (Spamcat Sender) Subject: [1/20] -X-SpamCat-Remaining: 20 +X-SpamCat-Remaining: 19 Sample email. diff --git a/t/lib.t b/t/lib.t index 41b5704..efe9b31 100644 --- a/t/lib.t +++ b/t/lib.t @@ -30,12 +30,12 @@ my $sch = SpamCat->new(%conf); ok(defined $sch, 'Constructor returns instance'); ok(SpamCat->can('decrement_count'), 'Has count decrementor'); -is($sch->decrement_count('foo'), $conf{default_count}, +is($sch->decrement_count('foo'), 19, 'Default count for new sender'); -is($sch->decrement_count('foo'), 19, 'Existing sender decrements'); +is($sch->decrement_count('foo'), 18, 'Existing sender decrements'); ok(SpamCat->can('get_count'), 'Has count getter'); -is($sch->get_count('foo'), 19, 'Returns existing sender count'); +is($sch->get_count('foo'), 18, 'Returns existing sender count'); ok(!defined $sch->get_count('doesntexist'), 'Non-existant sender has undefined count'); @@ -77,11 +77,11 @@ is($rows[0]->{count}, -1); is($rows[1]->{sender}, 'bar'); is($rows[1]->{count}, 0); is($rows[2]->{sender}, 'foo'); -is($rows[2]->{count}, 16); +is($rows[2]->{count}, 15); is($rows[3]->{sender}, 'name1'); -is($rows[3]->{count}, 20); +is($rows[3]->{count}, 19); is($rows[4]->{sender}, 'nosubj'); -is($rows[4]->{count}, 20); +is($rows[4]->{count}, 19); sub test_file { my ($filen) = @_; -- cgit v1.3