-
-
Notifications
You must be signed in to change notification settings - Fork 27.3k
Update BallThreadTest.java #3320
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
base: master
Are you sure you want to change the base?
Conversation
fix: resolve Spotless formatting violations in BallThreadTest - Restore required imports for ArgumentMatchers.any and eq - Fix import ordering to follow alphabetical convention - Add proper spacing between imports and class declaration - Remove commented-out import statements Fixes Maven build failure caused by spotless:check violations.
PR SummaryThis PR fixes Spotless code formatting violations in the Changes
autogenerated by presubmit.ai |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ LGTM!
Review Summary
Commits Considered (1)
- 2f7eb7b: Update BallThreadTest.java
fix: resolve Spotless formatting violations in BallThreadTest
- Restore required imports for ArgumentMatchers.any and eq
- Fix import ordering to follow alphabetical convention
- Add proper spacing between imports and class declaration
- Remove commented-out import statements
Fixes Maven build failure caused by spotless:check violations.
Files Processed (1)
- twin/src/test/java/com/iluwatar/twin/BallThreadTest.java (3 hunks)
Actionable Comments (0)
Skipped Comments (0)
|
|
The original issue was caused by unused imports that were commented out, but the code still referenced eq() and any() methods in the test. This created a mismatch between what was imported and what was actually used, triggering Spotless formatting violations. Now its Fixed |
|
This PR is stale because it has been open 60 days with no activity. |



What does this PR do?
This PR resolves Spotless code formatting violations that were causing Maven build failures in the
twinmodule.Fixes #3303
Changes made:
ArgumentMatchers.anyandArgumentMatchers.eqthat are used in thetestInterrupt()methodRoot cause:
The original issue was caused by unused imports that were commented out, but the code still referenced
eq()andany()methods in the test. This created a mismatch between what was imported and what was actually used, triggering Spotless formatting violations.Testing:
mvn clean compilemvn spotless:checkThis change ensures the codebase maintains consistent formatting standards while preserving all test functionality.