fatmanverse opened a new pull request, #12891:
URL: https://github.com/apache/seatunnel/pull/12891
Follow-up to #12801 (#12797). Test only, no production code change.
### Purpose of this pull request
The triage of #12797 asked the fix to "prove the zero-seconds case and a
source/sink round trip". #12801 covered the zero-seconds case in
`XmlWriterTest`, but no unit test reads the sink output back with the XML
source.
This adds `XmlWriterTest#testDefaultFormatsRoundTripThroughXmlSource`. It
writes rows with `XmlWriter` using the sink default formats, including a
timestamp with zero seconds, then reads the XML with `XmlReadStrategy` using
the source defaults and checks that the DATE, TIME and TIMESTAMP values come
back unchanged. It runs in element mode and attribute mode. Before #12801 the
source failed to parse `2026-01-02T10:13`.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
`./mvnw -pl seatunnel-connectors-v2/connector-file/connector-file-base test`
on JDK 17: all tests pass.
### Check list
* [ ] If any new Jar binary package adding in your PR, please add License
Notice according
[New License
Guide](https://github.com/apache/seatunnel/blob/dev/docs/en/developer/new-license.md)
* [x] If necessary, please update the documentation to describe the new
feature. https://github.com/apache/seatunnel/tree/dev/docs (not needed)
* [x] If necessary, please update `incompatible-changes.md` to describe the
incompatibility caused by this PR. (not needed)
--
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]
