Skip to content

Commit

Permalink
Cleaning up asserts
Browse files Browse the repository at this point in the history
  • Loading branch information
boostchicken authored and jpkrohling committed Jan 18, 2022
1 parent 54bee73 commit 2c603bd
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions processor/filterprocessor/filter_processor_traces_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,16 +142,16 @@ func TestFilterTraceProcessor(t *testing.T) {
cfg,
next,
)
assert.NotNil(t, fmp)
assert.Nil(t, err)
require.NotNil(t, fmp)
require.Nil(t, err)

caps := fmp.Capabilities()
assert.True(t, caps.MutatesData)
require.True(t, caps.MutatesData)

assert.NoError(t, fmp.Start(ctx, nil))
require.NoError(t, fmp.Start(ctx, nil))

cErr := fmp.ConsumeTraces(ctx, test.inTraces)
assert.Nil(t, cErr)
require.Nil(t, cErr)
got := next.AllTraces()

// If all traces got filtered you shouldn't even have ResourceSpans
Expand All @@ -160,7 +160,7 @@ func TestFilterTraceProcessor(t *testing.T) {
} else {
require.Equal(t, test.spanCountExpected, got[0].SpanCount())
}
assert.NoError(t, fmp.Shutdown(ctx))
require.NoError(t, fmp.Shutdown(ctx))
})
}
}
Expand Down

0 comments on commit 2c603bd

Please sign in to comment.