Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: openrewrite/rewrite-testing-frameworks
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v3.32.0
Choose a base ref
...
head repository: openrewrite/rewrite-testing-frameworks
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v3.33.0
Choose a head ref
  • 7 commits
  • 37 files changed
  • 3 contributors

Commits on Mar 30, 2026

  1. OpenRewrite recipe best practices (#949)

    * OpenRewrite recipe best practices
    
    Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.recipes.rewrite.OpenRewriteRecipeBestPractices?organizationId=QUxML01vZGVybmUvTW9kZXJuZSArIE9wZW5SZXdyaXRl
    
    Co-authored-by: Moderne <[email protected]>
    
    * Fix compilation errors: cast visitor return type to MethodDeclaration
    
    RemoveAnnotationVisitor.visit() and RemoveAnnotation.getVisitor().visit()
    return generic J, but the calling methods expect J.MethodDeclaration.
    Added explicit casts in three files to match the existing pattern.
    
    ---------
    
    Co-authored-by: Moderne <[email protected]>
    timtebeek and TeamModerne authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    130d158 View commit details
    Browse the repository at this point in the history
  2. Add RemoveDoNothingForDefaultMocks recipe (#947)

    * Add RemoveDoNothingForDefaultMocks recipe (#946)
    
    Remove unnecessary `doNothing()` stubbings on `@Mock` fields that cause
    UnnecessaryStubbingException after migrating to Mockito 3+.
    
    * Remove redundant `false` from MethodMatcher constructors
    
    `false` is the default for `matchOverrides`, so the single-arg
    constructor is sufficient and consistent with the rest of the codebase.
    
    ---------
    
    Co-authored-by: Tim te Beek <[email protected]>
    bmuschko and timtebeek authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    d4b1c86 View commit details
    Browse the repository at this point in the history
  3. Add PowerMockitoDoStubbingToMockito recipe (#948)

    * Add PowerMockitoDoStubbingToMockito recipe (#946)
    
    Rewrite PowerMockito's string-based private method stubbing into
    standard Mockito chained form so migrated code compiles.
    
    * Fix CSV escaping for fields containing commas and quotes
    
    * Refine PowerMockitoDoStubbingToMockito per review feedback
    
    Use UsesMethod precondition instead of UsesType for more precise
    matching, static imports for Collections methods, emptyList() for
    no-arg calls, and ListUtils.mapFirst to simplify arg remapping.
    
    ---------
    
    Co-authored-by: Tim te Beek <[email protected]>
    bmuschko and timtebeek authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    04c94f8 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    135a45b View commit details
    Browse the repository at this point in the history
  5. Add AssertEqualsIntegralDeltaToAssertEquals recipe (#950)

    * Add AssertEqualsIntegralDeltaToAssertEquals recipe (#869)
    
    Remove the unnecessary delta/precision argument from assertEquals when
    both expected and actual are int or long types, since the delta is
    meaningless for exact integer comparison.
    
    * Inline JavaParser initializer instead of caching in field
    
    * Update generated recipes.csv
    timtebeek authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    8aeb674 View commit details
    Browse the repository at this point in the history
  6. Add AssertEqualsIntegralDeltaToAssertEquals recipe (#950)

    * Add AssertEqualsIntegralDeltaToAssertEquals recipe (#869)
    
    Remove the unnecessary delta/precision argument from assertEquals when
    both expected and actual are int or long types, since the delta is
    meaningless for exact integer comparison.
    
    * Inline JavaParser initializer instead of caching in field
    
    * Update generated recipes.csv
    timtebeek authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    9ae6e43 View commit details
    Browse the repository at this point in the history

Commits on Mar 31, 2026

  1. Inline JavaParser and JavaTemplate fields in recipes (#951)

    * Inline JavaParser and JavaTemplate fields in recipes
    
    Remove the legacy pattern of storing JavaParser.Builder and JavaTemplate
    instances as cached fields on visitors, inlining them at each usage site
    instead.
    
    * Also inline javaParser helper methods
    
    Address PR feedback to fully inline the javaParser(ctx) helper methods
    in JUnitParamsRunnerToParameterized and ParameterizedRunnerToParameterized.
    timtebeek authored Mar 31, 2026
    Configuration menu
    Copy the full SHA
    df743de View commit details
    Browse the repository at this point in the history
Loading