You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "codewarnab (via GitHub)" <gi...@apache.org> on 2026/09/16 19:31:58 UTC

[PR] feat(arrow-cast): move list cast tests to integration suite [arrow-rs]

codewarnab opened a new pull request, #11111:
URL: https://github.com/apache/arrow-rs/pull/11111

   # Which issue does this PR close?
   
   Part of [#11033](https://github.com/apache/arrow-rs/issues/11033).
   
   # Rationale for this change
   
   The public-behavior cast tests in `arrow-cast/src/cast/mod.rs` are being moved into the crate-level integration test suite so individual cast families are easier to locate and maintain. This is the list, fixed-size-list, and list-view test slice of that work.
   
   # What changes are included in this PR?
   
   - Adds `arrow-cast/tests/cast/list.rs` to the existing `cast` integration-test target.
   - Moves 43 list, fixed-size-list, and list-view cast tests and their private helpers out of the inline `cast/mod.rs` test module.
   - Removes imports that are no longer needed by the inline test module.
   
   # Are these changes tested?
   
   - `cargo test -p arrow-cast`
   - `cargo clippy -p arrow-cast --all-targets --all-features -- -D warnings`
   - `cargo fmt --all -- --check`
   - `git diff --check`
   
   # Are there any user-facing changes?
   
   No. This is a test-organization-only change.
   
   # AI usage disclosure
   
   OpenAI Codex assisted with the mechanical test extraction. I reviewed the resulting changes and verified the full `arrow-cast` test suite and targeted lint checks locally.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(arrow-cast): move list cast tests to integration suite [arrow-rs]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on PR #11111:
URL: https://github.com/apache/arrow-rs/pull/11111#issuecomment-6005377140

   I took the opportunity to merge up from main to fix the conflict to get this one in


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(arrow-cast): move list cast tests to integration suite [arrow-rs]

Posted by "Jefffrey (via GitHub)" <gi...@apache.org>.
Jefffrey commented on PR #11111:
URL: https://github.com/apache/arrow-rs/pull/11111#issuecomment-6009806266

   thanks @codewarnab & @alamb 


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(arrow-cast): move list cast tests to integration suite [arrow-rs]

Posted by "Jefffrey (via GitHub)" <gi...@apache.org>.
Jefffrey merged PR #11111:
URL: https://github.com/apache/arrow-rs/pull/11111


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org