-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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] add round off difference to last row in landed cost voucher #8989
Conversation
pr.submit() | ||
|
||
submit_landed_cost_voucher("Purchase Receipt", pr.name, 123.22) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a assertEqual
statement as well, to compare the distributed charges value.
lcv = submit_landed_cost_voucher("Purchase Receipt", pr.name, 123.22) | ||
|
||
self.assertEquals(lcv.items[0].applicable_charges, 41.07) | ||
self.assertEquals(lcv.items[2].applicable_charges, 41.07999999999999) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be 42.08. What is the precision of the applicable_charges field?
…t_issue * 'develop' of https://github.com/frappe/erpnext: bumped to version 8.0.40 [fix] uppercase filter frappe#8996 (frappe#9026) Financial Statements Button translateabled (frappe#9032) [minor] added get_terms methods to erpnext.utils so that it can be used in non transactional documents (frappe#9037) Party Dashboard: Consider any random company if default company not set (frappe#9046) [fix] add round off difference to last row in landed cost voucher (frappe#8989) Update code to fix issue #erpnext WN-SUP25349 : ValueError: max() arg is an empty sequence. (frappe#9025) add milestone field
No description provided.