search

partiql / partiql-tests

Test data for checking conformance to the PartiQL specification.
Apache License 2.0
7 stars 1 forks source link

Adds `ANY`, `SOME`, and `EVERY` SQL aggregation function tests #80

Closed alancai98 closed 1 year ago

alancai98 commented 1 year ago

Addresses https://github.com/partiql/partiql-tests/issues/79.

Adds the ANY, SOME, and EVERY SQL aggregation function tests from partiql-lang-kotlin.

Also does a slight refactor (captured in the first commit https://github.com/partiql/partiql-tests/commit/2b6f95b3f60a55bd9a963aad14d78a75d2415624) of the syntax COLL_ aggregation tests. The second commit adds the new aggregation function tests.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

vgapeyev commented 1 year ago

Following up, maybe this is a good opportunity to also include tests for COLL_X(NULL), COLL_X(MISSING), COLL_X([MISSING]), and perhaps, COLL_X(<<MISSING, MISSING>>) for a good measure? Not sure whether this needs to be done for all X.

alancai98 commented 1 year ago

Thanks again for the review and discussion. Helps clear up further specify the spec issue: https://github.com/partiql/partiql-lang/issues/17.

Corrected that previous search-and-replace error and added more tests involving