[Bug 21787] New: GetHardDueDate.t has a silly test
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Bug ID: 21787 Summary: GetHardDueDate.t has a silly test Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Test Suite Assignee: jonathan.druart@bugs.koha-community.org Reporter: jonathan.druart@bugs.koha-community.org QA Contact: testopia@bugs.koha-community.org When a new column is added to the issuingrules table, one of the test from GetHardDueDate.t is failing. As it does not test anything useful we should remove it instead of fixing it. See commit face7819393b17a3fb4bab6f8c2ff26cc9a450d8 Author: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Date: Thu Nov 8 11:15:04 2018 -0300 Bug 12365: Fix GetHardDueDate.t test diff --git a/t/db_dependent/Circulation/GetHardDueDate.t b/t/db_dependent/Circulation/GetHardDueDate.t index a50e06b..bfb624d 100644 --- a/t/db_dependent/Circulation/GetHardDueDate.t +++ b/t/db_dependent/Circulation/GetHardDueDate.t @@ -148,6 +148,7 @@ my $sampleissuingrule1 = { no_auto_renewal_after_hard_limit => undef, suspension_chargeperiod => 1, holds_per_day => undef, + note => undef, }; my $sampleissuingrule2 = { branchcode => $samplebranch2->{branchcode}, commit 1ce984ea1dd12391d39507cd21c4d3813eab9754 Author: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Date: Wed Oct 24 17:08:05 2018 -0300 Bug 15486: Fix failing test in GetHardDueDate.t Signed-off-by: Nick Clemens <nick@bywatersolutions.com> diff --git a/t/db_dependent/Circulation/GetHardDueDate.t b/t/db_dependent/Circulation/GetHardDueDate.t index b39c32c..a50e06b 100644 --- a/t/db_dependent/Circulation/GetHardDueDate.t +++ b/t/db_dependent/Circulation/GetHardDueDate.t @@ -147,6 +147,7 @@ my $sampleissuingrule1 = { no_auto_renewal_after => undef, no_auto_renewal_after_hard_limit => undef, suspension_chargeperiod => 1, + holds_per_day => undef, }; my $sampleissuingrule2 = { branchcode => $samplebranch2->{branchcode}, commit f32cf00e132c747a008163e5f40810d4a47bff58 Author: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Date: Mon Apr 9 17:33:37 2018 -0300 Bug 19804: Fix test in GetHardDueDate.t Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> diff --git a/t/db_dependent/Circulation/GetHardDueDate.t b/t/db_dependent/Circulation/GetHardDueDate.t index b5a571b..b39c32c 100644 --- a/t/db_dependent/Circulation/GetHardDueDate.t +++ b/t/db_dependent/Circulation/GetHardDueDate.t @@ -146,6 +146,7 @@ my $sampleissuingrule1 = { article_requests => 'yes', no_auto_renewal_after => undef, no_auto_renewal_after_hard_limit => undef, + suspension_chargeperiod => 1, }; my $sampleissuingrule2 = { branchcode => $samplebranch2->{branchcode}, -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 --- Comment #1 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 82091 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82091&action=edit Bug 21787: Remove unecessary test from GetHardDueDate.t See description on the bug report for more info. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #82091|0 |1 is obsolete| | --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 82137 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82137&action=edit Bug 21787: Remove unecessary test from GetHardDueDate.t See description on the bug report for more info. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |tomascohen@gmail.com --- Comment #3 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed for 18.11. Thanks all! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |martin.renvoize@ptfs-europe | |.com --- Comment #4 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Pushed to 18.05.x for 18.05.06 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21787 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Stable |RESOLVED CC| |fridolin.somers@biblibre.co | |m Resolution|--- |FIXED --- Comment #5 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Pushed to 17.11.x for 17.11.13 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org