Bugzilla – Attachment 71740 Details for
Bug 15524
Set limit on maximum possible holds per patron by category
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 15524: (QA follow-up) Fix meaningless test
Bug-15524-QA-follow-up-Fix-meaningless-test.patch (text/plain), 1.39 KB, created by
Marcel de Rooy
on 2018-02-16 11:20:15 UTC
(
hide
)
Description:
Bug 15524: (QA follow-up) Fix meaningless test
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2018-02-16 11:20:15 UTC
Size:
1.39 KB
patch
obsolete
>From 51ffcd0ba716ac2feb2971f42bdf64db86d093a8 Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Fri, 16 Feb 2018 09:02:39 +0100 >Subject: [PATCH] Bug 15524: (QA follow-up) Fix meaningless test >Content-Type: text/plain; charset=utf-8 > >The last test claims to allow a hold when branch=5 and patron=5, but look >at the preceding statements: > $rule_branch->max_holds(5); > $rule_branch->update(); > $rule_branch->max_holds(5); > $rule_branch->insert(); >The last insert will not be done, since the record is already present. >A create should have triggered an error on the primary key. >Obviously, we should use $rule_all. > >Test plan: >Run the test again. > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >--- > t/db_dependent/Holds.t | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/t/db_dependent/Holds.t b/t/db_dependent/Holds.t >index 29ce0c5..69eff35 100755 >--- a/t/db_dependent/Holds.t >+++ b/t/db_dependent/Holds.t >@@ -472,8 +472,8 @@ subtest 'Test max_holds per library/patron category' => sub { > > $rule_branch->max_holds(5); > $rule_branch->update(); >- $rule_branch->max_holds(5); >- $rule_branch->insert(); >+ $rule_all->max_holds(5); >+ $rule_all->insert(); > > $ret = CanItemBeReserved( $borrowernumbers[0], $itemnumber ); > is( $ret, 'OK', 'Patron can place hold with branch/category rule of 5, category rule of 5' ); >-- >2.1.4
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 15524
:
64707
|
64708
|
67180
|
67181
|
67531
|
70389
|
70390
|
70391
|
70392
|
71704
|
71705
|
71706
|
71737
|
71738
|
71739
|
71740
|
71741
|
77648
|
77681
|
77682
|
78134
|
78135
|
78136
|
78137
|
78138
|
78139
|
78140