[coverage] Conformance findings: SESSION-019

[peco-engineer-bot]

Summary

Surfaced by the multi-language coverage fan-out while conformance-testing these SPEC-IDs against databricks/databricks-sql-go. Each finding is committed as an expected-failure (xfail) test in the coverage PR — the test asserts the CORRECT (post-fix) behavior and stays red until THIS driver (databricks/databricks-sql-go) is fixed, then flips green as a tripwire.

Findings

• SESSION-019 [thrift]: An unrecognised DSN connection property is forwarded to the server as a session configuration, so the first statement fails with CONFIG_NOT_AVAILABLE.WITHOUT_SUGGESTION (SQLSTATE 42K0I) instead of the unknown key being ignored, breaking the shared-configuration tolerance contract
  • failing test: TestUnknownConnectionPropertyIsAcceptedAndIgnored (see the coverage PR diff under tests/)

Reproduce & Expected

SESSION-019 — Validates that a connection property the driver does not recognise at all is tolerated: the connection opens and queries work.

Reproduce:

SELECT 1 AS test_value

Expected (per the shared spec):

• completes without an exception
• result has exactly 1 row(s)
• [thrift] exactly 1 OpenSession call(s)
• [sea] exactly 1 CreateSession call(s)

Context

• The behavior was first fixed in a DIFFERENT driver — reference PR: https://github.com/databricks/databricks-odbc/pull/180 — which seeded the shared language-neutral spec. This issue tracks the same conformance gap in databricks/databricks-sql-go; the reference PR is for cross-referencing the intended behavior, NOT a change to this repo.
• Coverage PR carrying the reproducing xfail test(s): https://github.com/databricks/databricks-driver-test/pull/1182