Skip to content

[16.0][FIX] account_payment_method_fs_storage: improve test by removing usage of datas#1430

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
acsone:16.0-fix-test-account-payment-method-fs-storage
Mar 13, 2025
Merged

[16.0][FIX] account_payment_method_fs_storage: improve test by removing usage of datas#1430
OCA-git-bot merged 1 commit intoOCA:16.0from
acsone:16.0-fix-test-account-payment-method-fs-storage

Conversation

@AnizR
Copy link
Copy Markdown

@AnizR AnizR commented Mar 13, 2025

No modification in the behavior, simply a correction in the test suite.

The usage of data within account_payment_method_fs_storage'tests seems to create an error:
#1360

I 'manually' created the same data in my tests.

@AnizR AnizR force-pushed the 16.0-fix-test-account-payment-method-fs-storage branch from 8f96ab3 to 4df82d5 Compare March 13, 2025 08:24
@AnizR
Copy link
Copy Markdown
Author

AnizR commented Mar 13, 2025

@pedrobaeza as promised, here is the correction of my tests 😄

@pedrobaeza pedrobaeza changed the title [FIX] account_payment_method_fs_storage: improve test by removing usage of datas [16.0][FIX] account_payment_method_fs_storage: improve test by removing usage of datas Mar 13, 2025
@pedrobaeza pedrobaeza added this to the 16.0 milestone Mar 13, 2025
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Thanks!

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1430-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 9404c52 into OCA:16.0 Mar 13, 2025
7 checks passed
@AnizR AnizR deleted the 16.0-fix-test-account-payment-method-fs-storage branch March 13, 2025 09:44
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.

3 participants