Skip to content

Commit 62cdda9

Browse files
authored
GH-49552: [C++][FlightRPC][ODBC] Enable ODBC test build on Linux (#49668)
### Rationale for this change The test suite needs to be updated so it will build on Linux in addition to Windows & Mac. Resolves #49552 ### What changes are included in this PR? Miscellaneous changes to get the tests building on Linux. ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * GitHub Issue: #49552 Authored-by: justing-bq <62349012+justing-bq@users.noreply.github.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
1 parent 0e37230 commit 62cdda9

12 files changed

Lines changed: 957 additions & 1007 deletions

ci/scripts/cpp_test.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ if ! type minio >/dev/null 2>&1; then
5555
fi
5656
case "$(uname)" in
5757
Linux)
58+
exclude_tests+=("arrow-flight-sql-odbc-test")
5859
n_jobs=$(nproc)
5960
;;
6061
Darwin)

cpp/src/arrow/flight/sql/odbc/CMakeLists.txt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,7 @@ endif()
2929

3030
add_subdirectory(odbc_impl)
3131
if(ARROW_BUILD_TESTS)
32-
if(WIN32 OR APPLE)
33-
# GH-49552 TODO: Enable Linux test build
34-
add_subdirectory(tests)
35-
endif()
32+
add_subdirectory(tests)
3633
endif()
3734

3835
arrow_install_all_headers("arrow/flight/sql/odbc")

cpp/src/arrow/flight/sql/odbc/tests/columns_test.cc

Lines changed: 149 additions & 247 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)