Skip to content

[C++][FlightRPC][ODBC] Add SQLGetInfo missing tests #50828

Description

@alinaliBQ

Describe the enhancement requested

There is a gap that 2 tests are missing from the original implementation:

  • TEST_F(ConnectionInfoRemoteTest, TestSQLGetInfoCreateTable) for SQL_CREATE_TABLE
  • TEST_F(ConnectionInfoRemoteTest, TestSQLGetInfoCreateSchema) for SQL_CREATE_SCHEMA
    If the expected value from the remote is same as the mock server, then we can merge the remote test with the mock test by using TYPED_TEST(ConnectionInfoHandleTest.

Theoretically these tests should also return 0 to signal table and schema DDL is not supported in Dremio instance. It also explains why we were only seeing inconsistencies in SQL_DROP_TABLE and SQL_DROP_SCHEMA in #49482.

Component(s)

C++, FlightRPC

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions