Repository navigation
Rename TestRecord - #40758
Rename TestRecord#40758aIbrahiim wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
📋 Review Summary
This PR renames nested Record helper classes in several unit tests (CloningBundleFactoryTest, CloudObjectsTest, and CreateTest) to TestRecord to resolve ErrorProne JavaLangClash warnings against java.lang.Record on Java 17+. The renaming is accurate and complete across all three test files, though a few modified lines now exceed the 100-character limit enforced by Spotless (google-java-format).
🔍 General Feedback
- Please run
./gradlew spotlessApply(or wrap the lines flagged below) so thebeam_PreCommit_Spotlesscheck passes.
💡 Tip: Comment /gemini review on this pull request to trigger a new Gemini code review.
|
Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment |
1f8d1c2 to
2213e54
Compare
|
Assigning reviewers: R: @ahmedabu98 for label java. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #40758 +/- ##
===========================================
- Coverage 80.04% 0 -80.05%
===========================================
Files 490 0 -490
Lines 82916 0 -82916
===========================================
- Hits 66373 0 -66373
+ Misses 16543 0 -16543
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Fixes: #39472