sqlite3: fix wrong schema table usage

[bcho]

fixes: #165

[coveralls]

Pull Request Test Coverage Report for Build 294

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 59.188%

---

Totals
Change from base Build 285:	0.0%
Covered Lines:	773
Relevant Lines:	1306

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 295

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 59.188%

---

Totals
Change from base Build 285:	0.0%
Covered Lines:	773
Relevant Lines:	1306

---

💛 - Coveralls

[dhui]

Thanks for the fix!

[bcho]

OK just copied and pasted from old test case 😂

…

On Sat, Feb 2, 2019 at 06:20 Dale Hui ***@***.***> wrote: ***@***.**** requested changes on this pull request. Thanks for the fix! ------------------------------ In database/sqlite3/sqlite3_test.go <#166 (comment)> : > @@ -59,3 +60,49 @@ func Test(t *testing.T) { t.Fatalf("%v", err) } } + +func TestMigrationTable(t *testing.T) { + dir, err := ioutil.TempDir("", "sqlite3-driver-test-migration-table") + if err != nil { + return + } + defer func() { + os.RemoveAll(dir) + }() + + fmt.Printf("DB path : %s\n", filepath.Join(dir, "sqlite3.db")) Use t.Log() instead of fmt.Printf() ------------------------------ In database/sqlite3/sqlite3_test.go <#166 (comment)> : > + }() + + config := &Config{ + MigrationsTable: "my_migration_table", + } + driver, err := WithInstance(db, config) + if err != nil { + t.Fatalf("%v", err) + } + m, err := migrate.NewWithDatabaseInstance( + "file://./migration", + "ql", driver) + if err != nil { + t.Fatalf("%v", err) + } + fmt.Println("UP") Same — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#166 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB4jTtcWuSXrvyK9m2iw2Fsu7Ra2WyWwks5vJL25gaJpZM4ad9jQ> .

[bcho]

@dhui test updated, PTAL