[Koha-bugs] [Bug 9053] Test::MockModule and DBD::Mock should be required

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9053

--- Comment #4 from Jared Camins-Esakov <jcamins at cpbibliography.com> ---
(In reply to comment #3)
> QA comment:
> 
> The patch is correct but I don't understand where stands the boundary
> between "required" and "not required". What is the idea behind this patch?
> 
> To me, launching tests is not useful for a fonctionnal use of Koha.
> If these 2 modules are required perhaps should we add the others Test::* as
> required, too?

These two modules are required for the tests in t/ to pass -- which is
reasonable -- and running the tests in t/ is part of the installation process.
I don't know that it's necessary to make all Test modules required, though I
don't know that there's any reason *not* to make all Test modules required.
Installing the Test modules is not a great hardship, and I'd rather new users
know if there's a problem with the version of Koha that they're installing.

-- 
You are receiving this mail because:
You are watching all bug changes.