felipecrv opened a new pull request, #43292:
URL: https://github.com/apache/arrow/pull/43292
### Rationale for this change
#41700 (as it is currently) passes all the C++ tests even though it contains
a few bugs (caught by manual repro steps and tests of of the Ruby bindings).
The C++ tests should be able to catch these kinds of bugs and exercise code
beyond the TakeAAA cases.
### What changes are included in this PR?
- Explicitly calling out which TakeXX variation is being checked in tests
and assert helpers
- Using `AssertChunkedEqual` instead of `AssertChunkedEquivalent` (via
`AssertDatumsEqual`)
-
### Are these changes tested?
Yes. The improved tests catch bugs now.
--
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]
