Bug 26143: Regression tests
authorTomas Cohen Arazi <tomascohen@theke.io>
Wed, 5 Aug 2020 15:04:47 +0000 (12:04 -0300)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 5 Aug 2020 15:36:28 +0000 (17:36 +0200)
commitd20bc39d759bc9321b1b7accb952f90351a14caa
tree02d0d931cc10f15d6be46834fb41dfab8a81ca17
parent61e8dc8c7d489329c27aca3b6b10137c289161cb
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>
t/Koha/REST/Plugin/Pagination.t
t/db_dependent/Koha/REST/Plugin/Objects.t