Skip to content

Use BuiltinMethodId for openMocksMethodId#2603

Merged
IlyaMuravjov merged 1 commit into
mainfrom
ilya_m/builtin-open-mocks
Sep 15, 2023
Merged

Use BuiltinMethodId for openMocksMethodId#2603
IlyaMuravjov merged 1 commit into
mainfrom
ilya_m/builtin-open-mocks

Conversation

@IlyaMuravjov
Copy link
Copy Markdown
Collaborator

Description

Fixes #2602 (builtins should be used for Mockito methods)

How to test

Manual tests

#2602 should no longer reproduce.

Self-check list

  • I've set the proper labels for my PR (at least, for category and component).
  • PR title and description are clear and intelligible.
  • I've added enough comments to my code, particularly in hard-to-understand areas.
  • The functionality I've repaired, changed or added is covered with automated tests.
  • Manual tests have been provided optionally.
  • The documentation for the functionality I've been working on is up-to-date.

@IlyaMuravjov IlyaMuravjov added comp-codegen Issue is related to code generator ctg-bug-fix PR is fixing a bug comp-spring Issue is related to Spring projects support labels Sep 15, 2023
@IlyaMuravjov IlyaMuravjov merged commit 5dff7df into main Sep 15, 2023
@IlyaMuravjov IlyaMuravjov deleted the ilya_m/builtin-open-mocks branch September 15, 2023 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-codegen Issue is related to code generator comp-spring Issue is related to Spring projects support ctg-bug-fix PR is fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't find method openMocks(Ljava/lang/Object;)Ljava/lang/AutoCloseable when user has Mockito 2.1.0-beta.123

2 participants