Skip to content

Commit

Permalink
Need to defer inside of a function or the Close gets evaluated immedi…
Browse files Browse the repository at this point in the history
…ately.
  • Loading branch information
notbdu committed Feb 28, 2020
1 parent 97d56ff commit f8a84d8
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/m3ninx/index/segment/builder/builder_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,9 @@ var (
func TestBuilderFields(t *testing.T) {
builder, err := NewBuilderFromDocuments(testOptions)
require.NoError(t, err)
defer require.NoError(t, builder.Close())
defer func() {
require.NoError(t, builder.Close())
}()

for i := 0; i < 10; i++ {
builder.Reset(0)
Expand Down Expand Up @@ -107,7 +109,9 @@ func TestBuilderFields(t *testing.T) {
func TestBuilderTerms(t *testing.T) {
builder, err := NewBuilderFromDocuments(testOptions)
require.NoError(t, err)
defer require.NoError(t, builder.Close())
defer func() {
require.NoError(t, builder.Close())
}()

for i := 0; i < 10; i++ {
builder.Reset(0)
Expand Down

0 comments on commit f8a84d8

Please sign in to comment.