GH-49552: [C++][FlightRPC][ODBC] Enable ODBC test build on Linux

[justing-bq]

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: [C++][FlightRPC][ODBC] Enable ODBC test build on Linux #49552

[github-actions]

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

• Other pull requests
• Contribution Guidelines - Contributing Overview

[github-actions]

⚠️ GitHub issue #49552 has been automatically assigned in GitHub to PR creator.

[justing-bq]

@lidavidm @kou
Please take a look when you have a chance.

[Copilot]

Pull request overview

Enables building the Flight SQL ODBC C++ test suite on Linux by removing the platform gate in CMake and updating tests/helpers to better handle platform differences in SQLWCHAR and wide-string handling.

Changes:

• Always add the ODBC tests subdirectory when ARROW_BUILD_TESTS is enabled (including Linux).
• Refactor many tests to avoid relying on L""/wcslen assumptions and to use helper macros/utilities for SQLWCHAR.
• Adjust CI to exclude running the ODBC test binary on Linux while still building it.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
cpp/src/arrow/flight/sql/odbc/tests/type_info_test.cc	Switches expected column name handling and replaces NULL sentinel usage with numeric values.
cpp/src/arrow/flight/sql/odbc/tests/tables_test.cc	Updates SQLWCHAR parameter initialization and column-name comparisons for portability.
cpp/src/arrow/flight/sql/odbc/tests/statement_test.cc	Introduces ASSIGN_SQLWCHAR_ARR_AND_LEN usage and adjusts string comparisons/conversions.
cpp/src/arrow/flight/sql/odbc/tests/statement_attr_test.cc	Uses the new SQLWCHAR assignment helper macro for query strings.
cpp/src/arrow/flight/sql/odbc/tests/odbc_test_suite.h	Adds Linux-specific SQLWCHAR assignment macros and declares new conversion helpers.
cpp/src/arrow/flight/sql/odbc/tests/odbc_test_suite.cc	Implements new SQLWCHAR length/conversion helpers and tweaks DSN registration call.
cpp/src/arrow/flight/sql/odbc/tests/errors_test.cc	Avoids constructing std::wstring directly from SQLWCHAR* by converting to UTF-8 first.
cpp/src/arrow/flight/sql/odbc/tests/connection_test.cc	Makes output buffers consistently zero-initialized.
cpp/src/arrow/flight/sql/odbc/tests/connection_info_test.cc	Renames helper and converts many wide-string assertions to use ConvertToWString.
cpp/src/arrow/flight/sql/odbc/tests/columns_test.cc	Updates SQLWCHAR parameter handling and metadata assertions for Linux builds.
cpp/src/arrow/flight/sql/odbc/CMakeLists.txt	Removes Windows/macOS-only gating so tests are built on Linux too.
ci/scripts/cpp_test.sh	Excludes arrow-flight-sql-odbc-test from Linux test execution.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[alinaliBQ]

I have 1 comment

[alinaliBQ]

LGTM

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[justing-bq]

@lidavidm @kou
Please take a look when you have time.

[justing-bq]

@lidavidm @kou
Please take a look when you have time.

[kou]

+1

The ODBC test on Linux is failing. It's out-of-scope of this PR, right?

[kou]

We remove this later, right?

[justing-bq]

Correct. This gets removed in 9e8272d.

[kou]

Which PR does include the commit?
GH-49786?

[justing-bq]

#49786 which will be undrafted as soon as this PR gets merged.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

[justing-bq]

@kou @lidavidm
Comments addressed. Please review.

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 62cdda9.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 3 possible false positives for unstable benchmarks that are known to sometimes produce them.

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 62cdda9.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 3 possible false positives for unstable benchmarks that are known to sometimes produce them.