mirrors/woodpecker
1
0
Fork 0
mirror of https://github.com/woodpecker-ci/woodpecker.git synced 2024-11-23 02:11:01 +00:00
Code Issues Projects Releases Wiki Activity

migrate/sqlite: create table only if it doesn't exist

Browse source 
fixes #224
This commit is contained in:
Rafael Garcia 2014-03-26 14:26:51 -07:00
parent 9f43d5c647
commit a284332a91
2 changed files with 24 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
pkg/database/migrate/sqlite.go
View file

@ -19,7 +19,7 @@ func SQLite(tx *sql.Tx) *MigrationDriver {
} }
func (s *sqliteDriver) CreateTable(tableName string, args []string) (sql.Result, error) { func (s *sqliteDriver) CreateTable(tableName string, args []string) (sql.Result, error) {
return s.Tx.Exec(fmt.Sprintf("CREATE TABLE %s (%s)", tableName, strings.Join(args, ", "))) return s.Tx.Exec(fmt.Sprintf("CREATE TABLE IF NOT EXISTS %s (%s)", tableName, strings.Join(args, ", ")))
} }
func (s *sqliteDriver) RenameTable(tableName, newName string) (sql.Result, error) { func (s *sqliteDriver) RenameTable(tableName, newName string) (sql.Result, error) {

23
pkg/database/migrate/sqlite_test.go
View file

@ -290,6 +290,29 @@ func TestMigrateCreateTable(t *testing.T) {
} }
} }
func TestMigrateExistingCreateTable(t *testing.T) {
defer tearDown()
if err := setUp(); err != nil {
t.Fatalf("Error preparing database: %q", err)
}
if _, err := db.Exec(testSchema); err != nil {
t.Fatalf("Can not create database: %q", err)
}
mgr := New(db)
rev := &revision1{}
if err := mgr.Add(rev).Migrate(); err != nil {
t.Fatalf("Can not migrate: %q", err)
}
var current int64
db.QueryRow("SELECT max(revision) FROM migration").Scan(&current)
if current != rev.Revision() {
t.Fatalf("Did not successfully migrate")
}
}
func TestMigrateRenameTable(t *testing.T) { func TestMigrateRenameTable(t *testing.T) {
defer tearDown() defer tearDown()
if err := setUp(); err != nil { if err := setUp(); err != nil {