Skip to content

Commit

Permalink
there is no need to nil the app.settings on ResetBootstrapState
Browse files Browse the repository at this point in the history
  • Loading branch information
ganigeorgiev committed Sep 1, 2023
1 parent 5840145 commit 78e70bd
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
1 change: 0 additions & 1 deletion core/base.go
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,6 @@ func (app *BaseApp) ResetBootstrapState() error {

app.dao = nil
app.logsDao = nil
app.settings = nil

return nil
}
Expand Down
4 changes: 0 additions & 4 deletions core/base_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -118,10 +118,6 @@ func TestBaseAppBootstrap(t *testing.T) {
if app.logsDao != nil {
t.Fatalf("Expected app.logsDao to be nil, got %v.", app.logsDao)
}

if app.settings != nil {
t.Fatalf("Expected app.settings to be nil, got %v.", app.settings)
}
}

func TestBaseAppGetters(t *testing.T) {
Expand Down

0 comments on commit 78e70bd

Please sign in to comment.