crate / sqlalchemy-cratedb

SQLAlchemy dialect for CrateDB.
https://cratedb.com/docs/sqlalchemy-cratedb/
Apache License 2.0
3 stars 2 forks source link

Integrate SqlAlchemy dialect compliance suite into testing framework #86

Open autophagy opened 4 years ago

autophagy commented 4 years ago

We recently encountered an issue with the CrateDB sqla dialect (#348). SqlAlchemy provides a dialect compliance testing suite (https://github.com/sqlalchemy/sqlalchemy/blob/master/README.dialects.rst) to ensure dialects adhere to certain compliance standards - which would have identified this issue before it was reported by an external user.

However, in the past, it seems we had problems with the compliance suite due to a lack of transactions. We should:

  1. Investigate if this is still the case with the testing suite
  2. Integrate the compliance suite into our testing