fix(sqlite3): handle execution of multiple SQL commands (#312)

[atopala]

Fixes #312:

• wraps Postgrator execSqlScript() around sqlite3 exec

Tested locally before/after (sqlite3 local db):

• before: creates only person table
• after: creates both person and animal tables

[perrin4869]

I'm wondering if this should instead be fixed upstream? Any idea why postgrator isn't already doing this?

[atopala]

I'm wondering if this should instead be fixed upstream? Any idea why postgrator isn't already doing this?

Not sure that postgrator needs the fix here. There's no sqlite3 db function that can do it all.
The sqlite3 client in postgrator-cli is calling sqlite3 db.all() - this function can only execute 1st statement in a sql script.

postgrator/postgrator-cli still need this to fetch the migration records.
sqlite3 db.exec() is the one that can fully execute a sqlite3 script.

[perrin4869]

ok I see what's going on! sorry for the confusion. If you can add a test I'll merge! thanks

[perrin4869]

hm... I think these ignore rules can be added to a .gitignore_global in a user's config to reflect the environment they are using

[coveralls]

Pull Request Test Coverage Report for Build 18937704252

Details

• 7 of 9 (77.78%) changed or added relevant lines in 2 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.3%) to 92.527%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
lib/clients/sqlite3.js	6	8	75.0%

Totals
Change from base Build 18514918470:	-0.3%
Covered Lines:	544
Relevant Lines:	578

---

💛 - Coveralls