action #153107
opencoordination #99303: [saga][epic] Future improvements for SUSE Maintenance QA workflows with fully automated testing, approval and release
coordination #97121: [epic] enable qem-bot comments on IBS (was: enable qa-maintenance/openQABot comments on smelt again)
Proper code coverage for at least commenter.py in qem-bot size:M
0%
Description
Motivation¶
We would like to provide proper updates about the current approval status of qem-bot in #121228 but the current code coverage within unit tests is insufficient to provide a stable development base so we should provide proper code coverage for at least commenter.py in qem-bot first.
Acceptance criteria¶
- AC1: Significant statement coverage reported in https://app.codecov.io/gh/openSUSE/qem-bot/blob/master/openqabot%2Fcommenter.py (aim for 100%)
Suggestions¶
- Run tests with coverage locally
- Add tests accordingly and observe the coverage increase
- Repeat
- As likely currently the commenter class is not used at all feel free to aggressively refactor
- Currently the Commenter class is implemented as a subcommand
inc-comment
and it takes aNamespace
argument. As we only want to post comments as part of theinc-approve
subcommand, theCommenter
class should not implement a subcommand
Updated by okurz 9 months ago
- Copied from action #121228: qem-bot comments on IBS added
Updated by okurz 9 months ago
- Subject changed from Proper code coverage for at least commenter.py in qem-bot to Proper code coverage for at least commenter.py in qem-bot size:M
- Description updated (diff)
- Status changed from New to Workable
- Target version changed from Ready to future
As discussed in collaborative session 2024-01-04 while this ticket is technically the correct next step to do I am not convinced it provides sufficient Return-on-Invest for the tools team to go further in this direction but rather we should focus on features that benefit more and multiple products and use cases, e.g. where it benefits both Tumbleweed and SLE maintenance tests.