[Koha-bugs] [Bug 15522] New interface for revamped circulation rules

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15522

--- Comment #100 from Lari Taskula <lari.taskula at hypernova.fi> ---
(In reply to Lari Taskula from comment #98)
> (In reply to Lisette Scheer from comment #62)
> > (In reply to Jonathan Druart from comment #51)
> > > Created attachment 99148 [details] [review] [review] [review]
> > > Bug 15522: Add API for circulation rules
> > > 
> > > Test plan:
> > >   1) Run t/db_dependent/api/v1/circ_rules.t .
> > 
> > kohadev-koha at kohadevbox:/home/vagrant/kohaclone$ prove
> > t/db_dependent/api/v1/circ_rules.t
> > t/db_dependent/api/v1/circ_rules.t .. 1/5
> >     #   Failed test '403 Forbidden'
> >     #   at t/db_dependent/api/v1/circ_rules.t line 128.
> >     #          got: '400'
> >     #     expected: '403'
> >     # Looks like you failed 1 test of 2.
> > t/db_dependent/api/v1/circ_rules.t .. 3/5
> > #   Failed test 'set_rules | unauthorized'
> > #   at t/db_dependent/api/v1/circ_rules.t line 132.
> > 
> >     #   Failed test '200 OK'
> >     #   at t/db_dependent/api/v1/circ_rules.t line 212.
> >     #          got: '403'
> >     #     expected: '200'
> >     # Looks like you failed 1 test of 6.
> > 
> > #   Failed test 'set_rules | restricted'
> > #   at t/db_dependent/api/v1/circ_rules.t line 223.
> > # Looks like you failed 2 tests of 5.
> > t/db_dependent/api/v1/circ_rules.t .. Dubious, test returned 2 (wstat 512,
> > 0x200)
> > Failed 2/5 subtests
> > 
> > Test Summary Report
> > -------------------
> > t/db_dependent/api/v1/circ_rules.t (Wstat: 512 Tests: 5 Failed: 2)
> >   Failed tests:  3, 5
> >   Non-zero exit status: 2
> > Files=1, Tests=5,  4 wallclock secs ( 0.02 usr  0.00 sys +  2.86 cusr  0.23
> > csys =  3.11 CPU)
> > Result: FAIL
> > 
> > I got this message when I tried to do this step in my devbox. Everything
> > else worked great for me.
> 
> I confirm this error. Will provide a patch to fix it.

Already fixed by "Bug 15522: Fix permission name
manage_circ_rules_from_any_libraries", maybe squash this patch to the first
one.

-- 
You are receiving this mail because:
You are watching all bug changes.