Code review comment for lp:~elbati/account-payment/adding_account_voucher_supplier_invoice_number_7

Revision history for this message
Lorenzo Battistini (elbati) wrote :

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

« Back to merge proposal