Skip to content

Commit

Permalink
Make sure macOS wheel verification has test data
Browse files Browse the repository at this point in the history
  • Loading branch information
nealrichardson authored and kszucs committed Feb 4, 2020
1 parent a605417 commit e165d54
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cpp/src/arrow/flight/test_util.cc
Expand Up @@ -454,7 +454,7 @@ Status TestClientBasicAuthHandler::GetToken(std::string* token) {
Status GetTestResourceRoot(std::string* out) {
const char* c_root = std::getenv("ARROW_TEST_DATA");
if (!c_root) {
return Status::IOError("Test resources not found, set ARROW_TEST_DATA");
return Status::IOError("Test resources not found, set ARROW_TEST_DATA to <repo root>/testing/data");
}
*out = std::string(c_root);
return Status::OK();
Expand Down
4 changes: 4 additions & 0 deletions dev/release/verify-release-candidate.sh
Expand Up @@ -637,6 +637,10 @@ test_linux_wheels() {

test_macos_wheels() {
local py_arches="2.7m 3.5m 3.6m 3.7m 3.8"
if [ ! -d "arrow-testing" ]; then
git clone https://github.com/apache/arrow-testing.git
fi
export ARROW_TEST_DATA=$PWD/arrow-testing/data

for py_arch in ${py_arches}; do
local env=_verify_wheel-${py_arch}
Expand Down
2 changes: 1 addition & 1 deletion python/pyarrow/tests/test_flight.py
Expand Up @@ -60,7 +60,7 @@ def resource_root():
"""Get the path to the test resources directory."""
if not os.environ.get("ARROW_TEST_DATA"):
raise RuntimeError("Test resources not found; set "
"ARROW_TEST_DATA to <repo root>/testing")
"ARROW_TEST_DATA to <repo root>/testing/data")
return pathlib.Path(os.environ["ARROW_TEST_DATA"]) / "flight"


Expand Down

0 comments on commit e165d54

Please sign in to comment.