Skip to content

Conversation

@kyleconroy
Copy link
Collaborator

Replaced custom test cases with queries directly from the ClickHouse
repository's tests/queries/0_stateless folder. Each test directory is
now named to match the original ClickHouse test file (e.g., 00007_array).
All tests are marked as todo until parser support is implemented.

Replaced custom test cases with queries directly from the ClickHouse
repository's tests/queries/0_stateless folder. Each test directory is
now named to match the original ClickHouse test file (e.g., 00007_array).
All tests are marked as todo until parser support is implemented.
- Generated ast.json for all 6823 test queries using ClickHouse EXPLAIN AST
- Removed ast/explain.go and ast/explain_test.go (no longer needed)
- Updated parser_test.go to use ast.json for expected AST comparison
- Tests now skip queries that ClickHouse also failed to parse
- Updated parser_test.go to parse only first line of query.sql
  (matches how ast.json was generated)
- Fixed empty ast.json files by marking them as error: true
- Tests now complete in ~4 seconds instead of timing out
@kyleconroy kyleconroy merged commit f484682 into main Dec 13, 2025
1 check passed
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.

3 participants