On 05/24/2013 07:12 PM, Stefan Rijnhart (Therp) wrote:
> Looks good, functionally. I am just wondering, why did you include the whole test suite of the voucher module, given that this module hardly influences any calculation or move line creation?
To be sure this module does not influence any calculation or move line
creation :-)
Anyway, I can remove some of them, if unnecessary.
On 05/24/2013 07:12 PM, Stefan Rijnhart (Therp) wrote:
> Looks good, functionally. I am just wondering, why did you include the whole test suite of the voucher module, given that this module hardly influences any calculation or move line creation?
To be sure this module does not influence any calculation or move line
creation :-)
Anyway, I can remove some of them, if unnecessary.
> Nit: l.2866 could be kept at < 80 chars
>
Done