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

Set price of agreement when sourcing with a framework #39

Merged
merged 5 commits into from
Nov 19, 2014

Conversation

yvaucher
Copy link
Member

No description provided.

@yvaucher yvaucher force-pushed the 8.0-fa_sourcing-set-price branch 2 times, most recently from 274e489 to 45bfc84 Compare November 12, 2014 15:19
@coveralls
Copy link

Coverage Status

Coverage increased (+9.96%) when pulling 45bfc84 on yvaucher:8.0-fa_sourcing-set-price into 19dfec7 on OCA:8.0.

@lepistone
Copy link
Member

@yvaucher this needs review or is it WIP? Can you label it please?

@yvaucher
Copy link
Member Author

@lepistone as travis is failing it is WIP

@gurneyalex
Copy link
Member

rebased

@coveralls
Copy link

Coverage Status

Coverage increased (+9.63%) when pulling 15e8b69 on yvaucher:8.0-fa_sourcing-set-price into 9a91f60 on OCA:8.0.

@gurneyalex
Copy link
Member

with the tests fixed, 👍

@coveralls
Copy link

Coverage Status

Coverage increased (+0.91%) when pulling 033c96c on yvaucher:8.0-fa_sourcing-set-price into 9a91f60 on OCA:8.0.

@@ -45,7 +45,8 @@ def setUp(self):
lid = self.requisition_line_model._generate_source_line(
cr, uid, line)
source_ids += lid
self.assertTrue(len(source_ids) == 2)
self.assertEqual(len(source_ids), 2,
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we really need to specify failure messages with self.assert* assertions.

Here assertEqual will automatically say, for example, that len(source_ids) is 0 != 2, which is a good failure message.

@lepistone
Copy link
Member

👍 with a minor comment on assertions.
@yvaucher ping me for a merge, with or without a change in assertions. Thanks!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.89%) when pulling 5921349 on yvaucher:8.0-fa_sourcing-set-price into 9a91f60 on OCA:8.0.

@yvaucher
Copy link
Member Author

@lepistone fixed

yvaucher and others added 5 commits November 19, 2014 13:18
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling 964e0b2 on yvaucher:8.0-fa_sourcing-set-price into 908dd34 on OCA:8.0.

lepistone added a commit that referenced this pull request Nov 19, 2014
Set price of agreement when sourcing with a framework
@lepistone lepistone merged commit 136217c into OCA:8.0 Nov 19, 2014
@yvaucher yvaucher deleted the 8.0-fa_sourcing-set-price branch May 11, 2015 12:15
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