apache / arrow-nanoarrow

Helpers for Arrow C Data & Arrow C Stream interfaces
https://arrow.apache.org/nanoarrow
Apache License 2.0
169 stars 35 forks source link

fix(r): Fix tests for platforms where arrow dataset and/or zip is not available #415

Closed paleolimbot closed 6 months ago

paleolimbot commented 6 months ago

The utils::zip() function is not all that reliable; however, we only use it to create test data anyway. This was causing release verification to fail on alpine.

While running the tests, I also noticed that one of the IPC skips that checks for Arrow dataset does so incorrectly.

codecov-commenter commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.76%. Comparing base (9935713) to head (bb8a5f4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #415 +/- ## ======================================= Coverage 88.76% 88.76% ======================================= Files 82 82 Lines 14633 14633 ======================================= Hits 12989 12989 Misses 1644 1644 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.