Bug 13282 - t/Message.t shouldn't depend on the DB
Summary: t/Message.t shouldn't depend on the DB
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Test Suite (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Chris Cormack
QA Contact: Galen Charlton
URL:
Keywords:
Depends on:
Blocks: 13275
  Show dependency treegraph
 
Reported: 2014-11-18 08:13 UTC by Magnus Enger
Modified: 2015-06-04 23:33 UTC (History)
1 user (show)

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


Attachments
Bug 13282: t/Message.t shouldn't depend on the DB (1.42 KB, patch)
2014-11-19 21:08 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 13282: t/Message.t shouldn't depend on the DB (1.47 KB, patch)
2014-11-20 00:34 UTC, Robin Sheat
Details | Diff | Splinter Review
Bug 13282: t/Message.t shouldn't depend on the DB (1.47 KB, patch)
2014-11-20 09:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 13282: t/Message.t shouldn't depend on the DB (1.54 KB, patch)
2014-11-20 09:59 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Magnus Enger 2014-11-18 08:13:24 UTC
With MySQL turned off:

$ sudo -u kohadev-koha KOHA_CONF=/etc/koha/sites/kohadev/koha-conf.xml prove t/Message.t 
t/Message.t .. DBI connect('dbname=koha_kohadev;host=localhost;port=3306','koha_kohadev',...) failed: Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2) at C4/Context.pm line 785
t/Message.t .. 1/1 
#   Failed test 'use C4::Message;'
#   at t/Message.t line 12.
#     Tried to use 'C4::Message'.
#     Error:  Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2) at C4/Context.pm line 785.
# Compilation failed in require at C4/Letters.pm line 27.
# BEGIN failed--compilation aborted at C4/Letters.pm line 27.
# Compilation failed in require at C4/Message.pm line 25.
# BEGIN failed--compilation aborted at C4/Message.pm line 25.
# Compilation failed in require at t/Message.t line 12.
# BEGIN failed--compilation aborted at t/Message.t line 12.
# Looks like you failed 1 test of 1.
t/Message.t .. Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests 

Test Summary Report
-------------------
t/Message.t (Wstat: 256 Tests: 1 Failed: 1)
  Failed test:  1
  Non-zero exit status: 1
Files=1, Tests=1,  1 wallclock secs ( 0.00 usr  0.02 sys +  0.68 cusr  0.48 csys =  1.18 CPU)
Result: FAIL
Comment 1 Tomás Cohen Arazi 2014-11-19 21:08:21 UTC Comment hidden (obsolete)
Comment 2 Robin Sheat 2014-11-20 00:34:14 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-11-20 09:58:02 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-11-20 09:59:51 UTC
Created attachment 33737 [details] [review]
Bug 13282: t/Message.t shouldn't depend on the DB

Signed-off-by: Robin Sheat <robin@catalyst.net.nz>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 5 Tomás Cohen Arazi 2014-11-20 12:49:57 UTC
Patch pushed to master.