On 13 May 14:58, Cédric Krier wrote: > On 13 May 13:48, Mathias Behrle wrote: > > > There is a problem with the new tests added to purchase (see > > > http://tests.tryton.org/~test/postgresql.html). > > > The test fails when running with account_stock_continental because this > > > module add a constraint on product account stock. > > > > > > I wrote a patch: http://codereview.tryton.org/13311003 > > > but I'm not sure it is the right way to go. > > > An other possibility would be to remove the required of > > > account_stock_continental for bad design reason (there are still the > > > *_used which raise error message). > > > > > > What do you think? > > > > What about testing in tests/test_purchase.py, if module > > account_stock_continental is installed and just in case provide the needed > > values? > > Basicly it is similar solution to my patch but as explained in previous > email, I think the issue is in the design of account_stock_continental.
I created an issue for this: https://bugs.tryton.org/issue3962 And the codereview: http://codereview.tryton.org/5381002 -- Cédric Krier - B2CK SPRL Email/Jabber: cedric.kr...@b2ck.com Tel: +32 472 54 46 59 Website: http://www.b2ck.com/
pgpWeP1MyE6vp.pgp
Description: PGP signature
