Skip to content

fix(deps): update module github.com/stretchr/testify to v1.11.1 (main)#7448

Merged
antoninbas merged 2 commits intomainfrom
renovate/main-github.com-stretchr-testify-1.x
Sep 23, 2025
Merged

fix(deps): update module github.com/stretchr/testify to v1.11.1 (main)#7448
antoninbas merged 2 commits intomainfrom
renovate/main-github.com-stretchr-testify-1.x

Conversation

@renovate
Copy link
Copy Markdown
Contributor

@renovate renovate Bot commented Sep 17, 2025

Coming soon: The Renovate bot (GitHub App) will be renamed to Mend. PRs from Renovate will soon appear from 'Mend'. Learn more here.

This PR contains the following updates:

Package Change Age Confidence
github.com/stretchr/testify v1.10.0 -> v1.11.1 age confidence

Release Notes

stretchr/testify (github.com/stretchr/testify)

v1.11.1

Compare Source

This release fixes #​1785 introduced in v1.11.0 where expected argument values implementing the stringer interface (String() string) with a method which mutates their value, when passed to mock.Mock.On (m.On("Method", <expected>).Return()) or actual argument values passed to mock.Mock.Called may no longer match one another where they previously did match. The behaviour prior to v1.11.0 where the stringer is always called is restored. Future testify releases may not call the stringer method at all in this case.

What's Changed

Full Changelog: stretchr/testify@v1.11.0...v1.11.1

v1.11.0

Compare Source

What's Changed

Functional Changes

v1.11.0 Includes a number of performance improvements.

Fixes
Documentation, Build & CI

New Contributors

Full Changelog: stretchr/testify@v1.10.0...v1.11.0


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 becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate Bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Sep 17, 2025
@renovate renovate Bot changed the title fix(deps): update module github.com/stretchr/testify to v1.11.1 (main) Update module github.com/stretchr/testify to v1.11.1 (main) Sep 17, 2025
@renovate renovate Bot changed the title Update module github.com/stretchr/testify to v1.11.1 (main) fix(deps): update module github.com/stretchr/testify to v1.11.1 (main) Sep 17, 2025
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate renovate Bot force-pushed the renovate/main-github.com-stretchr-testify-1.x branch from 3a3b007 to 64994d5 Compare September 17, 2025 20:44
@antoninbas
Copy link
Copy Markdown
Contributor

/test-all

@antoninbas
Copy link
Copy Markdown
Contributor

Some test failures that I need to resolve manually. I had the same issue with vmware/go-ipfix.

@antoninbas antoninbas self-assigned this Sep 23, 2025
The tests were not correct to begin with, and a change in the
`assert.Eventually` implementation exposed the bug:
stretchr/testify#1427

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
@renovate
Copy link
Copy Markdown
Contributor Author

renovate Bot commented Sep 23, 2025

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@antoninbas
Copy link
Copy Markdown
Contributor

@luolanzone some manual fixes were required as 2 unit tests started failing after the dependency upgrade (because of this change: stretchr/testify#1427). PTAL.

@luolanzone
Copy link
Copy Markdown
Contributor

/test-all

@antoninbas antoninbas merged commit 03f9e58 into main Sep 23, 2025
57 of 61 checks passed
@antoninbas antoninbas deleted the renovate/main-github.com-stretchr-testify-1.x branch September 23, 2025 17:39
antoninbas added a commit to antoninbas/antrea that referenced this pull request Sep 23, 2025
This is a follow-up to antrea-io#7448.
The change in behavor of assert.Eventually has caused to the unit test
to become flaky. This change improves how the result of egress selection
is validated, and resolves the flakiness issue.
It is expected that the test function can be greatly improved when we
upgrade to Go 1.25 and we can start using the new testing/synctest
package. Upgrading to Go 1.25 is not immediately possible because of
some dependency issues.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
antoninbas added a commit to antoninbas/antrea that referenced this pull request Sep 23, 2025
This is a follow-up to antrea-io#7448.
The change in behavor of assert.Eventually has caused to the unit test
to become flaky. This change improves how the result of egress selection
is validated, and resolves the flakiness issue.
It is expected that the test function can be greatly improved when we
upgrade to Go 1.25 and we can start using the new testing/synctest
package. Upgrading to Go 1.25 is not immediately possible because of
some dependency issues.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
antoninbas added a commit to antoninbas/antrea that referenced this pull request Sep 24, 2025
The test had an invalid usage of assert.EventuallyWithT: the assertions
included in the condition function applied to the outer *testing.T
instead of to the condition function parameter.

After antrea-io#7448, the test started failing due to the change in the
implementation of assert.EventuallyWithT.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
luolanzone pushed a commit that referenced this pull request Sep 24, 2025
The test had an invalid usage of assert.EventuallyWithT: the assertions
included in the condition function applied to the outer *testing.T
instead of to the condition function parameter.

After #7448, the test started failing due to the change in the
implementation of assert.EventuallyWithT.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
antoninbas added a commit that referenced this pull request Oct 6, 2025
This is a follow-up to #7448.
The change in behavor of assert.Eventually has caused to the unit test
to become flaky. This change improves how the result of egress selection
is validated, and resolves the flakiness issue.
It is expected that the test function can be greatly improved when we
upgrade to Go 1.25 and we can start using the new testing/synctest
package. Upgrading to Go 1.25 is not immediately possible because of
some dependency issues.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
luolanzone pushed a commit to luolanzone/antrea that referenced this pull request Mar 24, 2026
antrea-io#7448)

* fix(deps): update module github.com/stretchr/testify to v1.11.1

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Fix race conditions in unit tests

The tests were not correct to begin with, and a change in the
`assert.Eventually` implementation exposed the bug:
stretchr/testify#1427

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>

---------

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Antonin Bas <antonin.bas@broadcom.com>
luolanzone pushed a commit to luolanzone/antrea that referenced this pull request Mar 25, 2026
…7469)

The test had an invalid usage of assert.EventuallyWithT: the assertions
included in the condition function applied to the outer *testing.T
instead of to the condition function parameter.

After antrea-io#7448, the test started failing due to the change in the
implementation of assert.EventuallyWithT.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
luolanzone pushed a commit to luolanzone/antrea that referenced this pull request Mar 25, 2026
…7469)

The test had an invalid usage of assert.EventuallyWithT: the assertions
included in the condition function applied to the outer *testing.T
instead of to the condition function parameter.

After antrea-io#7448, the test started failing due to the change in the
implementation of assert.EventuallyWithT.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
luolanzone added a commit that referenced this pull request Mar 26, 2026
…tretchr/testify to v1.11.1 #7433 to update module github.com/spf13/pflag to v1.0.10 #7469 Fix typo in TestPreventDefunctRuleReuse (#7886)

* fix(deps): update module github.com/stretchr/testify to v1.11.1 (main) (#7448)

* fix(deps): update module github.com/stretchr/testify to v1.11.1

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Fix race conditions in unit tests

The tests were not correct to begin with, and a change in the
`assert.Eventually` implementation exposed the bug:
stretchr/testify#1427

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>

---------

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Antonin Bas <antonin.bas@broadcom.com>

* fix(deps): update module github.com/spf13/pflag to v1.0.10 (#7433)

Signed-off-by: Lan Luo <lan.luo@broadcom.com>

* [Flaky unit test] Fix typo in TestPreventDefunctRuleReuse (#7469)

The test had an invalid usage of assert.EventuallyWithT: the assertions
included in the condition function applied to the outer *testing.T
instead of to the condition function parameter.

After #7448, the test started failing due to the change in the
implementation of assert.EventuallyWithT.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>

---------

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
Signed-off-by: Lan Luo <lan.luo@broadcom.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Antonin Bas <antonin.bas@broadcom.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants