django-multiselectfield icon indicating copy to clipboard operation
django-multiselectfield copied to clipboard

Add Coveralls integration

Open blag opened this issue 1 year ago • 2 comments

Add Coveralls integration so we can track coverage over time, as well as enable contributors to see how their changes affect code coverage.

blag avatar May 21 '24 00:05 blag

It is interesting, that Coverals report the Django <5.0 branch of _get_flatchoices untested: Snímek obrazovky_2024-05-21_08-27-42

I am not sure why that happened.

PetrDlouhy avatar May 21 '24 06:05 PetrDlouhy

@blag I think that there must be some kind of problem with the different Django versions. While in Django 5.0 the tree looks like this: Snímek obrazovky_2024-05-30_12-03-02

On Django 4.2 the tree looks like this: Snímek obrazovky_2024-05-30_12-05-39 And the results are not accessible.

Maybe it uses different configuration with each Django version.

Also on Django 5.0 are examples part of the results, but I think they should not be.

PetrDlouhy avatar May 30 '24 10:05 PetrDlouhy

Added coveralls integration with last changes.

@PetrDlouhy and @blag you need combine all coverages: https://stackoverflow.com/questions/19274492/how-calculate-the-global-coverage.

Now this line is tested:

        flat_choices = super()._get_flatchoices()

_get_flatchoices

goinnn avatar Jun 10 '25 10:06 goinnn