codewarnab opened a new pull request, #12151: URL: https://github.com/apache/arrow-rs/pull/12151
# Which issue does this PR close? - Part of #11033 # Rationale for this change Continue splitting public cast tests out of the large inline test module. Why this group next: of what remains in `cast/mod.rs` (about 75 timestamp/time tests, about 15 date tests, and these 10 interval/duration tests), this is the smallest group that stands on its own. The date tests are mixed up with timestamp conversions, so they are better moved together with or after the timestamp tests, which are too big for one PR and will be split. This group also does not overlap with open PR #11572 (string to duration casts), which adds its tests in the timestamp area of the module, so the two should not conflict. # What changes are included in this PR? Move 10 interval and duration cast tests, plus their helper functions and macros, into a new `arrow-cast/tests/cast/interval.rs`: - string to interval (year-month, day-time, month-day-nano, error cases) - duration to/from interval, and between interval units - duration to `Int64`, and between durations and numerics `test_cast_between_durations_and_numerics` uses the crate-private `time_unit_multiple`, so the test file has a small local copy of it. No production code changes. # Are these changes tested? Yes. The moved tests pass in the `cast` integration suite, and the `arrow-cast` tests pass locally. # Are there any significant user-facing changes? No. # AI usage disclosure Claude Code assisted with the test migration and ran validation. I reviewed and verified the changes. -- 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]
