Skip to content

Commit

Permalink
[Code Health] chore: cleanup localnet testutils (#515)
Browse files Browse the repository at this point in the history
  • Loading branch information
bryanchriswhite authored May 8, 2024
1 parent c6cb4d0 commit f488165
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -450,7 +450,7 @@ todo_count: ## Print a count of all the TODOs in the project

.PHONY: todo_this_commit
todo_this_commit: ## List all the TODOs needed to be done in this commit
grep --exclude-dir={.git,vendor,.vscode} --exclude=Makefile -r -e "TODO_IN_THIS_"
grep -n --exclude-dir={.git,vendor,.vscode,.idea} --exclude={Makefile,reviewdog.yml} -r -e "TODO_IN_THIS_"

####################
### Gateways ###
Expand Down
11 changes: 7 additions & 4 deletions testutil/testclient/localnet.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,11 @@ var (
// CometLocalWebsocketURL provides a default URL pointing to the localnet websocket endpoint.
CometLocalWebsocketURL = "ws://localhost:36657/websocket"

// TODO_IN_THIS_COMMIT: godoc comments...
TxConfig client.TxConfig
Marshaler codec.Codec
// TxConfig provided by app.AppConfig(), intended as a convenience for use in tests.
TxConfig client.TxConfig
// Marshaler provided by app.AppConfig(), intended as a convenience for use in tests.
Marshaler codec.Codec
// InterfaceRegistry provided by app.AppConfig(), intended as a convenience for use in tests.
InterfaceRegistry codectypes.InterfaceRegistry
)

Expand All @@ -39,10 +41,11 @@ func init() {
deps := depinject.Configs(
app.AppConfig(),
depinject.Supply(
app.AppConfig(),
log.NewLogger(os.Stderr),
),
)

// Ensure that the global variables are initialized.
if err := depinject.Inject(
deps,
&TxConfig,
Expand Down

0 comments on commit f488165

Please sign in to comment.