Skip to content

Commit

Permalink
linter fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
mfreeman451 committed Jan 20, 2025
1 parent 504817c commit b85fd23
Showing 1 changed file with 10 additions and 13 deletions.
23 changes: 10 additions & 13 deletions pkg/sweeper/sweeper_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ package sweeper

import (
"context"
"errors"
"testing"
"time"

Expand All @@ -12,8 +11,6 @@ import (
"go.uber.org/mock/gomock"
)

var errTest = errors.New("test error")

func TestMockSweeper(t *testing.T) {
ctrl := gomock.NewController(t)
defer ctrl.Finish()
Expand All @@ -28,7 +25,7 @@ func TestMockSweeper(t *testing.T) {
Return(nil)

err := mockSweeper.Start(ctx)
assert.NoError(t, err)
require.NoError(t, err)

// Test Stop
mockSweeper.EXPECT().
Expand Down Expand Up @@ -76,7 +73,7 @@ func TestMockSweeper(t *testing.T) {
Return(expectedResults, nil)

results, err := mockSweeper.GetResults(ctx, filter)
assert.NoError(t, err)
require.NoError(t, err)
assert.Equal(t, expectedResults, results)
})

Expand All @@ -91,7 +88,7 @@ func TestMockSweeper(t *testing.T) {
Return(nil)

err := mockSweeper.UpdateConfig(newConfig)
assert.NoError(t, err)
require.NoError(t, err)
})
}

Expand Down Expand Up @@ -131,11 +128,11 @@ func TestMockResultProcessor(t *testing.T) {
Return(expectedSummary, nil)

summary, err := mockProcessor.GetSummary(ctx)
assert.NoError(t, err)
require.NoError(t, err)
assert.Equal(t, expectedSummary, summary)
})

t.Run("Reset", func(t *testing.T) {
t.Run("Reset", func(*testing.T) {
mockProcessor.EXPECT().
Reset()

Expand Down Expand Up @@ -190,7 +187,7 @@ func TestMockStore(t *testing.T) {
Return(expectedResults, nil)

results, err := mockStore.GetResults(ctx, filter)
assert.NoError(t, err)
require.NoError(t, err)
assert.Equal(t, expectedResults, results)
})

Expand All @@ -217,7 +214,7 @@ func TestMockStore(t *testing.T) {
Return(expectedSummary, nil)

summary, err := mockStore.GetSweepSummary(ctx)
assert.NoError(t, err)
require.NoError(t, err)
assert.Equal(t, expectedSummary, summary)
})
}
Expand Down Expand Up @@ -258,7 +255,7 @@ func TestMockSweepService(t *testing.T) {
Return(nil)

err := mockService.Start(ctx)
assert.NoError(t, err)
require.NoError(t, err)

mockService.EXPECT().
Stop().
Expand All @@ -280,7 +277,7 @@ func TestMockSweepService(t *testing.T) {
Return(expectedStatus, nil)

status, err := mockService.GetStatus(ctx)
assert.NoError(t, err)
require.NoError(t, err)
assert.Equal(t, expectedStatus, status)
})

Expand All @@ -300,7 +297,7 @@ func TestMockSweepService(t *testing.T) {
})
}

// Helper function to verify gomock matchers
// Helper function to verify gomock matchers.
func TestGomockMatchers(t *testing.T) {
ctrl := gomock.NewController(t)
defer ctrl.Finish()
Expand Down

0 comments on commit b85fd23

Please sign in to comment.