neurobagel / annotation_tool

https://annotate.neurobagel.org/
MIT License
3 stars 7 forks source link

Compute and show code coverage reports #649

Open surchs opened 8 months ago

surchs commented 8 months ago

We have a nice test suite, we should have a coverage report. This should help: https://docs.cypress.io/guides/tooling/code-coverage

TODO:

github-actions[bot] commented 5 months ago

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days. We have applied the _flag:stale label to indicate that this issue should be reviewed again. When you review, please reread the spec and then apply one of these three options:

willtryagain commented 4 months ago

Hi,

while installing code-coverage I am getting this error. Please guide how to proceed.

'Log files: /home/a/.npm/_logs/2024-03-12T17_22_42_094Z-debug-0.log

npm resolution error report

While resolving: annotation_tool@1.0.0 Found: webpack@4.47.0 node_modules/webpack webpack@"^4.47.0" from the root project peer webpack@"^4 || ^5" from @cypress/code-coverage@3.12.29 node_modules/@cypress/code-coverage dev @cypress/code-coverage@"*" from the root project

Could not resolve dependency: peer webpack@">=5" from babel-loader@9.1.3 node_modules/babel-loader peer babel-loader@"^8.3 || ^9" from @cypress/code-coverage@3.12.29 node_modules/@cypress/code-coverage dev @cypress/code-coverage@"*" from the root project

Fix the upstream dependency conflict, or retry this command with --force or --legacy-peer-deps to accept an incorrect (and potentially broken) dependency resolution.

rmanaem commented 4 months ago

Hi @willtryagain Thanks for looking into this. I believe we've come across this dependency conflict before while trying to set up code coverage. The annotation tool uses Nuxt 2 which uses Webpack 4 for its build process and since there is no official support for Webpack 5, upgrading is not an option see https://github.com/nuxt/nuxt/issues/20055#issuecomment-1497299665. We plan to address this issue after transitioning the tool to React see https://github.com/neurobagel/query-tool/issues/62.

As a side note If you'd like to work on an issue please ping one of the maintainers to let us know so we can avoid conflicts and give you the required context. For more information on how to contribute checkout the Neurobagel contributing guide.

github-actions[bot] commented 2 months ago

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days. We have applied the _flag:stale label to indicate that this issue should be reviewed again. When you review, please reread the spec and then apply one of these three options: