Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.24.4: Unable to load custom reporters by package name #2245

Closed
6 tasks done
AriPerkkio opened this issue Oct 31, 2022 · 2 comments 路 Fixed by #2249
Closed
6 tasks done

v0.24.4: Unable to load custom reporters by package name #2245

AriPerkkio opened this issue Oct 31, 2022 · 2 comments 路 Fixed by #2249

Comments

@AriPerkkio
Copy link
Member

Describe the bug

Likely related to #2184 (馃憢 @poyoho).

Custom reporters cannot be loaded by using the package name in vitest@0.24.4. There are no errors or any logging printed, the program just exits. Everything works fine with vitest@0.24.3.

As this has broken before #1194, I think it is time to add test cases for this.

Reproduction

https://stackblitz.com/edit/vitest-dev-vitest-hubyyo

Stackblitz is working weirdly now so here is minimal vitest config also:

export default defineConfig({
  test: {
    outputFile: 'report.xml',

    // Works on 0.24.3, breaks on 0.24.4
    reporters: ['vitest-sonar-reporter'],

    // Works on 0.24.4
    // reporters: ['node_modules/vitest-sonar-reporter/dist/index'],
  },
});

System Info

Stackblitz, Github Codespaces

Used Package Manager

npm

Validations

@AriPerkkio AriPerkkio changed the title 0.24.4: Unable to load custom reporters by package name v0.24.4: Unable to load custom reporters by package name Oct 31, 2022
@sheremet-va
Copy link
Member

Time for regression 馃帀

@poyoho
Copy link
Member

poyoho commented Oct 31, 2022

I add the test case in #2249, but the case will run pass in my machine 馃憖

@github-actions github-actions bot locked and limited conversation to collaborators Jun 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants