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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cypress] add support for cypress-multi-reporter #701

Closed
aramissennyeydd opened this issue Jun 26, 2024 · 1 comment · Fixed by #726
Closed

[cypress] add support for cypress-multi-reporter #701

aramissennyeydd opened this issue Jun 26, 2024 · 1 comment · Fixed by #726
Labels
feature request Feature request

Comments

@aramissennyeydd
Copy link
Contributor

aramissennyeydd commented Jun 26, 2024

We're using the https://www.npmjs.com/package/cypress-multi-reporters Cypress plugin, which takes in a config file that lists other reporters.

Example,

// in cypress.config.ts
export default defineConfig({
  reporter: 'cypress-multi-reporters',
  reporterOptions: {
    configFile: 'reporter-config.json',
  },
  // other stuff
})

and in the referenced config file,

{
  "reporterEnabled": "mochawesome, mocha-junit-reporter", // comma + space separated list
}

Which currently reports,

## Unused dependencies (3)

| Name                    | Location     | Severity |
| :---------------------- | :----------- | :------- |
| cypress-multi-reporters | package.json | error    |
| mocha-junit-reporter    | package.json | error    |
| mochawesome             | package.json | error    |

With the following package.json dependencies,

{
"name": "my-e2e-package",
"dependencies": {
    "cypress": "12.17.4",
    "cypress-multi-reporters": "^1.6.3",
    "mocha": "^10.2.0",
    "mocha-junit-reporter": "^2.2.0",
    "mochawesome": "^6.2.2"
  }
}

It would be awesome to see this supported in knip! Happy to submit a PR!

@aramissennyeydd aramissennyeydd added the feature request Feature request label Jun 26, 2024
@webpro
Copy link
Collaborator

webpro commented Jun 27, 2024

Thanks @aramissennyeydd, a PR for this is welcome. I think it makes sense to add it to the Cypress plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants