Skip Menu |

This queue is for tickets about the Business-OnlinePayment-AuthorizeNet CPAN distribution.

Report information
The Basics
Id: 33223
Status: resolved
Priority: 0/
Queue: Business-OnlinePayment-AuthorizeNet
People
Owner: Nobody in particular
Requestors: dsteinbrunner [...] pobox.com
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in:
  • 3.18
  • 3.19
Fixed in: 3.20

History Show all quoted text
  Wed Feb 13 13:00:29 2008 dsteinbrunner [...] pobox.com - Ticket created
Subject: card_arb.t failure due to Start Date
Hello, While installing I got the following test error: t/card_arb.............2/5 # Failed test at t/card_arb.t line 36. # Start Date must not occur before the submission date. Use of uninitialized value in pattern match (m//) at t/card_arb.t line 39. # Failed test 'Get order number' # at t/card_arb.t line 39. # undef # doesn't match '(?-xism:^[0-9]{1,13}$)' # Looks like you failed 2 tests of 5. t/card_arb............. Dubious, test returned 2 (wstat 512, 0x200) Failed 2/5 subtests (less 2 skipped subtests: 1 okay) I changed the start param in the test file to todays date and that allowed all tests to pass. I figured I should not that and suggest that the date give be automated to be todays date to ease installation.
  Tue Jun 17 23:46:31 2008 http://coffeecode.net/ - Correspondence added
From: dan [...] coffeecode.net
I was running into the same problem, so I have attached a patch that automates the start date, as the bug reporter suggested. This solves the test failure for me.
--- Business-OnlinePayment-AuthorizeNet-3.19/t/card_arb.t 2007-11-19 22:03:07.000000000 -0500 +++ Business-OnlinePayment-AuthorizeNet-3.19.arb/t/card_arb.t 2008-06-17 20:42:27.000000000 -0400 @@ -1,6 +1,7 @@ #!/usr/bin/perl -w use Test::More; +use POSIX "strftime"; require "t/lib/test_account.pl"; my($login, $password) = test_account_or_skip('arb'); @@ -27,7 +28,7 @@ card_number => '4007000000027', expiration => expiration_date(), interval => '1 month', - start => '2007-12-01', + start => strftime("%Y-%m-%d", localtime()), periods => '3', ); $tx->test_transaction(1); # test, dont really charge
  Tue Jun 17 23:46:35 2008 The RT System itself - Status changed from 'new' to 'open'
  Wed Jun 18 19:58:23 2008 ivan-pause [...] 420.am - Correspondence added
Fixed in 3.20 On Tue Jun 17 23:46:31 2008, http://coffeecode.net/ wrote: Show quoted text
> I was running into the same problem, so I have attached a patch that > automates the start date, as the bug reporter suggested. This solves
the Show quoted text
> test failure for me. >
  Sat Nov 14 19:05:20 2009 ivan-pause [...] 420.am - Status changed from 'open' to 'resolved'
  Sat Nov 14 19:05:29 2009 ivan-pause [...] 420.am - Fixed in 3.20 added
  Sat Nov 14 19:05:29 2009 ivan-pause [...] 420.am - Severity Unimportant added
  Sat Nov 14 19:05:29 2009 ivan-pause [...] 420.am - Broken in 3.18 added
  Sat Nov 14 19:05:31 2009 ivan-pause [...] 420.am - Broken in 3.19 added