Summary: | Fix t/db_dependent/www/batch.t so it matches new layout | ||
---|---|---|---|
Product: | Koha | Reporter: | Tomás Cohen Arazi <tomascohen> |
Component: | Test Suite | Assignee: | Tomás Cohen Arazi <tomascohen> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | normal | ||
Priority: | P5 - low | CC: | bgkriegel, kyle |
Version: | master | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | |||
Bug Blocks: | 18807 | ||
Attachments: |
Bug 16423: Fix t/db_dependent/www/batch.t so it matches new layout
Bug 16423: (followup) Add more tests [SIGNED-OFF] Bug 16423: Fix t/db_dependent/www/batch.t so it matches new layout [SIGNED-OFF] Bug 16423: (followup) Add more tests Bug 16423: Fix t/db_dependent/www/batch.t so it matches new layout Bug 16423: (followup) Add more tests Bug 16423: [QA Follow-up] Remove $opac |
Description
Tomás Cohen Arazi
2016-05-02 20:30:29 UTC
Created attachment 51116 [details] [review] Bug 16423: Fix t/db_dependent/www/batch.t so it matches new layout This patch adjusts batch.t so it matches the use of datatables on the 'Manage staged records' page, and small layout changes already fixed on search_utf8.t. The tests are slightly modified so they actually test interesting stuff. Some were passing only because an undefined value was passed. To test: - On master, run $ prove t/db_dependent/www/batch.t => FAIL: Tests fail notably - Apply the patch - Run: $ prove t/db_dependent/www/batch.t => SUCCESS: Notice tests now pass. - Sign off :-D Created attachment 51117 [details] [review] Bug 16423: (followup) Add more tests On fixing this tests, we retrieved the JSON data for the datatable used on the 'Manage staged records' page. It would be cool to check the data it carries makes sense. To test: - Run t/db_dependent/www/batch.t => SUCCESS: Tests pass - Sign off :-D Before patch I found not ok 7 - upload succeeded and this message malformed JSON string, neither array, object, number, string or atom, at character offset 0 (before "<!DOCTYPE html>\n<!-...") at t/db_dependent/www/batch.t line 104. After patch, test 7 pass and no message, but test 23 fails not ok 23 - bib is gone # Failed test 'bib is gone' # at t/db_dependent/www/batch.t line 201. # searched: "\x{0a}\x{0a}\x{0a}\x{0a}\x{0a}\x{0a}\x{0a}\x{0a}<!DOCTYPE html>\x{0a}<!-- TEMPLATE FILE: detail"... # can't find: "The record you requested does not exist" # LCSS: " record y" # LCSS context: "ave been uploaded for this bibliographic record ye" Is that on current master?? (In reply to Tomás Cohen Arazi from comment #4) > Is that on current master?? Yes. a7446eb Bug 16423: (followup) Add more tests c802f5b Bug 16423: Fix t/db_dependent/www/batch.t so it matches new layout 4d0d52e Revert "Bug 16104: Remove warnings "used only once: possible typo"" b20ba02 Bug 15533 - DBRev - Update OldReserve schema file to match database schema 68dbe84 Bug 14632: Add Copyright for the Koha Dev Team .... Passes with me (one warning): 1..24 Use of uninitialized value $opac in substitution (s///) at t/db_dependent/www/batch.t line 55. ok 1 - connect to intranet ok 2 - login to staff client ok 3 - load main page ok 4 - open tools module ok 5 - go to stage MARC ok 6 - uploaded file ok 7 - upload succeeded ok 8 - reopen stage MARC page ok 9 - stage MARC ok 10 - have job ID ok 11 - job was completed ok 12 - reopen stage MARC page at end of upload ok 13 - stage MARC ok 14 - view batch ok 15 - get the datatable for the new batch id ok 16 - found book ok 17 - record marked as staged ok 18 - record has no matches ok 19 - imported records into catalog ok 20 - record marked as imported ok 21 - revert import ok 22 - getting reverted bib ok 23 - bib is gone ok 24 - record marked as reverted Created attachment 51149 [details] [review] [SIGNED-OFF] Bug 16423: Fix t/db_dependent/www/batch.t so it matches new layout This patch adjusts batch.t so it matches the use of datatables on the 'Manage staged records' page, and small layout changes already fixed on search_utf8.t. The tests are slightly modified so they actually test interesting stuff. Some were passing only because an undefined value was passed. To test: - On master, run $ prove t/db_dependent/www/batch.t => FAIL: Tests fail notably - Apply the patch - Run: $ prove t/db_dependent/www/batch.t => SUCCESS: Notice tests now pass. - Sign off :-D Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Test pass Created attachment 51150 [details] [review] [SIGNED-OFF] Bug 16423: (followup) Add more tests On fixing this tests, we retrieved the JSON data for the datatable used on the 'Manage staged records' page. It would be cool to check the data it carries makes sense. To test: - Run t/db_dependent/www/batch.t => SUCCESS: Tests pass - Sign off :-D Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> All test pass prove t/db_dependent/www/batch.t t/db_dependent/www/batch.t .. ok All tests successful. Created attachment 51151 [details] [review] Bug 16423: Fix t/db_dependent/www/batch.t so it matches new layout This patch adjusts batch.t so it matches the use of datatables on the 'Manage staged records' page, and small layout changes already fixed on search_utf8.t. The tests are slightly modified so they actually test interesting stuff. Some were passing only because an undefined value was passed. To test: - On master, run $ prove t/db_dependent/www/batch.t => FAIL: Tests fail notably - Apply the patch - Run: $ prove t/db_dependent/www/batch.t => SUCCESS: Notice tests now pass. - Sign off :-D Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Test pass Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 51152 [details] [review] Bug 16423: (followup) Add more tests On fixing this tests, we retrieved the JSON data for the datatable used on the 'Manage staged records' page. It would be cool to check the data it carries makes sense. To test: - Run t/db_dependent/www/batch.t => SUCCESS: Tests pass - Sign off :-D Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> All test pass prove t/db_dependent/www/batch.t t/db_dependent/www/batch.t .. ok All tests successful. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 51155 [details] [review] Bug 16423: [QA Follow-up] Remove $opac It is not used. And even may raise an uninitialized warnings. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Test still passes :) Pushed to master for Koha 16.05, thanks Marcel, Tomas! |