singhpratech commented on code in PR #51236:
URL: https://github.com/apache/arrow/pull/51236#discussion_r4030948195
##########
r/tests/testthat/test-bridge.R:
##########
@@ -81,3 +81,29 @@ test_that("Pointer wrapper errors for unknown object", {
"Can't parse 'this is not an integer'"
)
})
+
+test_that("the C Data Interface allocators are exported", {
+ exported <- getNamespaceExports("arrow")
+ for (f in c(
+ "allocate_arrow_schema",
+ "delete_arrow_schema",
+ "allocate_arrow_array",
+ "delete_arrow_array",
+ "allocate_arrow_array_stream",
+ "delete_arrow_array_stream"
+ )) {
+ expect_true(f %in% exported, label = paste(f, "is exported"))
+ }
+})
Review Comment:
It pins the six names as part of the exported interface, so a later roxygen
run that loses an `@export` fails here, naming the function, rather than in a
downstream package's check. The round-trip test below reaches them through
`arrow::` and would fail too, so if one test is enough I'll drop this one and
keep the round trip.
--
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]
