Skip to content

Repo: add a test-coverage script to simplify test flags #10607

Open
@43081j

Description

@43081j

Suggestion

This is explained already in #10582 but the summary is as follows:

We currently enable coverage in our test scripts and pass coverage=false when we want to disable it

This results in jest --coverage --coverage=false once everything is expanded

The duplicate flag works in jest but doesn't seem right. More importantly, it does not work with vitest (which we may soon want to move to).

We can solve this by having two scripts in each package:

  • test with coverage disabled
  • test-coverage with coverage enabled (if the package supports coverage)

CI can then call test-coverage instead of manually passing flag overrides

Additional Info

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    accepting prsGo ahead, send a pull request that resolves this issuerepo maintenancethings to do with maintenance of the repo, and not with code/docs

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions