Contributors mailing list archives

Browse archives


Re: 100 % coverage not enough?

Acsone SA/NV, Denis Roussel
- 11/09/2023 10:19:47
The answer is in the details link.

@jairo I don't agree about removing that check.

The 'patch' coverage report concerns exclusively the code you introduce with your PR.
The 'project' coverage report concerns the already existing code outside your PR. So, a decrease there means that your PR influences other modules (not executing a super in an inherited method, ...)

Even if coverage is not mandatory in OCA, it gives indicators on technical tests quality.

My two cents

On Mon, Sep 11, 2023 at 10:11 AM jairo <> wrote:
El vie, 08-09-2023 a las 18:57 +0000, Ronald Portier escribió:
Supposedly the PR would decrease the overall coverage percentage. How??

The 1st thing you need to know is that hose checks are not required.

Now, this is how your PR looks:

That ✅ is telling the reviewer that the PR has a 100% of coverage. That's the important part.

The ❌ however is the weirdo. Sometimes it happens, so I systematically ignore that check. IMHO it could be removed. My guess is that the base branch evolved and got more coverage, and that if you rebased it would always increase. But maybe it's just a bug somewhere.

Post to:


Denis Roussel
Software Engineer
T    : +32 2 888 31 49
M : +32 472 22 00 57

Val Benoit, Quai Banning 6 | B-4000 Liège | Belgium
Atrium Building, Drève Richelle 167 | B-1410 Waterloo | Belgium
Zone industrielle 22 | L-8287 Kehlen | Luxembourg