Fix sqlite driver using double quotes for string literals

[rasteiner]

Description

Fix the table discovery query in the sqlite class using double quotes for string literals.

Summary of changes

• Using single quotes in sqlite table discovery query fixes SQLite 3.41.0 double-quoted string literals are disabled by default #6769

Reasoning

Double quotes to enclose string literals are sometimes supported by sqlite configurations (mainly for backwards compatibility).
Single quotes are always supported.

Additional context

https://www.sqlite.org/quirks.html#double_quoted_string_literals_are_accepted

Changelog

• Using single quotes in sqlite table discovery query fixes SQLite 3.41.0 double-quoted string literals are disabled by default #6769

Breaking changes

None

Ready?

• In-code documentation (wherever needed)
• Unit tests for fixed bug/feature
• Tests and CI checks all pass

For review team

• Add lab and/or sandbox examples (wherever helpful)
• Add changes & docs to release notes draft in Notion

[rasteiner]

PS: the tests actually pass, I think they timed out here :/

[afbora]

@rasteiner Thank you for the PR. I've re-run the tests and all passed 🎉

I'm just curious about the Sqlite::quoteIdentifier() method. This method also uses double-quotes. Does this method also need to be updated?

[rasteiner]

The quoteIdentifier function is fine, as it correctly puts the quotes around identifiers, such as table or column names and not around string literals.

[bastianallgeier]

Thanks for the PR!