Skip to content

Commit 6fed4f3

Browse files
alinaliBQrscales
andauthored
GH-46575: [C++][FlightRPC] Add Diagnostic tests (#47764)
### Rationale for this change Putting the tests for diagnostics in a separate PR from #47763, because non-diagnostic ODBC APIs are required to get diagnostics. ### What changes are included in this PR? - Add tests for ODBC diagnostics ### Are these changes tested? PR depends on #47971 for tests to work. Tested locally. ### Are there any user-facing changes? No * GitHub Issue: #46575 Co-authored-by: rscales <[email protected]>
1 parent 082377a commit 6fed4f3

File tree

5 files changed

+584
-23
lines changed

5 files changed

+584
-23
lines changed

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ add_arrow_test(flight_sql_odbc_test
3636
odbc_test_suite.h
3737
connection_attr_test.cc
3838
connection_test.cc
39+
errors_test.cc
3940
statement_attr_test.cc
4041
statement_test.cc
4142
# Enable Protobuf cleanup after test execution

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,9 @@ TYPED_TEST_SUITE(ConnectionTest, TestTypes);
3737
template <typename T>
3838
class ConnectionHandleTest : public T {};
3939

40-
class ConnectionRemoteTest : public FlightSQLOdbcHandleRemoteTestBase {};
41-
using TestTypesHandle =
42-
::testing::Types<FlightSQLOdbcHandleMockTestBase, FlightSQLOdbcHandleRemoteTestBase>;
40+
class ConnectionRemoteTest : public FlightSQLOdbcEnvConnHandleRemoteTestBase {};
41+
using TestTypesHandle = ::testing::Types<FlightSQLOdbcEnvConnHandleMockTestBase,
42+
FlightSQLOdbcEnvConnHandleRemoteTestBase>;
4343
TYPED_TEST_SUITE(ConnectionHandleTest, TestTypesHandle);
4444

4545
TEST(ODBCHandles, TestSQLAllocAndFreeEnv) {

0 commit comments

Comments
 (0)