Skip to content

Commit a4092df

Browse files
authored
useless-assert: add NotElementsMatch and NotErrorAs (#220)
* Upgrade testdata to v1.10.0 of testify * self-review fixes * issue 213 debug * useless-assert: add NotElementsMatch and NotErrorAs
1 parent 6b7bd9b commit a4092df

File tree

4 files changed

+30
-0
lines changed

4 files changed

+30
-0
lines changed

analyzer/testdata/src/checkers-default/useless-assert/useless_assert_test.go

Lines changed: 4 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
package debug
2+
3+
import (
4+
"os"
5+
"testing"
6+
7+
"github.com/stretchr/testify/require"
8+
)
9+
10+
func TestErrorAsGenericTarget(t *testing.T) {
11+
var err error = new(os.PathError)
12+
13+
errorAs[*os.PathError]()(t, err)
14+
errorAs[*os.LinkError]()(t, err)
15+
}
16+
17+
func errorAs[T error]() require.ErrorAssertionFunc {
18+
return func(t require.TestingT, err error, msgAndArgs ...interface{}) {
19+
var target T
20+
require.ErrorAs(t, err, &target, msgAndArgs...)
21+
}
22+
}

internal/checkers/useless_assert.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,10 @@ func (checker UselessAssert) checkSameVars(pass *analysis.Pass, call *CallMeta)
127127
"JSONEq",
128128
"Less",
129129
"LessOrEqual",
130+
"NotElementsMatch",
130131
"NotEqual",
131132
"NotEqualValues",
133+
"NotErrorAs",
132134
"NotErrorIs",
133135
"NotRegexp",
134136
"NotSame",

internal/testgen/gen_useless_assert.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,10 @@ func (g UselessAssertTestsGenerator) TemplateData() any {
4141
"JSONEq": "str, str",
4242
"Less": "value, value",
4343
"LessOrEqual": "value, value",
44+
"NotElementsMatch": "value, value",
4445
"NotEqual": "value, value",
4546
"NotEqualValues": "value, value",
47+
"NotErrorAs": "err, err",
4648
"NotErrorIs": "err, err",
4749
"NotRegexp": "value, value",
4850
"NotSame": "value, value",

0 commit comments

Comments
 (0)