uros-b opened a new pull request, #57518: URL: https://github.com/apache/spark/pull/57518
### What changes were proposed in this pull request? "`DateTimeUtil.formatTimestampMillis` is a public helper used to build user-facing timestamps in error messages (e.g. in `RemoveSnapshots`), but it had no direct unit test. This adds a focused JUnit5 + AssertJ test covering whole-second, fractional-millisecond, epoch, and pre-epoch (negative) values. ### Why are the changes needed? Improve test coverage. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Expected strings were verified against the actual implementation on a JVM. Test-only; no production code changed. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Opus 4.8 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
