Skip to content

Adjust inline mocking snippet to allow task relocatability #3606

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 14, 2025

Conversation

jprinet
Copy link
Contributor

@jprinet jprinet commented Mar 7, 2025

Issue

Inline mocking configuration snippet for Gradle should allow task relocatability

Fix

Use a CommandLineArgumentProvider

Fixes #3605

Checklist

  • Read the contributing guide
  • PR should be motivated, i.e. what does it fix, why, and if relevant how
  • If possible / relevant include an example in the description, that could help all readers
    including project members to get a better picture of the change
  • Avoid other runtime dependencies
  • Meaningful commit history ; intention is important please rebase your commit history so that each
    commit is meaningful and help the people that will explore a change in 2 years
  • The pull request follows coding style
  • Mention Fixes #<issue number> in the description if relevant
  • At least one commit should mention Fixes #<issue number> if relevant

@jprinet
Copy link
Contributor Author

jprinet commented Mar 7, 2025

Thanks @marcphilipp for noticing 🙏

@TimvdLippe TimvdLippe requested a review from bric3 March 7, 2025 10:44
@codecov-commenter
Copy link

codecov-commenter commented Mar 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.56%. Comparing base (c81be5d) to head (fba6c3e).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #3606   +/-   ##
=========================================
  Coverage     85.56%   85.56%           
  Complexity     2957     2957           
=========================================
  Files           341      341           
  Lines          9028     9028           
  Branches       1119     1119           
=========================================
  Hits           7725     7725           
  Misses         1013     1013           
  Partials        290      290           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TimvdLippe
Copy link
Contributor

@jprinet If you would like to keep track of how often this build code is copied, here's a GitHub search link: https://github.com/search?q=%22val+mockitoAgent+%3D+configurations.create%28%5C%22mockitoAgent%5C%22%29%22&type=code

@TimvdLippe TimvdLippe merged commit d185035 into mockito:main Mar 14, 2025
18 checks passed
svc-squareup-copybara pushed a commit to cashapp/misk that referenced this pull request Mar 17, 2025
| Package | Type | Package file | Manager | Update | Change |
|---|---|---|---|---|---|
| [org.mockito:mockito-core](https://github.com/mockito/mockito) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`5.16.0` -> `5.16.1` |
| [org.junit.jupiter:junit-jupiter-params](https://junit.org/junit5/)
([source](https://github.com/junit-team/junit5)) | dependencies |
misk/gradle/libs.versions.toml | gradle | patch | `5.12.0` -> `5.12.1` |
| [org.junit.jupiter:junit-jupiter-engine](https://junit.org/junit5/)
([source](https://github.com/junit-team/junit5)) | dependencies |
misk/gradle/libs.versions.toml | gradle | patch | `5.12.0` -> `5.12.1` |
| [org.junit.jupiter:junit-jupiter-api](https://junit.org/junit5/)
([source](https://github.com/junit-team/junit5)) | dependencies |
misk/gradle/libs.versions.toml | gradle | patch | `5.12.0` -> `5.12.1` |
| [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.31.0` -> `2.31.1` |
| [software.amazon.awssdk:sqs](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.31.0` -> `2.31.1` |
|
[software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.31.0` -> `2.31.1` |
| [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.31.0` -> `2.31.1` |
| [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.31.0` -> `2.31.1` |
| [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.31.0` -> `2.31.1` |
| [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.31.0` -> `2.31.1` |

---

### Release Notes

<details>
<summary>mockito/mockito (org.mockito:mockito-core)</summary>

### [`v5.16.1`](https://github.com/mockito/mockito/releases/tag/v5.16.1)

<sup><sup>*Changelog generated by [Shipkit Changelog Gradle
Plugin](https://github.com/shipkit/shipkit-changelog)*</sup></sup>

##### 5.16.1

- 2025-03-15 - [3
commit(s)](mockito/mockito@v5.16.0...v5.16.1)
by Adrian Roos, Jérôme Prinet, Rafael Winterhalter
- Remove Arrays.asList from critical stubbing path in GenericMetadataSu…
[(#&#8203;3610)](mockito/mockito#3610)
- Rework of injection strategy in the context of modules
[(#&#8203;3608)](mockito/mockito#3608)
- Adjust inline mocking snippet to allow task relocatability
[(#&#8203;3606)](mockito/mockito#3606)
- Inline mocking configuration snippet for Gradle should allow task
relocatability
[(#&#8203;3605)](mockito/mockito#3605)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am
every weekday" in timezone Australia/Melbourne, Automerge - At any time
(no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Never, or you tick the rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://github.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

GitOrigin-RevId: 1939516a6a445b5c4812a6d4f27f483cdb66b66e
philipp94831 added a commit to bakdata/gradle-plugins that referenced this pull request Mar 21, 2025
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.

Inline mocking configuration snippet for Gradle should allow task relocatability
4 participants