Bug 17836: (ILSDI) Make GetPatronInfo fill 'charges' correctly
authorTomas Cohen Arazi <tomascohen@theke.io>
Tue, 3 Jan 2017 16:14:25 +0000 (13:14 -0300)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 20 Jan 2017 13:44:56 +0000 (13:44 +0000)
commit13923e7b0598b09d21647558f42ca54c31abf470
tree92d67d1ccc9ca84044bdab13fa2aebd45522ca58
parentcd771f36867cab33ad74e47234eac5a7782ffab2
Bug 17836: (ILSDI) Make GetPatronInfo fill 'charges' correctly

This trivial fix corrects a typo on C4/ILSDI/Services.pm.
It was hidden because the tests for ILSDI only cover the 'attributes'
portion of the response. I added regression tests for this.

To test:
- Have the regression test patch applied
- Run:
  $ prove t/db_dependent/ILSDI_Services.t
=> FAIL: Tests fail because 'charges' is always set to 1
- Apply the patch
- Run:
  $ prove t/db_dependent/ILSDI_Services.t
=> SUCCESS: Tests pass
- Sign off :-D

Sponsored-by: ByWater Solutions

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
C4/ILSDI/Services.pm