You've already forked opentelemetry-go
mirror of
https://github.com/open-telemetry/opentelemetry-go.git
synced 2025-08-10 22:31:50 +02:00
chore(deps): update module github.com/antonboom/testifylint to v1.6.0 (#6440)
This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/Antonboom/testifylint](https://redirect.github.com/Antonboom/testifylint) | `v1.5.2` -> `v1.6.0` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>Antonboom/testifylint (github.com/Antonboom/testifylint)</summary> ### [`v1.6.0`](https://redirect.github.com/Antonboom/testifylint/releases/tag/v1.6.0): – new `equal-values` and `suite-method-signature` [Compare Source](https://redirect.github.com/Antonboom/testifylint/compare/v1.5.2...v1.6.0) #### What's Changed ##### New checkers - new checker `equal-values` by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/223](https://redirect.github.com/Antonboom/testifylint/pull/223) - new checker `suite-method-signature` by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/228](https://redirect.github.com/Antonboom/testifylint/pull/228) ##### New features - `len`: support len-len and value-len cases by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/204](https://redirect.github.com/Antonboom/testifylint/pull/204) - `error-is-as`: support NotErrorAs by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/219](https://redirect.github.com/Antonboom/testifylint/pull/219) - `useless-assert`: add NotElementsMatch and NotErrorAs by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/220](https://redirect.github.com/Antonboom/testifylint/pull/220) - `formatter`: support non-string-message checks by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/221](https://redirect.github.com/Antonboom/testifylint/pull/221) - `formatter`: warn on empty message by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/225](https://redirect.github.com/Antonboom/testifylint/pull/225) - `empty`: support empty strings, Zero for strings and len + bubbled useless-assert cases by [@​ccoVeille](https://redirect.github.com/ccoVeille) in [https://github.com/Antonboom/testifylint/pull/129](https://redirect.github.com/Antonboom/testifylint/pull/129) ##### New fixes - `negative-positive`: remove untyping, ignore Negative for len comparisons by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/226](https://redirect.github.com/Antonboom/testifylint/pull/226) - fixes: support `assert.CollectT` by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/233](https://redirect.github.com/Antonboom/testifylint/pull/233) ##### Bump deps - Upgrade testdata to v1.10.0 of testify by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/218](https://redirect.github.com/Antonboom/testifylint/pull/218) - Go 1.24 by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/234](https://redirect.github.com/Antonboom/testifylint/pull/234) - build(deps): bump golang.org/x/tools from 0.26.0 to 0.27.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/Antonboom/testifylint/pull/206](https://redirect.github.com/Antonboom/testifylint/pull/206) - build(deps): bump rlespinasse/github-slug-action from 4.4.1 to 5.0.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/Antonboom/testifylint/pull/207](https://redirect.github.com/Antonboom/testifylint/pull/207) - build(deps): bump golang.org/x/tools from 0.27.0 to 0.29.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/Antonboom/testifylint/pull/214](https://redirect.github.com/Antonboom/testifylint/pull/214) **Full Changelog**: https://github.com/Antonboom/testifylint/compare/v1.5.2...v1.6.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/open-telemetry/opentelemetry-go). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4yMDAuMiIsInVwZGF0ZWRJblZlciI6IjM5LjIwMC4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJTa2lwIENoYW5nZWxvZyIsImRlcGVuZGVuY2llcyJdfQ==--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> Co-authored-by: Tyler Yahn <codingalias@gmail.com>
This commit is contained in:
@@ -285,7 +285,7 @@ func TestLoggerProviderLogger(t *testing.T) {
|
||||
assert.Equal(t, 1, l.level, "logged level")
|
||||
assert.Equal(t, "Invalid Logger name.", l.msg, "logged message")
|
||||
require.Len(t, l.keysAndValues, 2, "logged key values")
|
||||
assert.Equal(t, "", l.keysAndValues[1], "logged name")
|
||||
assert.Empty(t, l.keysAndValues[1], "logged name")
|
||||
})
|
||||
|
||||
t.Run("Stopped", func(t *testing.T) {
|
||||
|
@@ -243,8 +243,8 @@ func TestLogConflictName(t *testing.T) {
|
||||
tc.existing, tc.name,
|
||||
)
|
||||
} else {
|
||||
assert.Equalf(
|
||||
t, "", msg,
|
||||
assert.Emptyf(
|
||||
t, msg,
|
||||
"warning logged for non-conflicting names: %s, %s",
|
||||
tc.existing, tc.name,
|
||||
)
|
||||
|
@@ -307,7 +307,7 @@ func TestDefaultAggregationSelector(t *testing.T) {
|
||||
}
|
||||
|
||||
for _, ik := range iKinds {
|
||||
assert.NoError(t, DefaultAggregationSelector(ik).err(), ik)
|
||||
assert.NoError(t, DefaultAggregationSelector(ik).err(), "%+v", ik)
|
||||
}
|
||||
}
|
||||
|
||||
|
@@ -81,7 +81,7 @@ func TestDetect(t *testing.T) {
|
||||
if c.wantErr != nil {
|
||||
assert.ErrorIs(t, err, c.wantErr)
|
||||
if errors.Is(c.wantErr, resource.ErrSchemaURLConflict) {
|
||||
assert.Zero(t, r.SchemaURL())
|
||||
assert.Empty(t, r.SchemaURL())
|
||||
}
|
||||
} else {
|
||||
assert.NoError(t, err)
|
||||
|
@@ -60,6 +60,6 @@ func TestStringDetectorErrors(t *testing.T) {
|
||||
for _, kv := range res.Attributes() {
|
||||
m[string(kv.Key)] = kv.Value.Emit()
|
||||
}
|
||||
require.EqualValues(t, map[string]string{"A": "B"}, m)
|
||||
require.Equal(t, map[string]string{"A": "B"}, m)
|
||||
}
|
||||
}
|
||||
|
@@ -128,7 +128,7 @@ PROP3='Final value'`)
|
||||
|
||||
t.Run(tc.Name, func(t *testing.T) {
|
||||
result := resource.ParseOSReleaseFile(tc.OSRelease)
|
||||
require.EqualValues(t, tc.Parsed, result)
|
||||
require.Equal(t, tc.Parsed, result)
|
||||
})
|
||||
}
|
||||
}
|
||||
@@ -157,7 +157,7 @@ func TestSkip(t *testing.T) {
|
||||
|
||||
t.Run(tc.Name, func(t *testing.T) {
|
||||
result := resource.Skip(tc.Line)
|
||||
require.EqualValues(t, tc.Expected, result)
|
||||
require.Equal(t, tc.Expected, result)
|
||||
})
|
||||
}
|
||||
}
|
||||
@@ -184,9 +184,9 @@ func TestParse(t *testing.T) {
|
||||
|
||||
t.Run(tc.Name, func(t *testing.T) {
|
||||
key, value, ok := resource.Parse(tc.Line)
|
||||
require.EqualValues(t, tc.ExpectedKey, key)
|
||||
require.EqualValues(t, tc.ExpectedValue, value)
|
||||
require.EqualValues(t, tc.OK, ok)
|
||||
require.Equal(t, tc.ExpectedKey, key)
|
||||
require.Equal(t, tc.ExpectedValue, value)
|
||||
require.Equal(t, tc.OK, ok)
|
||||
})
|
||||
}
|
||||
}
|
||||
@@ -224,7 +224,7 @@ func TestUnquote(t *testing.T) {
|
||||
|
||||
t.Run(tc.Name, func(t *testing.T) {
|
||||
result := resource.Unquote(tc.Text)
|
||||
require.EqualValues(t, tc.Expected, result)
|
||||
require.Equal(t, tc.Expected, result)
|
||||
})
|
||||
}
|
||||
}
|
||||
@@ -248,7 +248,7 @@ func TestUnescape(t *testing.T) {
|
||||
|
||||
t.Run(tc.Name, func(t *testing.T) {
|
||||
result := resource.Unescape(tc.Text)
|
||||
require.EqualValues(t, tc.Expected, result)
|
||||
require.Equal(t, tc.Expected, result)
|
||||
})
|
||||
}
|
||||
}
|
||||
@@ -290,7 +290,7 @@ func TestBuildOSRelease(t *testing.T) {
|
||||
|
||||
t.Run(tc.Name, func(t *testing.T) {
|
||||
result := resource.BuildOSRelease(tc.Values)
|
||||
require.EqualValues(t, tc.Expected, result)
|
||||
require.Equal(t, tc.Expected, result)
|
||||
})
|
||||
}
|
||||
}
|
||||
|
@@ -49,7 +49,7 @@ func TestMapRuntimeOSToSemconvOSType(t *testing.T) {
|
||||
|
||||
t.Run(tc.Name, func(t *testing.T) {
|
||||
osTypeAttribute := resource.MapRuntimeOSToSemconvOSType(tc.Goos)
|
||||
require.EqualValues(t, osTypeAttribute, tc.OSType)
|
||||
require.Equal(t, osTypeAttribute, tc.OSType)
|
||||
})
|
||||
}
|
||||
}
|
||||
|
@@ -104,23 +104,23 @@ func TestWithProcessFuncsErrors(t *testing.T) {
|
||||
}
|
||||
|
||||
func TestCommandArgs(t *testing.T) {
|
||||
require.EqualValues(t, os.Args, resource.CommandArgs())
|
||||
require.Equal(t, os.Args, resource.CommandArgs())
|
||||
}
|
||||
|
||||
func TestRuntimeName(t *testing.T) {
|
||||
if runtime.Compiler == "gc" {
|
||||
require.EqualValues(t, "go", resource.RuntimeName())
|
||||
require.Equal(t, "go", resource.RuntimeName())
|
||||
} else {
|
||||
require.EqualValues(t, runtime.Compiler, resource.RuntimeName())
|
||||
require.Equal(t, runtime.Compiler, resource.RuntimeName())
|
||||
}
|
||||
}
|
||||
|
||||
func TestRuntimeOS(t *testing.T) {
|
||||
require.EqualValues(t, runtime.GOOS, resource.RuntimeOS())
|
||||
require.Equal(t, runtime.GOOS, resource.RuntimeOS())
|
||||
}
|
||||
|
||||
func TestRuntimeArch(t *testing.T) {
|
||||
require.EqualValues(t, runtime.GOARCH, resource.RuntimeArch())
|
||||
require.Equal(t, runtime.GOARCH, resource.RuntimeArch())
|
||||
}
|
||||
|
||||
func testWithProcessExecutablePathError(t *testing.T) {
|
||||
@@ -131,7 +131,7 @@ func testWithProcessExecutablePathError(t *testing.T) {
|
||||
)
|
||||
|
||||
require.Error(t, err)
|
||||
require.EqualValues(t, map[string]string{}, toMap(res))
|
||||
require.Equal(t, map[string]string{}, toMap(res))
|
||||
}
|
||||
|
||||
func testWithProcessOwnerError(t *testing.T) {
|
||||
@@ -142,5 +142,5 @@ func testWithProcessOwnerError(t *testing.T) {
|
||||
)
|
||||
|
||||
require.Error(t, err)
|
||||
require.EqualValues(t, map[string]string{}, toMap(res))
|
||||
require.Equal(t, map[string]string{}, toMap(res))
|
||||
}
|
||||
|
@@ -184,7 +184,7 @@ func TestMerge(t *testing.T) {
|
||||
} else {
|
||||
assert.NoError(t, err)
|
||||
}
|
||||
assert.EqualValues(t, c.schemaURL, res.SchemaURL())
|
||||
assert.Equal(t, c.schemaURL, res.SchemaURL())
|
||||
if diff := cmp.Diff(
|
||||
res.Attributes(),
|
||||
c.want,
|
||||
@@ -197,8 +197,8 @@ func TestMerge(t *testing.T) {
|
||||
|
||||
func TestEmpty(t *testing.T) {
|
||||
var res *resource.Resource
|
||||
assert.Equal(t, "", res.SchemaURL())
|
||||
assert.Equal(t, "", res.String())
|
||||
assert.Empty(t, res.SchemaURL())
|
||||
assert.Empty(t, res.String())
|
||||
assert.Equal(t, []attribute.KeyValue(nil), res.Attributes())
|
||||
|
||||
it := res.Iter()
|
||||
@@ -441,12 +441,12 @@ func TestNew(t *testing.T) {
|
||||
assert.NoError(t, err)
|
||||
}
|
||||
|
||||
require.EqualValues(t, tt.resourceValues, toMap(res))
|
||||
require.Equal(t, tt.resourceValues, toMap(res))
|
||||
|
||||
// TODO: do we need to ensure that resource is never nil and eliminate the
|
||||
// following if?
|
||||
if res != nil {
|
||||
assert.EqualValues(t, tt.schemaURL, res.SchemaURL())
|
||||
assert.Equal(t, tt.schemaURL, res.SchemaURL())
|
||||
}
|
||||
})
|
||||
}
|
||||
@@ -482,7 +482,7 @@ func TestWithHostID(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"host.id": "f2c668b579780554f70f72a063dc0864",
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -498,7 +498,7 @@ func TestWithHostIDError(t *testing.T) {
|
||||
)
|
||||
|
||||
assert.ErrorIs(t, err, assert.AnError)
|
||||
require.EqualValues(t, map[string]string{}, toMap(res))
|
||||
require.Equal(t, map[string]string{}, toMap(res))
|
||||
}
|
||||
|
||||
func TestWithOSType(t *testing.T) {
|
||||
@@ -512,7 +512,7 @@ func TestWithOSType(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"os.type": "linux",
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -528,7 +528,7 @@ func TestWithOSDescription(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"os.description": "Test",
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -544,7 +544,7 @@ func TestWithOS(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"os.type": "linux",
|
||||
"os.description": "Test",
|
||||
}, toMap(res))
|
||||
@@ -559,7 +559,7 @@ func TestWithProcessPID(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.pid": fmt.Sprint(fakePID),
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -573,7 +573,7 @@ func TestWithProcessExecutableName(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.executable.name": fakeExecutableName,
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -587,7 +587,7 @@ func TestWithProcessExecutablePath(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.executable.path": fakeExecutablePath,
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -602,7 +602,7 @@ func TestWithProcessCommandArgs(t *testing.T) {
|
||||
|
||||
require.NoError(t, err)
|
||||
jsonCommandArgs, _ := json.Marshal(fakeCommandArgs)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.command_args": string(jsonCommandArgs),
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -616,7 +616,7 @@ func TestWithProcessOwner(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.owner": fakeOwner,
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -630,7 +630,7 @@ func TestWithProcessRuntimeName(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.runtime.name": fakeRuntimeName,
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -644,7 +644,7 @@ func TestWithProcessRuntimeVersion(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.runtime.version": fakeRuntimeVersion,
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -658,7 +658,7 @@ func TestWithProcessRuntimeDescription(t *testing.T) {
|
||||
)
|
||||
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.runtime.description": fakeRuntimeDescription,
|
||||
}, toMap(res))
|
||||
}
|
||||
@@ -673,7 +673,7 @@ func TestWithProcess(t *testing.T) {
|
||||
|
||||
require.NoError(t, err)
|
||||
jsonCommandArgs, _ := json.Marshal(fakeCommandArgs)
|
||||
require.EqualValues(t, map[string]string{
|
||||
require.Equal(t, map[string]string{
|
||||
"process.pid": fmt.Sprint(fakePID),
|
||||
"process.executable.name": fakeExecutableName,
|
||||
"process.executable.path": fakeExecutablePath,
|
||||
|
@@ -192,7 +192,7 @@ func TestSchemaURL(t *testing.T) {
|
||||
|
||||
// Verify that the SchemaURL of the constructed Tracer is correctly populated.
|
||||
tracerStruct := tracerIface.(*tracer)
|
||||
assert.EqualValues(t, schemaURL, tracerStruct.instrumentationScope.SchemaURL)
|
||||
assert.Equal(t, schemaURL, tracerStruct.instrumentationScope.SchemaURL)
|
||||
}
|
||||
|
||||
func TestRegisterAfterShutdownWithoutProcessors(t *testing.T) {
|
||||
|
@@ -1583,7 +1583,7 @@ func TestReadOnlySpan(t *testing.T) {
|
||||
assert.Equal(t, kv.Key, ro.Events()[0].Attributes[0].Key)
|
||||
assert.Equal(t, kv.Value, ro.Events()[0].Attributes[0].Value)
|
||||
assert.Equal(t, codes.Ok, ro.Status().Code)
|
||||
assert.Equal(t, "", ro.Status().Description)
|
||||
assert.Empty(t, ro.Status().Description)
|
||||
assert.Equal(t, "ReadOnlySpan", ro.InstrumentationLibrary().Name)
|
||||
assert.Equal(t, "3", ro.InstrumentationLibrary().Version)
|
||||
assert.Equal(t, "ReadOnlySpan", ro.InstrumentationScope().Name)
|
||||
|
Reference in New Issue
Block a user