-
Notifications
You must be signed in to change notification settings - Fork 366
Open
Labels
area/toolingAffects the dev or user toolchain (e.g. tests, ci, build tools)Affects the dev or user toolchain (e.g. tests, ci, build tools)hint/good first issueSomeone new could handle thisSomeone new could handle thishint/help wantedSomeone else needs to handle thisSomeone else needs to handle this
Description
This code: https://github.com/authzed/spicedb/blob/main/pkg/cmd/migrate.go#L49
has zero coverage, as seen in Codecov: https://app.codecov.io/gh/authzed/spicedb/blob/main/pkg%2Fcmd%2Fmigrate.go#L49
A refactor might be needed to make this function more testable.
For similar tests that can help as a base, see https://github.com/authzed/spicedb/blob/main/pkg/cmd/datastore_test.go
Metadata
Metadata
Assignees
Labels
area/toolingAffects the dev or user toolchain (e.g. tests, ci, build tools)Affects the dev or user toolchain (e.g. tests, ci, build tools)hint/good first issueSomeone new could handle thisSomeone new could handle thishint/help wantedSomeone else needs to handle thisSomeone else needs to handle this