Add integration tests for ksp library modules. #5054
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I've added a new module ksp:integrationtest so that we can keep the pollution from the dependent module out of the normal library glide module tests. There's more detail in the documentation for the build.gradle file in ksp:integrationtests. This does at least make sure that if a dependent module is added, it is succesfully included by ksp. Previously the tests only tested either library modules compiled with the app glide module or library modules compiled by ksp. This new set of tests now tests library modules compiled by java. If we ever compile a module with ksp, we should add that module to these tests as well so that we have integration tests for both types of modules. For now all modules are compiled only with the java processor. In an ideal world we'd be able to just run the Java annotation processor using kotlin compile testing the same way we are with ksp. The library seems to support it. However I can't figure out a way to get the javax annotations and kotlin and android all available in a single gradle module. For now using a separate module is reasonable (if somewhat clunky) compromise.
nikclayton
referenced
this pull request
in pachli/pachli-android
Oct 13, 2023
[](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.github.bumptech.glide:okhttp3-integration](https://togithub.com/bumptech/glide) | `4.15.1` -> `4.16.0` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [com.github.bumptech.glide:glide](https://togithub.com/bumptech/glide) | `4.15.1` -> `4.16.0` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [com.github.bumptech.glide:ksp](https://togithub.com/bumptech/glide) | `4.15.1` -> `4.16.0` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>bumptech/glide (com.github.bumptech.glide:okhttp3-integration)</summary> ### [`v4.16.0`](https://togithub.com/bumptech/glide/releases/tag/v4.16.0): Glide 4.16.0 This release focuses on some build improvements and Compose. The two major Compose improvements are adding support for Compose specific transitions (e.g. cross fade) and supporting recomposition based on request state using `GlideSubcomposition`. There's also been a bunch of internal refactoring to move away from Painters to Modifier nodes based on feedback from the Compose team. This is still an alpha release of Compose, but barring unexpectedly negative feedback, the next release should be beta. This should be the last release of Glide that targets Java 7. That probably means our next release will be a major version change. ##### Features - Allow passing an executor into ChromiumRequestSerializer in [https://github.com/bumptech/glide/pull/5077](https://togithub.com/bumptech/glide/pull/5077) - Allow host app to provide a way to clear all resources onStop() by [@​osamaaftab](https://togithub.com/osamaaftab) in [https://github.com/bumptech/glide/pull/5145](https://togithub.com/bumptech/glide/pull/5145) ##### Compose - Add a Transition API and a CrossFade Transition for Compose by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5235](https://togithub.com/bumptech/glide/pull/5235) - Influence layout using intrinsics in GlideNode by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5240](https://togithub.com/bumptech/glide/pull/5240) \* Log instead of throwing parsing manifests to fix compose previews by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5167](https://togithub.com/bumptech/glide/pull/5167) - Launch no more than one request per onRemembered by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5062](https://togithub.com/bumptech/glide/pull/5062) - Remove GlidePainter in favor of Modifier nodes / Flows by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5230](https://togithub.com/bumptech/glide/pull/5230) - Replace flows in GlideSubcomposition with a listener on GlideNode by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5238](https://togithub.com/bumptech/glide/pull/5238) ##### Bugs - Read library glide module names from Java indexes by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5052](https://togithub.com/bumptech/glide/pull/5052) - Fix typo. anay -> any in GlideSymbolProcessor.kt. by [@​trevorhackman](https://togithub.com/trevorhackman) in [https://github.com/bumptech/glide/pull/5029](https://togithub.com/bumptech/glide/pull/5029) - Include URL in error log by [@​paulsowden](https://togithub.com/paulsowden) in [https://github.com/bumptech/glide/pull/5164](https://togithub.com/bumptech/glide/pull/5164) - Add `isInitialized` visible for testing method by [@​paulsowden](https://togithub.com/paulsowden) in [https://github.com/bumptech/glide/pull/5163](https://togithub.com/bumptech/glide/pull/5163) - Use onIdle to avoid a race in FlowTests by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5202](https://togithub.com/bumptech/glide/pull/5202) - Add a isEquivalentTo method to correctly check equality by [@​mori-atsushi](https://togithub.com/mori-atsushi) in [https://github.com/bumptech/glide/pull/5232](https://togithub.com/bumptech/glide/pull/5232) - Add [@​RequiresPermission](https://togithub.com/RequiresPermission) to NotificationTarget by [@​TWiStErRob](https://togithub.com/TWiStErRob) in [https://github.com/bumptech/glide/pull/5220](https://togithub.com/bumptech/glide/pull/5220) ##### Deprecations - `placeholderOf(@​Composable)` in `GlideImage` is deprecated, use `GlideSubcomposition` instead. Keep in mind that using either forces a recomposition each time the state of the image load changes. Recomposition will have a significant performance penalty in scrolling lists and should be avoided. ##### Behavior Changes - Hard code disabling hardware bitmaps on O/OMR1. by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5115](https://togithub.com/bumptech/glide/pull/5115) - Do not set requireOriginal on Android photo picker uris. by [@​phoenixli](https://togithub.com/phoenixli) in [https://github.com/bumptech/glide/pull/5162](https://togithub.com/bumptech/glide/pull/5162) ##### Breaking Changes ##### Build Changes - Add integration tests for ksp library modules. by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5054](https://togithub.com/bumptech/glide/pull/5054) - Update README.md to use https by [@​simoarpe](https://togithub.com/simoarpe) in [https://github.com/bumptech/glide/pull/5058](https://togithub.com/bumptech/glide/pull/5058) - Use dokka to build scripts/update_javadocs.sh by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5104](https://togithub.com/bumptech/glide/pull/5104) - avif integration: Update libavif dependency by [@​vigneshvg](https://togithub.com/vigneshvg) in [https://github.com/bumptech/glide/pull/5128](https://togithub.com/bumptech/glide/pull/5128) - Disable java 7 source obsolete warning. by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5168](https://togithub.com/bumptech/glide/pull/5168) - Update mockito version to fix j16 compilation. by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5169](https://togithub.com/bumptech/glide/pull/5169) - Switch Glide's dependencies to a version catalog. by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5183](https://togithub.com/bumptech/glide/pull/5183) - Remove jetifier by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5184](https://togithub.com/bumptech/glide/pull/5184) - Add an updated proguard plugin to compile on Java 17. by [@​sjudd](https://togithub.com/sjudd) in [https://github.com/bumptech/glide/pull/5185](https://togithub.com/bumptech/glide/pull/5185) - Configure Renovate in [https://github.com/bumptech/glide/pull/5186](https://togithub.com/bumptech/glide/pull/5186) - Increment ROBOLECTRIC_SDK to 19 from 18. by [@​brettchabot](https://togithub.com/brettchabot) in [https://github.com/bumptech/glide/pull/5208](https://togithub.com/bumptech/glide/pull/5208) and [https://github.com/bumptech/glide/pull/5207](https://togithub.com/bumptech/glide/pull/5207) - AGP: Upgrade AndroidManifest.xml's package to build.gradle's namespace. by [@​TWiStErRob](https://togithub.com/TWiStErRob) in [https://github.com/bumptech/glide/pull/5221](https://togithub.com/bumptech/glide/pull/5221) ##### New Contributors - [@​trevorhackman](https://togithub.com/trevorhackman) made their first contribution in [https://github.com/bumptech/glide/pull/5029](https://togithub.com/bumptech/glide/pull/5029) - [@​simoarpe](https://togithub.com/simoarpe) made their first contribution in [https://github.com/bumptech/glide/pull/5058](https://togithub.com/bumptech/glide/pull/5058) - [@​paulsowden](https://togithub.com/paulsowden) made their first contribution in [https://github.com/bumptech/glide/pull/5164](https://togithub.com/bumptech/glide/pull/5164) - [@​phoenixli](https://togithub.com/phoenixli) made their first contribution in [https://github.com/bumptech/glide/pull/5162](https://togithub.com/bumptech/glide/pull/5162) - [@​osamaaftab](https://togithub.com/osamaaftab) made their first contribution in [https://github.com/bumptech/glide/pull/5145](https://togithub.com/bumptech/glide/pull/5145) - [@​brettchabot](https://togithub.com/brettchabot) made their first contribution in [https://github.com/bumptech/glide/pull/5207](https://togithub.com/bumptech/glide/pull/5207) - [@​mori-atsushi](https://togithub.com/mori-atsushi) made their first contribution in [https://github.com/bumptech/glide/pull/5232](https://togithub.com/bumptech/glide/pull/5232) **Full Changelog**: bumptech/glide@v4.15.0...v4.16.0 Note - there's been a change in the gpg key used to sign these releases. The new public key is attached </details> --- ### 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 these updates again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/pachli/pachli-android). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMDcuMiIsInVwZGF0ZWRJblZlciI6IjM3LjguMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Nik Clayton <[email protected]>
](https://renovatebot.com){kind=link}
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've added a new module ksp:integrationtest so that we can keep the pollution from the dependent module out of the normal library glide module tests. There's more detail in the documentation for the build.gradle file in ksp:integrationtests.
This does at least make sure that if a dependent module is added, it is successfully included by ksp. Previously the tests only tested either library modules compiled with the app glide module or library modules compiled by ksp. This new set of tests now tests library modules compiled by java. If we ever compile a module with ksp, we should add that module to these tests as well so that we have integration tests for both types of modules. For now all modules are compiled only with the java processor.
In an ideal world we'd be able to just run the Java annotation processor using kotlin compile testing the same way we are with ksp. The library seems to support it. However I can't figure out a way to get the javax annotations and kotlin and android all available in a single gradle module. For now using a separate module is reasonable (if somewhat clunky) compromise.