Bugzilla – Attachment 36869 Details for
Bug 13814
Add unit tests for C4::Boolean
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 13814 : Add Lines Alignment
PASSED-QA-Bug-13814--Add-Lines-Alignment.patch (text/plain), 2.07 KB, created by
Kyle M Hall (khall)
on 2015-03-13 15:04:29 UTC
(
hide
)
Description:
[PASSED QA] Bug 13814 : Add Lines Alignment
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2015-03-13 15:04:29 UTC
Size:
2.07 KB
patch
obsolete
>From 72ba4e063de8d74cb0490f3f7257c11075dbbf62 Mon Sep 17 00:00:00 2001 >From: Julian FIOL <julian.fiol@biblibre.com> >Date: Tue, 10 Mar 2015 17:10:58 +0100 >Subject: [PATCH] [PASSED QA] Bug 13814 : Add Lines Alignment > >Signed-off-by: Marc Veron <veron@veron.ch> > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > t/Boolean.t | 14 +++++++------- > 1 files changed, 7 insertions(+), 7 deletions(-) > >diff --git a/t/Boolean.t b/t/Boolean.t >index af81f13..b82068c 100755 >--- a/t/Boolean.t >+++ b/t/Boolean.t >@@ -7,22 +7,22 @@ use Test::More tests => 19; > BEGIN { use_ok( 'C4::Boolean', qw( true_p ) ); } > > is( true_p('0'), '0', 'recognizes \'0\' as false' ); >-is( true_p('nil'), '0', 'recognizes \'nil\' as false' ); >+is( true_p('nil'), '0', 'recognizes \'nil\' as false' ); > is( true_p('false'), '0', 'recognizes \'false\' as false' ); > is( true_p('off'), '0', 'recognizes \'off\' as false' ); > is( true_p('no'), '0', 'recognizes \'no\' as false' ); >-is( true_p('n'), '0', 'recognizes \'n\' as false' ); >-is( true_p('NO'), '0', 'verified case insensitivity' ); >+is( true_p('n'), '0', 'recognizes \'n\' as false' ); >+is( true_p('NO'), '0', 'verified case insensitivity' ); > > is( true_p('1'), '1', 'recognizes \'1\' as true' ); >-is( true_p('-1'), '1', 'recognizes \'-1\' as true' ); >-is( true_p('t'), '1', 'recognizes \'t\' as true' ); >+is( true_p('-1'), '1', 'recognizes \'-1\' as true' ); >+is( true_p('t'), '1', 'recognizes \'t\' as true' ); > is( true_p('true'), '1', 'recognizes \'true\' as true' ); > is( true_p('on'), '1', 'recognizes \'on\' as true' ); > is( true_p('yes'), '1', 'recognizes \'yes\' as true' ); >-is( true_p('y'), '1', 'recognizes \'y\' as true' ); >+is( true_p('y'), '1', 'recognizes \'y\' as true' ); > is( true_p('YES'), '1', 'verified case insensitivity' ); > > is( true_p(undef), undef, 'recognizes undefined as not boolean' ); > is( true_p('foo'), undef, 'recognizes \'foo\' as not boolean' ); >-is( true_p([]), undef, 'recognizes a reference as not a boolean' ); >+is( true_p([]), undef, 'recognizes a reference as not a boolean' ); >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 13814
:
36758
|
36765
|
36771
|
36773
|
36774
|
36868
| 36869 |
36891