-
-
Notifications
You must be signed in to change notification settings - Fork 314
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
[16.0] base_wamas_ubl: run lib tests #1068
Conversation
jbaudoux
commented
Nov 16, 2024
•
edited
Loading
edited
- Add missing freezetime
- Fix html escaping
- Move lib tests to lib (some tests were inside the lib but not run by the CI, some other pure lib tests were inside the module)
- Run all lib tests as module unittests
2c98cbf
to
a9ffd7e
Compare
ping @cyrilmanuel @santostelmo @simahawk |
@simahawk Can you review and merge this if good for you? |
@@ -60,6 +62,7 @@ def dict2wamas(dict_input, msg_type): | |||
return wamas | |||
|
|||
|
|||
@freeze_time("2024-02-11 22:14: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.
why freezing this?
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.
That's the datetime in the xml, so that the test succeeds
a9ffd7e
to
8e6280a
Compare
rebased |
/ocabot merge patch |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 811254f. Thanks a lot for contributing to OCA. ❤️ |