travertino icon indicating copy to clipboard operation
travertino copied to clipboard

Normalize CI and tox configuration with Briefcase

Open freakboy3742 opened this issue 1 year ago • 2 comments

What is the problem or limitation you are having?

Travertino has a tox configuration; however, for historical reasons, it uses slightly different targets to the pattern that are currently used by Briefcase (and other BeeWare projects) - most notably those related to coverage reporting.

Describe the solution you'd like

Travertino's tox configuration should provide the same interface used by other BeeWare projects. This includes:

  • a tox -m test3X set of meta-targets
  • Adding coverage generation targets
  • Including a coverage report as part of CI tasks.

Describe alternatives you've considered

Additional context

Briefcase currently enforces a 100% code coverage requirement; It is highly unlikely that Travertino's test suite actually meets this requirement. Being able to easily report current coverage is an important first step to getting to coverage; the "fail if coverage < 100%" rule can be omitted as part of this work.

(Of course, if we're only missing a couple of lines of coverage, adding the extra test cases to get to 100% would also be appreciated)

freakboy3742 avatar Mar 17 '24 22:03 freakboy3742

I would like to work on this issue. Can You please assign this issue to me.

VinayakPaka avatar Aug 15 '24 10:08 VinayakPaka

We don't have a formal assignment process – it's enough to post a comment as you already have, and then create a PR.

mhsmith avatar Aug 15 '24 12:08 mhsmith

@freakboy3742 I imagine this can be closed, now that Travertino's moving to the Toga repository, right?

HalfWhitt avatar Jan 18 '25 17:01 HalfWhitt

@HalfWhitt Yeah - I think we can close it down.

freakboy3742 avatar Jan 19 '25 03:01 freakboy3742