[coverage] Conformance findings: SESSION-017

tests/ 配下の coverage PR diff にある TestTelemetryEnablementPropertyIsHonoredAndOptional から始め、Go ドライバーの DSN パースと、接続時のテレメトリプロパティの処理を追跡してください。EnableTelemetry についてドライバー自身が使用している綴りを確認してください。完了の条件は、不正な値によって接続を開けること、デフォルト値が維持されること、指定された警告診断が生成されること、そして有効な値と値が存在しない場合が引き続き機能することです。

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-017 [thrift]: A malformed enableTelemetry DSN value is fatal: ParseDSN rejects the entire DSN (strconv.ParseBool error) so the connection never opens, instead of leaving the telemetry overlay default untouched and diagnosing the typo
  • failing test: TestTelemetryEnablementPropertyIsHonoredAndOptional (see the coverage PR diff under tests/)

Reproduce & Expected

SESSION-017 — Validates the connect-time contract of the driver's telemetry-enablement connection property (reference ODBC key EnableTelemetry; map to the driver's own spelling for the SAME concept, e.g.

Reproduce:

SELECT 1 AS value
SELECT 1 AS value
SELECT 1 AS value
SELECT 1 AS value

Expected (per the shared spec):

• completes without an exception
• result has exactly 1 row(s)
• col 0 is named value
• completes without an exception
• result has exactly 1 row(s)
• completes without an exception
• result has exactly 1 row(s)
• completes without an exception
• result has exactly 1 row(s)
• full assertion contract:

result:
- label: opt_in
  no_exception: true
- label: opt_in
  row_count: 1
- label: opt_in
  column:
    index: 0
    name: value
- label: opt_in
  warning_diagnostic:
    mentions: EnableTelemetry
    present: false
- label: opt_out
  no_exception: true
- label: opt_out
  row_count: 1
- label: opt_out
  warning_diagnostic:
    mentions: EnableTelemetry
    present: false
- label: malformed
  no_exception: true
- label: malformed
  row_count: 1
- label: malformed
  warning_diagnostic:
    mentions: EnableTelemetry
    present: true
    sql_state: 01S02
- label: absent
  no_exception: true
- label: absent
  row_count: 1

Context

• The behavior was first fixed in a DIFFERENT driver — reference PR: https://github.com/databricks/databricks-odbc/pull/333 — 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/1641