You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
the API project test coverage was ignoring all code in notifications-utils. Previously we had a separate notification-utils project in GitHub that ran its own tests. Now that notifications-utils is integrated into notifications-admin, we should run coverage on notifications-utils.
The API project was also counting coverage against all code under tests/, which makes no sense.
When we run coverage properly (ignore migrations and tests, count everything else), it turns out we only have 91% coverage. We need to bring this back up to 95%.
The text was updated successfully, but these errors were encountered:
the API project test coverage was ignoring all code in notifications-utils. Previously we had a separate notification-utils project in GitHub that ran its own tests. Now that notifications-utils is integrated into notifications-admin, we should run coverage on notifications-utils.
The API project was also counting coverage against all code under tests/, which makes no sense.
When we run coverage properly (ignore migrations and tests, count everything else), it turns out we only have 91% coverage. We need to bring this back up to 95%.
The text was updated successfully, but these errors were encountered: