search

polydbms / sheetreader-duckdb

MIT License
38 stars 3 forks source link

Feature: Add a sqllogic test #51

Open carlopi opened 1 week ago

carlopi commented 1 week ago

I see that currently there are no tests available, and in the future we might want for community extensions to rely more on them.

Could you consider adding an sqllogic test? to the test/sql directory?

freddie-freeloader commented 1 week ago

Hey @carlopi,

We had some Python scripts for checking the correctness of the parse results -- but I could definitely add tests that work with the DuckDB scripts.

~Are there test files using sqllogictest that you know of? These could be helpful, since I could use them as a starting point.~ ☺️ Edit: I found the page in the DuckDB docs. 👍 For reference: https://duckdb.org/docs/dev/sqllogictest/intro.html

carlopi commented 1 week ago

Just to state, this is very much a suggestion, but extension is great as is obviously.