[17.0][ADD] odoo_test_xmlrunner module #2957
Conversation
madecsmile
force-pushed
the
17.0-add-odoo_test_xmlrunner
branch
3 times, most recently
from
deae97b to
cb35dca
Compare
madecsmile
force-pushed
the
17.0-add-odoo_test_xmlrunner
branch
from
cb35dca to
e2dcbd8
Compare
|
|
||
| module.current_test = module_name | ||
| threading.current_thread().testing = True | ||
| results = OdooTestResult() |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Hi @paradoxxxzero I trierd your elegant patch on gitlab v17, I just have 1 test in my results
Without your patch I should have 500 tests. Do you have the same behavior in v16 ?
Martin
There was a problem hiding this comment.
Hmm no :
Something might have changed between 16 and 17.
There was a problem hiding this comment.
Ok I will dig, thanks.
odoo_test_xmlrunner/__manifest__.py
Outdated
| "website": "https://github.com/OCA/server-tools", | ||
| "category": "Tools", | ||
| "sequence": 20, | ||
| "auto_install": True, |
There was a problem hiding this comment.
which is the sense of Autoinstall True?
#3156
| @@ -0,0 +1,3 @@ | |||
| Install python library https://pypi.org/project/unittest-xml-reporting/ | |||
|
|
|||
| The module is automatically installed on the Odoo instance. | |||
There was a problem hiding this comment.
should be removed also.
odoo_test_xmlrunner/requirements.txt
Outdated
| @@ -0,0 +1 @@ | |||
| unittest-xml-reporting | |||
There was a problem hiding this comment.
this file is useless ? or is it a new OCA convention ?
There was a problem hiding this comment.
Hello @legalsylvain I tried to remove it and the generation of test-requirements.txt looks fine. I will remove the odoo_test_xmlrunner/requirements.txt file.
Martin
https://github.com/orgs/OCA/discussions/167