Jonatandb / weather-app-with-fastify

WeatherApp-Fastify
https://weatherapp-jdb.web.app
0 stars 0 forks source link

(backend) Jest coverage report is wrong #1

Open Jonatandb opened 2 years ago

Jonatandb commented 2 years ago

By running npm run test:coverage the results are wrong and even Jest is checking files that it shouldn't verify like .eslintrc.js

To check:

Jonatandb commented 2 years ago

Parcialy solved by 0c552664b620d87fe0feae0ff7c69c051ae762a8

Still pending check why Coverage do not detect that controllers/handlers and app.js / server.js has no tests

Jonatandb commented 2 years ago

Possibly useful/related: - https://stackoverflow.com/questions/63696265/how-to-get-a-code-coverage-report-with-react-create-app

tiago154 commented 2 years ago

Hi @Jonatandb , I can help with this. What folders and files should be in coverage? controllers, lib, routes and utils?

Jonatandb commented 2 years ago

Hi @Jonatandb , I can help with this. What folders and files should be in coverage? controllers, lib, routes and utils?

Hey Tiago 🙋🏻‍♂️ thank you so much!

Really apreciate your motivation and spirit, but to be honest this was just a kind of job interview sample and I'm not currently working with it neither using this project anymore... Anyway, if you really want to cooperate (just for fun/practice or hobby), feel free to make any porcentage of coverage that you consider useful, and I will check and accept your PR!

Again, thanks!