Bug 18763

Summary: swagger/definitions.t is failing
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Galen Charlton <gmcharlt>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, katrin.fischer, ldjamison, mtompset
Version: master   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18508
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 18763: Fix swagger/definitions.t tests
[SIGNED OFF] Bug 18763: Fix swagger/definitions.t

Description Jonathan Druart 2017-06-08 19:28:23 UTC
#   Failed test 'Columns are missing for patron: lang, overdrive_auth_token, login_attempts'
Comment 1 Jonathan Druart 2017-06-08 19:29:15 UTC
Created attachment 64118 [details] [review]
Bug 18763: Fix swagger/definitions.t tests

Test plan:
  prove api/v1/swagger/definitions/patron.json
must return green
Comment 2 Lee Jamison 2017-06-09 00:18:35 UTC
Created attachment 64119 [details] [review]
[SIGNED OFF] Bug 18763: Fix swagger/definitions.t

Test plan:
  prove api/v1/swagger/definitions/patron.json
must return green

Signed-off-by: Lee Jamison <ldjamison@marywood.edu>
swagger/definitions.t test returns green
Comment 3 Mark Tompsett 2017-06-14 04:23:23 UTC
For what it's worth, I git bz'd 18508 and then 18763. I had to tweak the git bz, but basically both combined fix noise and test failures.
Comment 4 Jonathan Druart 2017-06-14 17:11:17 UTC
Skipping QA.
Comment 5 Jonathan Druart 2017-06-14 20:39:47 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 6 Fridolin Somers 2017-06-20 08:23:45 UTC
Pushed to 17.05.x, will be in 17.05.01

Test plan says :
  prove api/v1/swagger/definitions/patron.json
It is :
  prove t/db_dependent/api/v1/swagger/definitions.t
Comment 7 Katrin Fischer 2017-06-21 06:24:44 UTC
Test fails for me locally, but columns listed are not in 16.11.x - please provide a separate patch for this version if it's needed.
Comment 8 Katrin Fischer 2017-06-21 06:25:05 UTC
Test Summary Report
-------------------
api/v1/swagger/definitions/patron.json (Wstat: 65280 Tests: 0 Failed: 0)
  Non-zero exit status: 255
  Parse errors: No plan found in TAP output
Files=1, Tests=0,  0 wallclock secs ( 0.01 usr +  0.01 sys =  0.02 CPU)
Result: FAIL
kohadev-koha@kohadevbox:/home/vagrant/kohaclone$ prove api/v1/swagger/definitions/patron.json
api/v1/swagger/definitions/patron.json .. syntax error at api/v1/swagger/definitions/patron.json line 2, near ""type":"
Execution of api/v1/swagger/definitions/patron.json aborted due to compilation errors.
api/v1/swagger/definitions/patron.json .. Dubious, test returned 255 (wstat 65280, 0xff00)
No subtests run
Comment 9 Katrin Fischer 2017-09-10 09:18:39 UTC
Won't apply to 16.11.x as is, closing!