Disable tests with DISABLED_

* Enable `TestSQLGetInfoDriverHdesc`

We missed this test before, enabling the test now

* Disable tests with `DISABLED_`

Author: alinaliBQ

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

@@ -212,12 +212,21 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoDriverHdbc) {
 
 // These information types are implemented by the Driver Manager alone.
 TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoDriverHdesc) {
-  // TODO This is failing due to no descriptor being created
-  // enable after SQL_HANDLE_DESC is supported
-  GTEST_SKIP();
   this->Connect();
 
-  Validate(this->conn, SQL_DRIVER_HDESC, static_cast<SQLULEN>(0));
+  SQLHDESC descriptor;
+
+  // Allocate a descriptor using alloc handle
+  ASSERT_EQ(SQL_SUCCESS, SQLAllocHandle(SQL_HANDLE_DESC, this->conn, &descriptor));
+
+  // Value returned from driver manager is the desc address
+  SQLHDESC local_desc = descriptor;
+  SQLRETURN ret = SQLGetInfo(this->conn, SQL_HANDLE_DESC, &local_desc, 0, 0);
+  EXPECT_EQ(SQL_SUCCESS, ret);
+  EXPECT_GT(local_desc, static_cast<SQLHSTMT>(0));
+
+  // Free descriptor handle
+  ASSERT_EQ(SQL_SUCCESS, SQLFreeHandle(SQL_HANDLE_DESC, descriptor));
 
   this->Disconnect();
 }
@@ -389,9 +398,8 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoOdbcInterfaceConformance) {
 
 // case SQL_ODBC_STANDARD_CLI_CONFORMANCE: - mentioned in SQLGetInfo spec with no
 // description and there is no constant for this.
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoOdbcStandardCliConformance) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLGetInfoOdbcStandardCliConformance) {
   // Type commented out in odbc_connection.cc
-  GTEST_SKIP();
   this->Connect();
 
   // Type does not exist in sql.h
@@ -703,9 +711,8 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoAlterDomain) {
   this->Disconnect();
 }
 
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoAlterSchema) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLGetInfoAlterSchema) {
   // Type commented out in odbc_connection.cc
-  GTEST_SKIP();
   this->Connect();
 
   // Type does not exist in sql.h
@@ -722,9 +729,8 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoAlterTable) {
   this->Disconnect();
 }
 
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetInfoAnsiSqlDatetimeLiterals) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLGetInfoAnsiSqlDatetimeLiterals) {
   // Type commented out in odbc_connection.cc
-  GTEST_SKIP();
   this->Connect();
 
   // Type does not exist in sql.h

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

@@ -231,11 +231,9 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetEnvAttrOutputNTS) {
   this->Disconnect();
 }
 
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetEnvAttrGetLength) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLGetEnvAttrGetLength) {
   // Test is disabled because call to SQLGetEnvAttr is handled by the driver manager on
   // Windows. This test case can be potentially used on macOS/Linux
-  GTEST_SKIP();
-
   this->Connect();
 
   SQLINTEGER length;
@@ -250,10 +248,9 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetEnvAttrGetLength) {
   this->Disconnect();
 }
 
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetEnvAttrNullValuePointer) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLGetEnvAttrNullValuePointer) {
   // Test is disabled because call to SQLGetEnvAttr is handled by the driver manager on
   // Windows. This test case can be potentially used on macOS/Linux
-  GTEST_SKIP();
   this->Connect();
 
   SQLRETURN return_get =

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

@@ -133,11 +133,9 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetDiagFieldWForConnectFailure) {
   EXPECT_EQ(SQL_SUCCESS, ret);
 }
 
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLGetDiagFieldWForConnectFailureNTS) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLGetDiagFieldWForConnectFailureNTS) {
   // Test is disabled because driver manager on Windows does not pass through SQL_NTS
   // This test case can be potentially used on macOS/Linux
-  GTEST_SKIP();
-  //  ODBC Environment
   SQLHENV env;
   SQLHDBC conn;
 

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

@@ -1164,10 +1164,9 @@ TEST_F(FlightSQLODBCMockTestBase, TestSQLExecDirectVarbinaryTruncation) {
   this->Disconnect();
 }
 
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLExecDirectFloatTruncation) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLExecDirectFloatTruncation) {
   // Test is disabled until float truncation is supported.
   // GH-46985: return warning message instead of error on float truncation case
-  GTEST_SKIP();
   this->Connect();
 
   std::wstring wsql;
@@ -2064,9 +2063,9 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLBindColRowArraySize) {
   this->Disconnect();
 }
 
-TYPED_TEST(FlightSQLODBCTestBase, TestSQLBindColIndicatorOnly) {
+TYPED_TEST(FlightSQLODBCTestBase, DISABLED_TestSQLBindColIndicatorOnly) {
   // GH-47021: implement driver to return indicator value when data pointer is null
-  GTEST_SKIP();
+
   // Verify driver supports null data pointer with valid indicator pointer
   this->Connect();
 
@@ -2207,11 +2206,10 @@ TYPED_TEST(FlightSQLODBCTestBase, TestSQLExtendedFetchRowFetching) {
   this->Disconnect();
 }
 
-TEST_F(FlightSQLODBCRemoteTestBase, TestSQLExtendedFetchQueryNullIndicator) {
+TEST_F(FlightSQLODBCRemoteTestBase, DISABLED_TestSQLExtendedFetchQueryNullIndicator) {
   // GH-47110: SQLExtendedFetch should return SQL_SUCCESS_WITH_INFO for 22002
   // Limitation on mock test server prevents null from working properly, so use remote
   // server instead. Mock server has type `DENSE_UNION` for null column data.
-  GTEST_SKIP();
   this->Connect();
 
   SQLINTEGER val;