-
Notifications
You must be signed in to change notification settings - Fork 24
Closed
Description
Yes, but I think we should always pick sqlitedb by default, and have an environment variable to forcibly change the DB type, such that if the db type is not compiled and the variable is set it will error out as well.
This way we can be sure of what we are testing in CI, currently it is almost a guess which DB is tested and we can't test "the other" DB when multiple are compiled in which is not great because applications can chose which one to use and we are not testing if the behavior of "the other" dbs is not broken by config options enabled by whichever one is chosen at test time.
Originally posted by @simo5 in #310 (comment)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels