The first patch is perfect ! The wizard is now computing the things correctly and all my tests case work well (except the validate a non-balanced entry) : Good !
But the second one breaks part of what was done with the first one...
I give you here what breaks with the second patch (you can test it by yourself using OERPScenario, stable branch):
Background :
Given I am loged as admin user with password admin used
And the company currency is set to EUR
And the following currency rate settings are:
|code|rate|name|
|EUR|1.000|01-01-2009|
|CHF|1.644|01-01-2009|
|CHF|1.500|09-09-2009|
|CHF|0.6547|10-10-2009|
|USD|1.3785|01-01-2009|
And a cash journal in USD exists
And a cash journal in CHF exists
And a cash journal in EUR exists
Scenario: make_and_validate_payments_with_pay_invoice_wizard
Given I have recorded on the 1 jan 2009 a supplier invoice (in_invoice) of 1000,0 CHF without tax called MySupplierInvoicePayWizard
When I press the validate button
Then I should see the invoice MySupplierInvoicePayWizard open
When I call the Pay invoice wizard
And I partially pay 200.0 CHF.- on the 10 jan 2009
Then I should see a residual amount of 800.0 CHF.-
When I call the Pay invoice wizard
And I partially pay 200.0 USD.- on the 11 jan 2009
Then I should see a residual amount of 561.48 CHF.-
When I call the Pay invoice wizard
And I partially pay 200.0 EUR.- on the 12 jan 2009
Then I should see a residual amount of 232.68 CHF.-
When I call the Pay invoice wizard
And I completely pay the residual amount in CHF on the 13 sep 2009
Then I should see a residual amount of 0.0 CHF.-
expected: 0.0,
got: -9.75 (using ==)
Diff:
@@ -1,2 +1,2 @@
-0.0
+-9.75 (Spec::Expectations::ExpectationNotMetError)
./features/account/wizard/step_definitions/wizard_pay_invoice.rb:78:in `/^I should see a residual amount of (.*) (\w+)\.\-$/'
features/account/wizard/pay_invoice.feature:34:in `Then I should see a residual amount of 0.0 CHF.-'
And I should see the invoice MySupplierInvoicePayWizard paid
Hi,
The first patch is perfect ! The wizard is now computing the things correctly and all my tests case work well (except the validate a non-balanced entry) : Good !
But the second one breaks part of what was done with the first one...
I suggest the following :
- Commit the first patch /bugs.launchpad .net/openobject -addons/ +bug/452854 to solve what the second patch try to address.
- Close this bug report as it's fixed.
- Use https:/
I give you here what breaks with the second patch (you can test it by yourself using OERPScenario, stable branch):
Background : code|rate| name| EUR|1.000| 01-01-2009| CHF|1.644| 01-01-2009| CHF|1.500| 09-09-2009| CHF|0.6547| 10-10-2009| USD|1.3785| 01-01-2009|
Given I am loged as admin user with password admin used
And the company currency is set to EUR
And the following currency rate settings are:
|
|
|
|
|
|
And a cash journal in USD exists
And a cash journal in CHF exists
And a cash journal in EUR exists
Scenario: make_and_ validate_ payments_ with_pay_ invoice_ wizard cePayWizard cePayWizard open
Given I have recorded on the 1 jan 2009 a supplier invoice (in_invoice) of 1000,0 CHF without tax called MySupplierInvoi
When I press the validate button
Then I should see the invoice MySupplierInvoi
When I call the Pay invoice wizard
And I partially pay 200.0 CHF.- on the 10 jan 2009
Then I should see a residual amount of 800.0 CHF.-
When I call the Pay invoice wizard
And I partially pay 200.0 USD.- on the 11 jan 2009
Then I should see a residual amount of 561.48 CHF.-
When I call the Pay invoice wizard
And I partially pay 200.0 EUR.- on the 12 jan 2009
Then I should see a residual amount of 232.68 CHF.-
When I call the Pay invoice wizard
And I completely pay the residual amount in CHF on the 13 sep 2009
Then I should see a residual amount of 0.0 CHF.-
expected: 0.0,
got: -9.75 (using ==)
Diff:
(Spec:: Expectations: :ExpectationNot MetError) /features/ account/ wizard/ step_definition s/wizard_ pay_invoice. rb:78:in `/^I should see a residual amount of (.*) (\w+)\.\-$/' account/ wizard/ pay_invoice. feature: 34:in `Then I should see a residual amount of 0.0 CHF.-'
@@ -1,2 +1,2 @@
-0.0
+-9.75
.
features/
And I should see the invoice MySupplierInvoi cePayWizard paid