Vitest/C8 coverage includes the unit test files themselves

[lloydjatkinson]

Describe the bug

As per the docs I have the following npm script to generate the coverage report:

"coverage": "vitest run --coverage"

However it includes the unit test files themselves which is not useful and also leads to incorrect coverage metrics. Please can this be addressed in Vite?

Reproduction

Follow getting started guide and make a basic function and a unit test that imports that function. Have "coverage": "vitest run --coverage" in npm scrips and run it. Observe the reports in both the console and HTML istanbul report contain both SUT's and the unit tests themselves.

System Info

Windows and Linux, Node 16.

Used Package Manager

npm

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[pandyarudra]

@lloydjatkinson Hey Lloyd, please try excluding the spec files in the coverage config options as follows:

// vite.config.js
export defineConfig({
  test: {
    coverage: {
      reporter: ['lcov', "html"],
      exclude: [
        '**/*.spec.ts',
      ],
    }
  }
});

All you need is the exclude config.

[sheremet-va]

All you need is the exclude config.

It should already be excluded. We have defaults.ts file with default coverage. We need to add spec alongside test there. PR welcome.

[seivan]

Which begs the question, what happens if you do in-source testing? I love that feature, and I can't leave it.

[pandyarudra]

@sheremet-va PR is here: #1627

[OmgImAlexis]

Which begs the question, what happens if you do in-source testing? I love that feature, and I can't leave it.

If this isn't fixed by #1627 I'd suggest making a new issue.