[18.0] [MIG] contract_sale_mandate

[samirGuesmi]

[rousseldenis]

/ocabot migration contract_sale_mandate

[rousseldenis]

@samirGuesmi Be careful, you mixed other changes in your commit for test-requirements.txt :

[rousseldenis]

To help reviewers, it is better to put links to depending PRs in this description.

[yvaucher]

One dependency to be merged to go. (added the list to the description)

[qgroulard]

@samirGuesmi The last dependency has been merged, let's finish this work today :muscle:

[samirGuesmi]

Thanks all, here we are.

[yvaucher]

There is still one issue with the tests:

odoo.addons.contract_sale_mandate.tests.test_contract_sale_mandate: FAIL: TestContractSaleMandate.test_01
Traceback (most recent call last):
  File "/__w/contract/contract/contract_sale_mandate/tests/test_contract_sale_mandate.py", line 61, in test_01
    self.assertEqual(contracts[0].mandate_id, self.mandate)
AssertionError: account.banking.mandate() != account.banking.mandate(3,)

[samirGuesmi]

@yvaucher Done

[qgroulard]

@rousseldenis Could you please update your review ?

[yvaucher]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 18.0-ocabot-merge-pr-1246-by-yvaucher-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 637116bf3005c697f5ed7f21fc4ba86720542859. Thanks a lot for contributing to OCA. ❤️