openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~numerigraphe-team/ocb-addons/7.0-inventory-dummy-moves-test into lp:ocb-addons

To: mp+215453@xxxxxxxxxxxxxxxxxx
From: "Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>
Date: Mon, 28 Apr 2014 12:19:03 -0000
In-reply-to: <20140411161722.29653.20939.launchpad@ackee.canonical.com>
Reply-to: mp+215453@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Information

Looks good in general, but I'm slightly puzzled the exception handling. Is it know when it raises and when it doesn't? Doesn't the occurrence of the exception render the test meaningless? And does that not make the whole excercise moot?

-- 
https://code.launchpad.net/~numerigraphe-team/ocb-addons/7.0-inventory-dummy-moves-test/+merge/215453
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons.

References

[Merge] lp:~numerigraphe-team/ocb-addons/7.0-inventory-dummy-moves-test into lp:ocb-addons
From: Lionel Sausin - Numérigraphe, 2014-04-11