From 3235d119222923e913522fe0a6b1cbd6b117a23b Mon Sep 17 00:00:00 2001 From: Jared Camins-Esakov Date: Thu, 9 Aug 2012 07:45:05 -0400 Subject: [PATCH] Bug 8524 follow-up: fix broken test Content-Type: text/plain; charset="UTF-8" The t/Barcodes_ValueBuilder.t test had the wrong number of tests declared, and rather than checking that a variable was undefined I was checking that it had a length of zero. Fixed both issues. --- t/Barcodes_ValueBuilder.t | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/t/Barcodes_ValueBuilder.t b/t/Barcodes_ValueBuilder.t index c3d816e..0bbad8c 100644 --- a/t/Barcodes_ValueBuilder.t +++ b/t/Barcodes_ValueBuilder.t @@ -2,7 +2,7 @@ use strict; use warnings; use DBI; -use Test::More tests => 15; +use Test::More tests => 10; use Test::MockModule; BEGIN { @@ -40,7 +40,7 @@ my ($nextnum, $scr, $history); ($nextnum, $scr) = C4::Barcodes::ValueBuilder::incremental::get_barcode(\%args); is($nextnum, 33333074344564, 'incremental barcode'); -ok(length($scr) == 0, 'incremental javascript'); +is($scr, undef, 'incremental javascript'); # This should run exactly one query so we can test $history = $dbh->{mock_all_history}; @@ -70,7 +70,7 @@ $dbh->{mock_add_resultset} = $annualdata; $dbh->{mock_clear_history} = 1; ($nextnum, $scr) = C4::Barcodes::ValueBuilder::annual::get_barcode(\%args); is($nextnum, '2012-0035', 'annual barcode'); -ok(length($scr) == 0, 'annual javascript'); +is($scr, undef, 'annual javascript'); # This should run exactly one query so we can test $history = $dbh->{mock_all_history}; -- 1.7.2.5