Add mechanism to fail on WARNING messages in Odoo test log

[sbidoul]

No description provided.

[rousseldenis]

Complementary of this, it should be great to have that level on check status : community/community#11592

[sbidoul]

Now that checklog-odoo is a standalone project, we can progress with this.

checklog-odoo can be installed with pipx in the Dockerfile.

One way to approach this is to add an OCA_ENABLE_CHECKLOG_ODOO environment vairable and an oca_checklog_odoo script. If that variable is set, the script runs checklog-odoo. If not set, the script run checklog-odoo -i ".*" to ignore all warnings`.

oca_init_test_database and oca_run_tests can then be adapted to pipe into oca_checklog_odoo.

[rousseldenis]

@sbidoul A POC on what can be the mechanism : OCA/stock-logistics-workflow#1632

[sbidoul]

@rousseldenis yes, I've seen that one. But I think the approach I outlined above will be simpler.

[rousseldenis]

@rousseldenis yes, I've seen that one. But I think the approach I outlined above will be simpler.

Yes, but the difference is that main tests will fail, correct ?

[sbidoul]

Yes, that's the point. I think the vast majority of warnings reveal real bugs that need to be resolved, and there is a way to ignore some with checklog-odoo config if needed.

[rousseldenis]

Yes, that's the point. I think the vast majority of warnings reveal real bugs that need to be resolved, and there is a way to ignore some with checklog-odoo config if needed.

Ok, I +100 for that

[sbidoul]

Done in #77