Bug 12472 - Unit Tests failing with "MySQL server has gone away"
Summary: Unit Tests failing with "MySQL server has gone away"
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Kyle M Hall (khall)
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-24 12:38 UTC by Kyle M Hall (khall)
Modified: 2015-12-03 22:00 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 12472 - Unit Tests failing with "MySQL server has gone away" (2.57 KB, patch)
2014-06-24 12:39 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall (khall) 2014-06-24 12:38:49 UTC
This bug is related to bug 11891 which was recently pushed to master. The issue is that the followup "Bug 11891: (follow-up) remove custom storage class" ( commit id 967f5cb51321a6a08176c7718a6ab933b3b5f926 ) removes the custom storage class that stopped DBIx::Class from destroying the dbh handle when it was done using it.
Comment 1 Kyle M Hall (khall) 2014-06-24 12:39:34 UTC
Created attachment 29190 [details] [review]
Bug 12472 - Unit Tests failing with "MySQL server has gone away"

This bug is related to bug 11891 which was recently pushed to master.
The issue is that the followup "Bug 11891: (follow-up) remove custom
storage class" ( commit id 967f5cb51321a6a08176c7718a6ab933b3b5f926 )
removes the custom storage class that stopped DBIx::Class from
destroying the dbh handle when it was done using it.

Test Plan:
1) Run 'perl t/db_dependent/Circulation.t'
2) Note the error
   DBD::mysql::st execute failed: MySQL server has gone away at
   /home/koha/kohaclone/C4/Members.pm line 881.
3) Apply this patch
4) Re-run 'perl t/db_dependent/Circulation.t'
5) No errors!
Comment 2 Jonathan Druart 2014-07-09 15:49:01 UTC
I don't reproduce the error.
Comment 3 Kyle M Hall (khall) 2014-08-07 20:31:28 UTC
I believe the requirement for a newer version of dbic has made this patch obsolete.