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

[RFC] adding test framework #725

Open
legalsylvain opened this issue Nov 5, 2021 · 3 comments
Open

[RFC] adding test framework #725

legalsylvain opened this issue Nov 5, 2021 · 3 comments
Labels
enhancement no stale Use this label to prevent the automated stale action from closing this PR/Issue.

Comments

@legalsylvain
Copy link
Contributor

Hi all,

Is your feature request related to a problem?

we regurlarly needs to makes python test, regarding PoS. for exemple, https://github.com/OCA/pos/blob/12.0/pos_session_closing_stock_error/tests/test_pos.py by @ivantodorovich, https://github.com/OCA/pos/blob/12.0/pos_multiple_control/tests/test_pos_multiple_control.py#L10 by @quentinDupont and Me.
The test are very verbose. It requires lot of lines of code, just to open a session, make a sale, add payment, etc... and it's quite a mess.

Describe the solution you'd like
Add a module pos_test_framework, that could provide test function like def open_session(), def sale(product=False, qty=False), def close_session(), etc... and make new modules, in V15 depends on that module.
We could also provide helper also for js scripts.

Thanks for your point of view.

CC : @OCA/pos-maintainers

@github-actions
Copy link

github-actions bot commented May 8, 2022

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 8, 2022
@legalsylvain legalsylvain added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Jul 8, 2022
@legalsylvain legalsylvain reopened this Jul 8, 2022
@ivantodorovich
Copy link
Contributor

It may make sense for older odoo versions, but testing the pos is getting easier and easier.
For >= 15.0 is actually pretty straightforward IMO.

@legalsylvain
Copy link
Contributor Author

It may make sense for older odoo versions, but testing the pos is getting easier and easier.

You're right. seems that there is PoS Test framework in new version, so maybe it's obsolete.
I'll develop some module for V16. I'll think if it's interesting to have extra OCA framework, or not.

thanks.

Note : could you take a look on #811 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Development

No branches or pull requests

2 participants