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

[17.0][ADD] odoo_test_xmlrunner module #2957

Open
wants to merge 1 commit into
base: 17.0
Choose a base branch
from

Conversation

madecsmile
Copy link
Contributor

@madecsmile madecsmile commented May 25, 2024

@madecsmile madecsmile force-pushed the 17.0-add-odoo_test_xmlrunner branch 3 times, most recently from deae97b to cb35dca Compare May 25, 2024 19:22
@pedrobaeza pedrobaeza added this to the 17.0 milestone May 27, 2024
Copy link
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.

I will test it in real soon

odoo_test_xmlrunner/__manifest__.py Outdated Show resolved Hide resolved
odoo_test_xmlrunner/readme/CREDITS.md Outdated Show resolved Hide resolved
@madecsmile madecsmile force-pushed the 17.0-add-odoo_test_xmlrunner branch from cb35dca to e2dcbd8 Compare May 27, 2024 15:43
@madecsmile madecsmile requested a review from pedrobaeza May 27, 2024 15:46
Copy link
Member

@yajo yajo left a comment

Choose a reason for hiding this comment

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

Tested on Gitlab. Works like a charm. Thanks! ❤️


module.current_test = module_name
threading.current_thread().testing = True
results = OdooTestResult()
Copy link
Contributor

@paradoxxxzero paradoxxxzero Nov 21, 2024

Choose a reason for hiding this comment

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

This returns an empty results which leads to displaying no test results and fails to exit the process with the correct code. Please have a look at my backport : #3127

"website": "https://github.com/OCA/server-tools",
"category": "Tools",
"sequence": 20,
"auto_install": True,
Copy link
Contributor

Choose a reason for hiding this comment

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

which is the sense of Autoinstall True?
#3156

Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

@@ -0,0 +1,3 @@
Install python library https://pypi.org/project/unittest-xml-reporting/

The module is automatically installed on the Odoo instance.
Copy link
Contributor

Choose a reason for hiding this comment

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

should be removed also.

@@ -0,0 +1 @@
unittest-xml-reporting
Copy link
Contributor

Choose a reason for hiding this comment

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

this file is useless ? or is it a new OCA convention ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants