Bugzilla – Attachment 35334 Details for
Bug 13554
Bug 12969 introduces a test case that fails
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 13554: t/Prices.t shouldn't depend on the DB
PASSED-QA-Bug-13554-tPricest-shouldnt-depend-on-th.patch (text/plain), 1.31 KB, created by
Kyle M Hall (khall)
on 2015-01-16 16:51:09 UTC
(
hide
)
Description:
[PASSED QA] Bug 13554: t/Prices.t shouldn't depend on the DB
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2015-01-16 16:51:09 UTC
Size:
1.31 KB
patch
obsolete
>From 79e04c13c82e6716f839e08b2dd9f33a9a8c2a0f Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Mon, 12 Jan 2015 09:35:51 +0100 >Subject: [PATCH] [PASSED QA] Bug 13554: t/Prices.t shouldn't depend on the DB > >To reproduce: >- Stop your MySQL server: > $ sudo service mysql stop >- Run > $ prove t/Prices.t >=> FAIL: some tests fail because of mysql stopped > >To test (MySQL still stopped) >- Apply the patch >- Run > $ prove t/Prices.t > >Signed-off-by: Paola Rossi <paola.rossi@cineca.it> > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > t/Prices.t | 16 +++++++++------- > 1 files changed, 9 insertions(+), 7 deletions(-) > >diff --git a/t/Prices.t b/t/Prices.t >index 3dc6c54..92bb432 100644 >--- a/t/Prices.t >+++ b/t/Prices.t >@@ -1,15 +1,17 @@ > use Modern::Perl; >-use Test::More tests => 8; >+use Test::More tests => 12; > use Test::MockModule; > >-use C4::Acquisition; >-use C4::Bookseller; >-use C4::Context; >- >-use Koha::Number::Price; >- > use t::lib::Mocks; > >+BEGIN { >+ my $context_module = t::lib::Mocks::mock_dbh; >+ use_ok('C4::Acquisition'); >+ use_ok('C4::Bookseller'); >+ use_ok('C4::Context'); >+ use_ok('Koha::Number::Price'); >+}; >+ > t::lib::Mocks::mock_preference( 'gist', '0.02|0.05|0.196' ); > > my $bookseller_module = Test::MockModule->new('Koha::Acquisition::Bookseller'); >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 13554
:
35099
|
35105
| 35334