Skip to content

Fix testifylint violations in fail_on_violation_test.go#5

Closed
Copilot wants to merge 2 commits intoadd-fail-on-violationfrom
copilot/fix-linter-error
Closed

Fix testifylint violations in fail_on_violation_test.go#5
Copilot wants to merge 2 commits intoadd-fail-on-violationfrom
copilot/fix-linter-error

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 5, 2026

Changes

  • require-error: Replace assert.EqualError with require.EqualError for the error message assertion
  • error-is-as: Replace assert.True(t, errors.Is(...)) with assert.ErrorIs(...) directly
  • Remove now-unused "errors" import
// Before
assert.EqualError(t, ErrViolationsFound, "poutine: violations found")
assert.True(t, errors.Is(ErrViolationsFound, ErrViolationsFound))

// After
require.EqualError(t, ErrViolationsFound, "poutine: violations found")
assert.ErrorIs(t, ErrViolationsFound, ErrViolationsFound)

Co-authored-by: mbarbero <452625+mbarbero@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix error raised by linter in PR #392 Fix testifylint violations in fail_on_violation_test.go Mar 5, 2026
@mbarbero mbarbero closed this Mar 5, 2026
@mbarbero mbarbero deleted the copilot/fix-linter-error branch March 5, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants