Skip to content

Commit

Permalink
CLI: Fix wire graph for reset-admin-password (#60846)
Browse files Browse the repository at this point in the history
fix(cli): correct wire binding for reset-admin-password

The cli methods were updated to use the db.DB interface, but that interface was still wired in as the fake DB - it had only been used for tests previously. This isn't an issue in the main branch now that it's been collapsed into a single binary all using the same wire graph.

Fixes #59915
  • Loading branch information
mildwonkey committed Dec 30, 2022
1 parent d7146dd commit c75abb0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/cmd/grafana-cli/runner/wire.go
Original file line number Diff line number Diff line change
Expand Up @@ -336,7 +336,7 @@ var wireSet = wire.NewSet(
wire.Bind(new(notifications.EmailSender), new(*notifications.NotificationServiceMock)),
dbtest.NewFakeDB,
wire.Bind(new(sqlstore.Store), new(*sqlstore.SQLStore)),
wire.Bind(new(db.DB), new(*dbtest.FakeDB)),
wire.Bind(new(db.DB), new(*sqlstore.SQLStore)),
prefimpl.ProvideService,
opentsdb.ProvideService,
acimpl.ProvideAccessControl,
Expand Down

0 comments on commit c75abb0

Please sign in to comment.