Skip to content

Commit 3dc614f

Browse files
AntonboomAntonboom
committed
Fix testifylint issues
1 parent 2135d38 commit 3dc614f

11 files changed

+27
-19
lines changed

‎pkg/golinters/gocritic_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ func Test_intersectStringSlice(t *testing.T) {
1818

1919
sort.Strings(s3)
2020

21-
assert.Equal(t, s3, []string{"experimental", "opinionated"})
21+
assert.Equal(t, []string{"experimental", "opinionated"}, s3)
2222
}
2323

2424
func Test_filterByDisableTags(t *testing.T) {

‎pkg/golinters/gofmt_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import (
66
diffpkg "github.com/sourcegraph/go-diff/diff"
77
"github.com/stretchr/testify/assert"
88
"github.com/stretchr/testify/mock"
9+
"github.com/stretchr/testify/require"
910

1011
"github.com/golangci/golangci-lint/pkg/logutils"
1112
"github.com/golangci/golangci-lint/pkg/result"
@@ -14,7 +15,7 @@ import (
1415
func testDiffProducesChanges(t *testing.T, log logutils.Log, diff string, expectedChanges ...Change) {
1516
diffs, err := diffpkg.ParseMultiFileDiff([]byte(diff))
1617
if err != nil {
17-
assert.NoError(t, err)
18+
require.NoError(t, err)
1819
}
1920

2021
assert.Len(t, diffs, 1)

‎pkg/result/processors/autogenerated_exclude_test.go

Lines changed: 3 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎pkg/result/processors/identifier_marker_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import (
44
"testing"
55

66
"github.com/stretchr/testify/assert"
7+
"github.com/stretchr/testify/require"
78

89
"github.com/golangci/golangci-lint/pkg/result"
910
)
@@ -50,7 +51,7 @@ func TestIdentifierMarker(t *testing.T) {
5051

5152
for _, c := range cases {
5253
out, err := p.Process([]result.Issue{{Text: c.in}})
53-
assert.NoError(t, err)
54+
require.NoError(t, err)
5455
assert.Equal(t, []result.Issue{{Text: c.out}}, out)
5556
}
5657
}

‎pkg/result/processors/nolint_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import (
88

99
"github.com/stretchr/testify/assert"
1010
"github.com/stretchr/testify/mock"
11+
"github.com/stretchr/testify/require"
1112

1213
"github.com/golangci/golangci-lint/pkg/config"
1314
"github.com/golangci/golangci-lint/pkg/golinters"
@@ -173,9 +174,9 @@ func TestNolintInvalidLinterNameWithViolationOnTheSameLine(t *testing.T) {
173174
processedIssues, err := p.Process(issues)
174175
p.Finish()
175176

177+
require.NoError(t, err)
176178
assert.Len(t, processedIssues, 1)
177179
assert.Equal(t, issues, processedIssues)
178-
assert.NoError(t, err)
179180
}
180181

181182
func TestNolintAliases(t *testing.T) {
@@ -288,7 +289,7 @@ func TestNolintUnused(t *testing.T) {
288289
enabledLintersSet := lintersdb.NewEnabledSet(dbManager, lintersdb.NewValidator(dbManager), enabledSetLog, cfg)
289290

290291
enabledLintersMap, err := enabledLintersSet.GetEnabledLintersMap()
291-
assert.NoError(t, err)
292+
require.NoError(t, err)
292293

293294
return NewNolint(log, dbManager, enabledLintersMap)
294295
}
@@ -351,7 +352,7 @@ func TestNolintUnused(t *testing.T) {
351352
enabledLintersSet := lintersdb.NewEnabledSet(dbManager, lintersdb.NewValidator(dbManager), enabledSetLog, cfg)
352353

353354
enabledLintersMap, err := enabledLintersSet.GetEnabledLintersMap()
354-
assert.NoError(t, err)
355+
require.NoError(t, err)
355356
p := NewNolint(log, dbManager, enabledLintersMap)
356357
defer p.Finish()
357358

‎pkg/result/processors/path_prefixer_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ func TestPathPrefixer_Process(t *testing.T) {
4747
got, err := p.Process(tt.issues)
4848
require.NoError(t, err)
4949

50-
assert.Equal(t, got, tt.want)
50+
assert.Equal(t, tt.want, got)
5151
})
5252
}
5353
}

‎pkg/result/processors/processor_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import (
55
"testing"
66

77
"github.com/stretchr/testify/assert"
8+
"github.com/stretchr/testify/require"
89

910
"github.com/golangci/golangci-lint/pkg/result"
1011
)
@@ -38,7 +39,7 @@ func process(t *testing.T, p Processor, issues ...result.Issue) []result.Issue {
3839
t.Helper()
3940

4041
processedIssues, err := p.Process(issues)
41-
assert.NoError(t, err)
42+
require.NoError(t, err)
4243
return processedIssues
4344
}
4445

‎pkg/result/processors/skip_files_test.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import (
77
"testing"
88

99
"github.com/stretchr/testify/assert"
10+
"github.com/stretchr/testify/require"
1011

1112
"github.com/golangci/golangci-lint/pkg/result"
1213
)
@@ -20,8 +21,9 @@ func newFileIssue(file string) result.Issue {
2021
}
2122

2223
func newTestSkipFiles(t *testing.T, patterns ...string) *SkipFiles {
24+
t.Helper()
2325
p, err := NewSkipFiles(patterns, "")
24-
assert.NoError(t, err)
26+
require.NoError(t, err)
2527
return p
2628
}
2729

@@ -48,6 +50,6 @@ func TestSkipFiles(t *testing.T) {
4850

4951
func TestSkipFilesInvalidPattern(t *testing.T) {
5052
p, err := NewSkipFiles([]string{"\\o"}, "")
51-
assert.Error(t, err)
53+
require.Error(t, err)
5254
assert.Nil(t, p)
5355
}

‎pkg/result/processors/sort_results_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import (
66
"testing"
77

88
"github.com/stretchr/testify/assert"
9+
"github.com/stretchr/testify/require"
910

1011
"github.com/golangci/golangci-lint/pkg/config"
1112
"github.com/golangci/golangci-lint/pkg/result"
@@ -158,8 +159,8 @@ func TestNoSorting(t *testing.T) {
158159
var sr = NewSortResults(&config.Config{})
159160

160161
results, err := sr.Process(tests)
162+
require.NoError(t, err)
161163
assert.Equal(t, tests, results)
162-
assert.Nil(t, err, nil)
163164
}
164165

165166
func TestSorting(t *testing.T) {
@@ -177,6 +178,6 @@ func TestSorting(t *testing.T) {
177178
var sr = NewSortResults(&cfg)
178179

179180
results, err := sr.Process(tests)
180-
assert.Equal(t, results, expected)
181-
assert.Nil(t, err, nil)
181+
require.NoError(t, err)
182+
assert.Equal(t, expected, results)
182183
}

‎test/run_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import (
44
"path/filepath"
55
"testing"
66

7-
"github.com/stretchr/testify/assert"
7+
"github.com/stretchr/testify/require"
88
_ "github.com/valyala/quicktemplate"
99

1010
"github.com/golangci/golangci-lint/pkg/exitcodes"
@@ -536,7 +536,7 @@ func TestEnabledPresetsAreNotDuplicated(t *testing.T) {
536536
func TestAbsPathDirAnalysis(t *testing.T) {
537537
dir := filepath.Join("testdata_etc", "abspath") // abs paths don't work with testdata dir
538538
absDir, err := filepath.Abs(dir)
539-
assert.NoError(t, err)
539+
require.NoError(t, err)
540540

541541
testshared.NewRunnerBuilder(t).
542542
WithNoConfig().
@@ -555,7 +555,7 @@ func TestAbsPathDirAnalysis(t *testing.T) {
555555
func TestAbsPathFileAnalysis(t *testing.T) {
556556
dir := filepath.Join("testdata_etc", "abspath", "with_issue.go") // abs paths don't work with testdata dir
557557
absDir, err := filepath.Abs(dir)
558-
assert.NoError(t, err)
558+
require.NoError(t, err)
559559

560560
testshared.NewRunnerBuilder(t).
561561
WithNoConfig().

‎test/testshared/runner.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,7 @@ func InstallGolangciLint(tb testing.TB) string {
349349
tb.Log(string(output))
350350
}
351351

352-
assert.NoError(tb, err, "Can't go install golangci-lint %s", string(output))
352+
require.NoError(tb, err, "Can't go install golangci-lint %s", string(output))
353353
}
354354

355355
abs, err := filepath.Abs(defaultBinaryName())

0 commit comments

Comments
 (0)