Bugzilla – Attachment 29894 Details for
Bug 7162
Factorize code for order cancellation and add field for giving a reason
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 7162: (follow-up) Add unit tests for DelOrder
PASSED-QA-Bug-7162-follow-up-Add-unit-tests-for-De.patch (text/plain), 1.44 KB, created by
Katrin Fischer
on 2014-07-20 20:21:13 UTC
(
hide
)
Description:
[PASSED QA] Bug 7162: (follow-up) Add unit tests for DelOrder
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2014-07-20 20:21:13 UTC
Size:
1.44 KB
patch
obsolete
>From d338aa55c73e047a135b02f0d2e4ca3d00896565 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Tue, 15 Jul 2014 14:00:25 +0200 >Subject: [PATCH] [PASSED QA] Bug 7162: (follow-up) Add unit tests for DelOrder > >The unit tests did not take into account the new DB field. > >Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Passes all tests and QA script. Also all Acquisition related >patches. >--- > t/db_dependent/Acquisition.t | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > >diff --git a/t/db_dependent/Acquisition.t b/t/db_dependent/Acquisition.t >index 2f7539e..7da17fd 100755 >--- a/t/db_dependent/Acquisition.t >+++ b/t/db_dependent/Acquisition.t >@@ -8,7 +8,7 @@ use POSIX qw(strftime); > > use C4::Bookseller qw( GetBookSellerFromId ); > >-use Test::More tests => 88; >+use Test::More tests => 91; > > BEGIN { > use_ok('C4::Acquisition'); >@@ -408,6 +408,7 @@ my @expectedfields = qw( > quantity_to_receive > subtotal > latesince >+ cancellationreason > ); > ( > $test_missing_fields, $test_extra_fields, >@@ -521,6 +522,7 @@ my @base_expectedfields = qw( > frameworkcode > volumedesc > datecancellationprinted >+ cancellationreason > ); > @expectedfields = > ( @base_expectedfields, >@@ -649,6 +651,7 @@ ok( > budget_id > authorisedby > booksellerid >+ cancellationreason > ); > > # note that authorisedby was added to the return of SearchOrder by the >-- >1.9.1
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 7162
:
6219
|
7007
|
9148
|
9238
|
13224
|
13225
|
16373
|
16374
|
16409
|
16457
|
18455
|
18456
|
19315
|
21333
|
21334
|
21335
|
22719
|
22720
|
22721
|
24075
|
24076
|
24077
|
26235
|
26236
|
26237
|
28353
|
28558
|
28559
|
28560
|
28836
|
29535
|
29536
|
29661
|
29662
|
29663
|
29664
|
29665
|
29698
|
29889
|
29890
|
29891
|
29892
|
29893
|
29894
|
29918
|
29943
|
32895
|
32896
|
32897
|
32898
|
32899
|
32900
|
32902