Skip to content

Commit

Permalink
Merge pull request #288 from jon-whit/master
Browse files Browse the repository at this point in the history
Trim statement leading and trailing whitespace.
  • Loading branch information
dhui authored Oct 15, 2019
2 parents 57b2ba7 + 241ad96 commit 7f20f01
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions database/spanner/spanner.go
Original file line number Diff line number Diff line change
Expand Up @@ -309,6 +309,7 @@ func migrationStatements(migration []byte) []string {
allStatements := strings.Split(migrationString, ";")
nonEmptyStatements := allStatements[:0]
for _, s := range allStatements {
s = strings.TrimSpace(s)
if s != "" {
nonEmptyStatements = append(nonEmptyStatements, s)
}
Expand Down
4 changes: 2 additions & 2 deletions database/spanner/spanner_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ func TestMultistatementSplit(t *testing.T) {
CREATE INDEX table_name_id_idx ON table_name (id);`,
expected: []string{`CREATE TABLE table_name (
id STRING(255) NOT NULL,
) PRIMARY KEY(id)`, "\n\nCREATE INDEX table_name_id_idx ON table_name (id)"},
) PRIMARY KEY(id)`, "CREATE INDEX table_name_id_idx ON table_name (id)"},
},
{
name: "multi statement, no trailing semicolon",
Expand All @@ -114,7 +114,7 @@ CREATE INDEX table_name_id_idx ON table_name (id);`,
CREATE INDEX table_name_id_idx ON table_name (id)`,
expected: []string{`CREATE TABLE table_name (
id STRING(255) NOT NULL,
) PRIMARY KEY(id)`, "\n\nCREATE INDEX table_name_id_idx ON table_name (id)"},
) PRIMARY KEY(id)`, "CREATE INDEX table_name_id_idx ON table_name (id)"},
},
}

Expand Down

0 comments on commit 7f20f01

Please sign in to comment.