chore(deps): update dependency org.assertj:assertj-core to v3.27.7 [security]#3385
Conversation
7fb47a6 to
2bf4d20
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3385 +/- ##
===========================================
+ Coverage 71.99% 72.04% +0.05%
===========================================
Files 956 956
Lines 35307 35307
Branches 5861 5861
===========================================
+ Hits 25418 25436 +18
+ Misses 8284 8264 -20
- Partials 1605 1607 +2 🚀 New features to boost your workflow:
|
2bf4d20 to
5cf0076
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5cf0076f2b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| /** | ||
| * A fake placeholder class to let AssertJ work in instrumented tests for API<24. | ||
| */ | ||
| class DirectoryStream |
There was a problem hiding this comment.
For low-API instrumented tests, this only defines java.nio.file.DirectoryStream, but AssertJ 3.27.7's org.assertj.core.internal.NioFilesWrapper.newDirectoryStream(...) also declares a java.nio.file.DirectoryStream.Filter parameter. On devices that lack java.nio.file, loading the AssertJ path helper will still fail with NoClassDefFoundError for DirectoryStream$Filter, so the dependency update remains broken for those instrumented tests unless the nested Filter type is stubbed as well.
Useful? React with 👍 / 👎.
| assertThat(readResult).satisfies { it.all { it.data.isNotEmpty() } } | ||
| assertThat(readResult).satisfies { it.all { it.metadata.isEmpty() } } | ||
| assertThat(readResult).allMatch { it.data.isNotEmpty() } | ||
| assertThat(readResult).allMatch { it.metadata.isNotEmpty() } |
3 participants
This PR contains the following updates:
3.18.1→3.27.7Warning
Some dependencies could not be looked up. Check the warning logs for more information.
🚨 Security Vulnerability Fix
Severity: HIGH
This PR addresses a security vulnerability detected by GitHub Dependabot. Please review and merge promptly.