Use EXPECT_FALSE

alinaliBQ · alinaliBQ · commit 2332bdae2b2f · 2025-10-23T11:58:44.000-07:00
diff --git a/cpp/src/arrow/flight/sql/odbc/tests/connection_attr_test.cc b/cpp/src/arrow/flight/sql/odbc/tests/connection_attr_test.cc
@@ -177,7 +177,7 @@ TYPED_TEST(ConnectionAttributeTest, TestSQLGetConnectAttrTraceFileDMOnly) {
   out_str_len /= arrow::flight::sql::odbc::GetSqlWCharSize();
   std::string out_connection_string =
       ODBC::SqlWcharToString(out_str, static_cast<SQLSMALLINT>(out_str_len));
-  EXPECT_TRUE(!out_connection_string.empty());
+  EXPECT_FALSE(out_connection_string.empty());
 }
 
 TYPED_TEST(ConnectionAttributeTest, TestSQLGetConnectAttrTranslateLibUnsupported) {

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ TYPED_TEST(ConnectionAttributeTest, TestSQLGetConnectAttrTraceFileDMOnly) {`
`177`	`177`	`out_str_len /= arrow::flight::sql::odbc::GetSqlWCharSize();`
`178`	`178`	`std::string out_connection_string =`
`179`	`179`	`ODBC::SqlWcharToString(out_str, static_cast<SQLSMALLINT>(out_str_len));`
`180`		`- EXPECT_TRUE(!out_connection_string.empty());`
	`180`	`+ EXPECT_FALSE(out_connection_string.empty());`
`181`	`181`	`}`
`182`	`182`
`183`	`183`	`TYPED_TEST(ConnectionAttributeTest, TestSQLGetConnectAttrTranslateLibUnsupported) {`