Skip to content

Fix: Add ErrorNotContains assertion - #1942

Open
TheRodzz wants to merge 1 commit into
stretchr:masterfrom
TheRodzz:add-error-not-contains
Open

Fix: Add ErrorNotContains assertion#1942
TheRodzz wants to merge 1 commit into
stretchr:masterfrom
TheRodzz:add-error-not-contains

Conversation

@TheRodzz

Copy link
Copy Markdown

Fixes #1940. Adds ErrorNotContains and its corresponding formatted/wrapper variants to complement ErrorContains, along with comprehensive test coverage.

@ccoVeille

Copy link
Copy Markdown
Collaborator

I feel like you didn't generate the files, or did it the wrong way.

The CI fails.

Could you please check?

fredbi added a commit to go-openapi/testify that referenced this pull request Aug 23, 2026
The August sweep of stretchr/testify turned up four reports describing bugs we
had inherited. Each now has a row in the implemented table saying what we
adopted and what we left alone, and stretchr#1942 joins stretchr#1940 as the PR behind
ErrorNotContains.

stretchr#1776 goes in as informational: upstream's generated require documentation
lists a bool return that the functions do not have, the same defect our own API
pages carried until the signature tables were corrected. Nothing to adopt, but
worth recording that both projects found it independently.

Two more bugs surfaced while checking those four, neither of them reported
upstream, so neither gets a row: InDeltaSlice named expected and actual the
wrong way round in its failures, and EqualValues compared a negative signed
integer equal to a large unsigned one.

Also in this file: the review-frequency line was three months stale, [stretchr#1937]
was referenced with no link definition and rendered as literal text, three
definitions were duplicated, and the summary counts are recounted from the
tables.

reference: github.com/stretchr#1776
reference: github.com/stretchr#1874
reference: github.com/stretchr#1875
reference: github.com/stretchr#1898
reference: github.com/stretchr#1899
reference: github.com/stretchr#1908
reference: github.com/stretchr#1931
reference: github.com/stretchr#1940
reference: github.com/stretchr#1942

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@TheRodzz
TheRodzz force-pushed the add-error-not-contains branch 2 times, most recently from 02126a8 to 26889b9 Compare August 25, 2026 03:42
@TheRodzz
TheRodzz force-pushed the add-error-not-contains branch from 26889b9 to 188fc30 Compare August 25, 2026 03:42
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.

Add ErrorNotContains assertion

2 participants