Add support for sqlite #105
Conversation
There was a problem hiding this comment.
First of all, thanks for this PR. While skimming older open PRs, I stumbled upon this one and its modernc.org/sqlite dependency.
Besides the one comment about the library, could you please describe the motivation for this change and its necessity? Where is it needed and why?
@jrauh01 It's for some kind of automatic tests, right? |
|
Right, like described above it's for mocking databases in e.g. unit tests. |
| mode = "mode=memory&" | ||
| } | ||
|
|
||
| addr = fmt.Sprintf("file:%s?%scache=shared", name, mode) |
There was a problem hiding this comment.
As I was just fiddling with some other SQLite application, I realized that by default foreign key constraints are not enforced unless _foreign_keys is set accordingly. After fixing this on my side, I remembered this PR and would suggest adding this here as well.
3 participants
Add sqlite support to use in memory databases for example in unit tests. (Like in #102)