GoogleChrome / chromium-dashboard

Chrome Status Dashboard
https://www.chromestatus.com
Apache License 2.0
641 stars 372 forks source link

Increase test coverage to 100% for api directory #2786

Open jrobbins opened 1 year ago

jrobbins commented 1 year ago

Describe the bug You can run a test coverage report by doing npm run coverage then npm run view-coverage and browsing localhost:8080.

Our unit test coverage for the /api directory is pretty good, but there are a few gaps:

Ideally untested code would become tested code by adding new unit tests. However, if needed we can mark specific lines as excluded from the coverage metrics. https://coverage.readthedocs.io/en/stable/excluding.html

KyleJu commented 1 year ago
Screenshot 2023-03-14 at 11 13 16 AM

Converters.py has been improved to 98%

KyleJu commented 1 year ago

With #2836, all API files have achieved at least 90% test coverage