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

[FIX] product_contract: Fix mock usage in tests #72

Merged

Conversation

lasley
Copy link
Contributor

@lasley lasley commented May 26, 2017

This fixes #71 (comment) by @pedrobaeza

@lasley lasley force-pushed the bugfix/10.0/product_contract-mock-test branch 3 times, most recently from 7775525 to ecd6b2d Compare May 26, 2017 01:21
@lasley lasley force-pushed the bugfix/10.0/product_contract-mock-test branch from ecd6b2d to 86a5b11 Compare May 26, 2017 01:26
@rafaelbn rafaelbn added this to the 10.0 milestone Jun 1, 2017
Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @lasley

@@ -20,6 +20,11 @@ def setUp(self):
self.product.product_tmpl_id.is_contract = True
self.product.product_tmpl_id.contract_template_id = self.contract.id

def tearDown(self):
self.env['account.analytic.account']._revert_method(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this do? 🤔

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -20,6 +20,11 @@ def setUp(self):
self.product.product_tmpl_id.is_contract = True
self.product.product_tmpl_id.contract_template_id = self.contract.id

def tearDown(self):
self.env['account.analytic.account']._revert_method(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pedrobaeza pedrobaeza merged commit d633e93 into OCA:10.0 Jun 1, 2017
@lasley lasley deleted the bugfix/10.0/product_contract-mock-test branch June 1, 2017 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants