Skip to content

Pin AssertJ to stable 3.27.7#913

Merged
timtebeek merged 2 commits intomainfrom
timtebeek/fix-ci-failure
Feb 21, 2026
Merged

Pin AssertJ to stable 3.27.7#913
timtebeek merged 2 commits intomainfrom
timtebeek/fix-ci-failure

Conversation

@timtebeek
Copy link
Member

Pins AssertJ to stable 3.27.7 to avoid corrupted SNAPSHOT TypeTable cache entries that cause CI test failures.

The dynamic version constraint 3.+ resolves to SNAPSHOT versions
(3.27.7-SNAPSHOT, 3.27.8-SNAPSHOT) from Maven snapshots repository,
causing a corrupted TypeTable cache entry. Fix AdoptAssertJDurationAssertionsTest
failure by pinning to stable release 3.27.7.
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Feb 21, 2026
@timtebeek timtebeek marked this pull request as draft February 21, 2026 12:10
@timtebeek
Copy link
Member Author

timtebeek commented Feb 21, 2026

Considering a change to https://github.com/openrewrite/rewrite-build-gradle-plugin as an alternative here, to avoid resolving snapshots for type tables, as that has a risk of leading to fragile tests / type tables.

@timtebeek timtebeek marked this pull request as ready for review February 21, 2026 12:41
@timtebeek timtebeek merged commit 00bc57f into main Feb 21, 2026
1 check passed
@timtebeek timtebeek deleted the timtebeek/fix-ci-failure branch February 21, 2026 12:42
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Feb 21, 2026
mergify bot added a commit to robfrank/linklift that referenced this pull request Feb 26, 2026
…rom 3.27.0 to 3.28.0 [skip ci]

Bumps [org.openrewrite.recipe:rewrite-testing-frameworks](https://github.com/openrewrite/rewrite-testing-frameworks) from 3.27.0 to 3.28.0.
Release notes

*Sourced from [org.openrewrite.recipe:rewrite-testing-frameworks's releases](https://github.com/openrewrite/rewrite-testing-frameworks/releases).*

> 3.28.0
> ------
>
> What's Changed
> --------------
>
> * Fix 6 issues in AssertJ recipe migrations by [`@​timtebeek`](https://github.com/timtebeek) in [openrewrite/rewrite-testing-frameworks#911](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/911)
> * Preserve throws clause when checked exceptions exist outside assertThrows. by [`@​motlin`](https://github.com/motlin) in [openrewrite/rewrite-testing-frameworks#899](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/899)
> * Pin AssertJ to stable 3.27.7 by [`@​timtebeek`](https://github.com/timtebeek) in [openrewrite/rewrite-testing-frameworks#913](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/913)
> * Add .contextSensitive() to Hamcrest conversion templates and improve type matching in `CollapseConsecutiveAssertThatStatements` by [`@​timtebeek`](https://github.com/timtebeek) in [openrewrite/rewrite-testing-frameworks#912](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/912)
> * Fix 2 AssertJ migration recipe issues found via large-scale validation by [`@​timtebeek`](https://github.com/timtebeek) in [openrewrite/rewrite-testing-frameworks#914](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/914)
> * Add Hamcrest to AssertJ recipes for hasProperty, everyItem, and hasItem(Matcher) by [`@​timtebeek`](https://github.com/timtebeek) in [openrewrite/rewrite-testing-frameworks#916](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/916)
> * Add missing Hamcrest not(Matcher) to AssertJ conversions by [`@​timtebeek`](https://github.com/timtebeek) in [openrewrite/rewrite-testing-frameworks#917](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/917)
> * Do not incorrectly strip method calls from assertion arguments by [`@​timtebeek`](https://github.com/timtebeek) in [openrewrite/rewrite-testing-frameworks#918](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/pull/918)
>
> **Full Changelog**: <openrewrite/rewrite-testing-frameworks@v3.27.0...v3.28.0>


Commits

* [`c2860b4`](openrewrite/rewrite-testing-frameworks@c2860b4) OpenRewrite recipe best practices
* [`c75ba77`](openrewrite/rewrite-testing-frameworks@c75ba77) Do not incorrectly strip method calls from assertion arguments ([#918](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/issues/918))
* [`bf27539`](openrewrite/rewrite-testing-frameworks@bf27539) Add missing Hamcrest `not(Matcher)` to AssertJ conversions ([#866](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/issues/866)) ([#917](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/issues/917))
* [`4f3f340`](openrewrite/rewrite-testing-frameworks@4f3f340) Update Gradle wrapper to 9.3.1
* [`a850e44`](openrewrite/rewrite-testing-frameworks@a850e44) Add Hamcrest to AssertJ recipes for hasProperty, everyItem, and hasItem(Match...
* [`16f1501`](openrewrite/rewrite-testing-frameworks@16f1501) Fix 2 AssertJ migration recipe issues found via large-scale validation ([#914](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/issues/914))
* [`c270c74`](openrewrite/rewrite-testing-frameworks@c270c74) Add .contextSensitive() to Hamcrest conversion templates and improve type mat...
* [`00bc57f`](openrewrite/rewrite-testing-frameworks@00bc57f) Pin AssertJ to stable 3.27.7 ([#913](https://redirect.github.com/openrewrite/rewrite-testing-frameworks/issues/913))
* [`36ebbba`](openrewrite/rewrite-testing-frameworks@36ebbba) Preserve throws clause when checked exceptions exist outside assertThrows. (#...
* [`58d2238`](openrewrite/rewrite-testing-frameworks@58d2238) OpenRewrite recipe best practices
* Additional commits viewable in [compare view](openrewrite/rewrite-testing-frameworks@v3.27.0...v3.28.0)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=org.openrewrite.recipe:rewrite-testing-frameworks&package-manager=maven&previous-version=3.27.0&new-version=3.28.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant