Skip AzureFS tests in cpp_test.sh not test itself

Author: kou

ci/scripts/cpp_test.sh

@@ -42,43 +42,50 @@ if [ -z "${ARROW_DEBUG_MEMORY_POOL}" ]; then
   export ARROW_DEBUG_MEMORY_POOL=trap
 fi
 
+exclude_tests=()
 ctest_options=()
+if ! type azurite >/dev/null 2>&1; then
+  exclude_tests+=("arrow-azurefs-test")
+fi
 case "$(uname)" in
   Linux)
     n_jobs=$(nproc)
     ;;
   Darwin)
     n_jobs=$(sysctl -n hw.ncpu)
     # TODO: https://github.com/apache/arrow/issues/40410
-    exclude_tests="arrow-s3fs-test"
-    ctest_options+=(--exclude-regex "${exclude_tests}")
+    exclude_tests+=("arrow-s3fs-test")
     ;;
   MINGW*)
     n_jobs=${NUMBER_OF_PROCESSORS:-1}
     # TODO: Enable these crashed tests.
     # https://issues.apache.org/jira/browse/ARROW-9072
-    exclude_tests="gandiva-binary-test"
-    exclude_tests="${exclude_tests}|gandiva-boolean-expr-test"
-    exclude_tests="${exclude_tests}|gandiva-date-time-test"
-    exclude_tests="${exclude_tests}|gandiva-decimal-single-test"
-    exclude_tests="${exclude_tests}|gandiva-decimal-test"
-    exclude_tests="${exclude_tests}|gandiva-filter-project-test"
-    exclude_tests="${exclude_tests}|gandiva-filter-test"
-    exclude_tests="${exclude_tests}|gandiva-hash-test"
-    exclude_tests="${exclude_tests}|gandiva-if-expr-test"
-    exclude_tests="${exclude_tests}|gandiva-in-expr-test"
-    exclude_tests="${exclude_tests}|gandiva-internals-test"
-    exclude_tests="${exclude_tests}|gandiva-literal-test"
-    exclude_tests="${exclude_tests}|gandiva-null-validity-test"
-    exclude_tests="${exclude_tests}|gandiva-precompiled-test"
-    exclude_tests="${exclude_tests}|gandiva-projector-test"
-    exclude_tests="${exclude_tests}|gandiva-utf8-test"
-    ctest_options+=(--exclude-regex "${exclude_tests}")
+    exclude_tests+=("gandiva-binary-test")
+    exclude_tests+=("gandiva-boolean-expr-test")
+    exclude_tests+=("gandiva-date-time-test")
+    exclude_tests+=("gandiva-decimal-single-test")
+    exclude_tests+=("gandiva-decimal-test")
+    exclude_tests+=("gandiva-filter-project-test")
+    exclude_tests+=("gandiva-filter-test")
+    exclude_tests+=("gandiva-hash-test")
+    exclude_tests+=("gandiva-if-expr-test")
+    exclude_tests+=("gandiva-in-expr-test")
+    exclude_tests+=("gandiva-internals-test")
+    exclude_tests+=("gandiva-literal-test")
+    exclude_tests+=("gandiva-null-validity-test")
+    exclude_tests+=("gandiva-precompiled-test")
+    exclude_tests+=("gandiva-projector-test")
+    exclude_tests+=("gandiva-utf8-test")
     ;;
   *)
     n_jobs=${NPROC:-1}
     ;;
 esac
+if [ "${#exclude_tests[@]}" -gt 0 ]; then
+  IFS="|"
+  ctest_options+=(--exclude-regex "${exclude_tests[*]}")
+  unset IFS
+fi
 
 if [ "${ARROW_EMSCRIPTEN:-OFF}" = "ON" ]; then
   n_jobs=1 # avoid spurious fails on emscripten due to loading too many big executables

cpp/src/arrow/filesystem/azurefs_test.cc

@@ -423,11 +423,8 @@ class TestGeneric : public ::testing::Test, public GenericFileSystemTest {
 class TestAzuriteGeneric : public TestGeneric {
  public:
   void SetUp() override {
-    auto env_result = AzuriteEnv::GetInstance();
-    if (!env_result.ok()) {
-      GTEST_SKIP() << "Failed to setup: " << env_result.status().ToString();
-    }
-    SetUpInternal(*env_result);
+    ASSERT_OK_AND_ASSIGN(auto env, AzuriteEnv::GetInstance());
+    SetUpInternal(env);
   }
 
  protected:
@@ -891,13 +888,9 @@ class TestAzureFileSystem : public ::testing::Test {
   }
 
   void SetUp() override {
-    auto env_result = GetAzureEnv();
-    if (!env_result.ok()) {
-      GTEST_SKIP() << "Failed to setup: " << env_result.status().ToString();
-    }
-    auto env = *env_result;
-    EXPECT_THAT(env, NotNull());
-    auto make_options = [this, &env]() -> Result<AzureOptions> {
+    auto make_options = [this]() -> Result<AzureOptions> {
+      ARROW_ASSIGN_OR_RAISE(auto env, GetAzureEnv());
+      EXPECT_THAT(env, NotNull());
       ARROW_ASSIGN_OR_RAISE(debug_log_start_, env->GetDebugLogSize());
       return MakeOptions(env);
     };