]> git.koha-community.org Git - koha.git/commit
Bug 26143: Regression tests
authorTomas Cohen Arazi <tomascohen@theke.io>
Wed, 5 Aug 2020 15:04:47 +0000 (12:04 -0300)
committerLucas Gass <lucas@bywatersolutions.com>
Thu, 13 Aug 2020 18:11:56 +0000 (18:11 +0000)
commit9a4910812f03fade73d27db8ce975dc0a15fe9e7
treed18d97e058afbd6dd6057ac9411c30dbc8c9c00b
parentdd3c09e8486eb79f709dd5ea484f1c09c1805289
Bug 26143: Regression tests

This patch introduces tests for the per_page=-1 handling use case. From
now on per_page=-1 means 'all resources'.

On writing this I noticed that we always paginate results no matter
what, but there was a weird condition under which on pagination headers
were sent back to the API consumer. This is highlighted in the precedent
patch, which is not the -1 situation this one tries to tackle.

Both pagination and searching are broken with per_page=-1, which is a
standard, and we actually didn't explicitly set a way to request all
resources.

To verify this:
1. Apply the previous tests patch and this one
2. Run:
   $ kshell
  k$ prove t/Koha/REST/Plugin/Pagination.t \
           t/db_dependent/Koha/REST/Plugin/Objects.t
=> FAIL: Things are damn broken

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit d20bc39d759bc9321b1b7accb952f90351a14caa)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
t/Koha/REST/Plugin/Pagination.t
t/db_dependent/Koha/REST/Plugin/Objects.t