Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.0] sale_exception_nostock tests fail #37

Closed
eLBati opened this issue Aug 20, 2014 · 4 comments
Closed

[7.0] sale_exception_nostock tests fail #37

eLBati opened this issue Aug 20, 2014 · 4 comments

Comments

@eLBati
Copy link
Member

eLBati commented Aug 20, 2014

See OCA/sale-workflow#22 (comment)

sale_exception_nostock OCB tests fail with Incorrect state manual instead of draft.

When the test 'Then I confirm the the 26th of March firesteel order' confirms SO so_1

  • Using OCB, at the end of future_orders_are_affected, prod_for_virtual_qty['virtual_available'] is 11000 and line.product_uom_qty is 10000. So, no exception is raised and the order is confirmed. The test fails because it expects the exception --> the order should stay draft.
  • Using odoo, at the end of future_orders_are_affected, prod_for_virtual_qty['virtual_available'] is 500 and line.product_uom_qty is 10000. So, the exception is raised and the order stays draft --> the test doesn't fail

Did not investigate further, so I don't know who is right between OCB and odoo about the virtual_available amount

@gurneyalex
Copy link
Member

I've used git bisect to find which commit in OCB causes the failure. It turns out it is 5d795c4 which changed the date of a delivery order generated by a SO from date_order of the SO to date_confirm.

@gurneyalex
Copy link
Member

I'll try to come up with a version of the test working in both branches.

@pedrobaeza
Copy link
Member

Thanks for investigating it!

@eLBati
Copy link
Member Author

eLBati commented Sep 12, 2014

I close this thanks to OCA/sale-workflow#40

@eLBati eLBati closed this as completed Sep 12, 2014
mistotebe pushed a commit to credativUK/OCB that referenced this issue Oct 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants