Skip to content

test(integration): coverage-driven SQL tests for under-exercised worker paths#3

Merged
rustyconover merged 1 commit into
mainfrom
cleanup-deadcode-typed-probe
Jun 17, 2026
Merged

test(integration): coverage-driven SQL tests for under-exercised worker paths#3
rustyconover merged 1 commit into
mainfrom
cleanup-deadcode-typed-probe

Conversation

@rustyconover

Copy link
Copy Markdown
Contributor

Adds language-agnostic sqllogictest coverage for VGI SDK paths the suite did not exercise, with matching example-worker fixtures (Python + Rust):

  • settings/settings_types — a float (DOUBLE) setting read via get_f64
  • secret/secret_fields — named/positional secret field accessors
  • scalar/numeric_promotion — float/unsigned/decimal/mixed-sign promotion
  • table/filtered_columns_pushdown — the pushed-filter column-introspection accessors (filtered_columns / has_filter_for_column / typed get_column_values)

Updates scalar/table function_registration inventories for the new fixtures (scalar 39→41, table 93→94).

🤖 Generated with Claude Code

…er paths

Add language-agnostic sqllogictest coverage for VGI SDK paths the suite did not
exercise, with matching example-worker fixtures (Python + Rust):

- settings_types: a float (DOUBLE) setting read via get_f64
- secret_fields: named/positional secret field accessors over a resolved secret
- numeric_promotion: float/unsigned/decimal/mixed-sign numeric promotion
- filtered_columns_pushdown: the pushed-filter column-introspection accessors
  (filtered_columns / has_filter_for_column / typed get_column_values)

Update scalar/table function_registration inventories for the new fixtures
(scalar 39->41, table 93->94).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@rustyconover rustyconover merged commit 2819414 into main Jun 17, 2026
10 checks passed
@rustyconover rustyconover deleted the cleanup-deadcode-typed-probe branch June 17, 2026 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant