-
Notifications
You must be signed in to change notification settings - Fork 42
librarian: add tests #202
Copy link
Copy link
Closed
18 / 1818 of 18 issues completedLabels
🚨 criticalP0 critical issue. Fix immediately. Resolving this is the highest priority.P0 critical issue. Fix immediately. Resolving this is the highest priority.
Description
In #201, I deleted code to migrate to Run, and no tests failed, suggesting none of the commands are tested.
In looking for test files, this was the only one that exists outside of all_test.go:
At the moment, it seems like tests do not exists to catch any problems in the librarian code outside of that. If that's the case, we need to add tests to librarian that run as part of CI for this codebase.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
🚨 criticalP0 critical issue. Fix immediately. Resolving this is the highest priority.P0 critical issue. Fix immediately. Resolving this is the highest priority.