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

[16.0] base_wamas_ubl: run lib tests #1068

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

jbaudoux
Copy link
Contributor

@jbaudoux jbaudoux commented Nov 16, 2024

  • 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

@jbaudoux jbaudoux force-pushed the 16-fix-wamas-lib-tests branch 3 times, most recently from 2c98cbf to a9ffd7e Compare November 16, 2024 13:19
@jbaudoux
Copy link
Contributor Author

ping @cyrilmanuel @santostelmo @simahawk
Some lib tests were failing (due to missing freezetime) but it was not visible as they were not run by the CI

@jbaudoux
Copy link
Contributor Author

@simahawk Can you review and merge this if good for you?

@jbaudoux
Copy link
Contributor Author

jbaudoux commented Dec 6, 2024

cc @nilshamerlinck

@@ -60,6 +62,7 @@ def dict2wamas(dict_input, msg_type):
return wamas


@freeze_time("2024-02-11 22:14:22")
Copy link
Contributor

Choose a reason for hiding this comment

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

why freezing this?

Copy link
Contributor Author

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

@jbaudoux jbaudoux force-pushed the 16-fix-wamas-lib-tests branch from a9ffd7e to 8e6280a Compare December 11, 2024 12:57
@jbaudoux
Copy link
Contributor Author

rebased

@jbaudoux
Copy link
Contributor Author

jbaudoux commented Jan 8, 2025

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1068-by-jbaudoux-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d35a89c into OCA:16.0 Jan 8, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

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