faster tests; add short and skip

pressly · Oct 14, 2023 · c3a1ca2 · c3a1ca2
1 parent dc8b261
commit c3a1ca2
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 0 deletions.
diff --git a/Makefile b/Makefile
@@ -33,6 +33,9 @@ tools:
 test-packages:
  go test $(GO_TEST_FLAGS) $$(go list ./... | grep -v -e /tests -e /bin -e /cmd -e /examples)
 
+test-packages-short:
+ go test -test.short $(GO_TEST_FLAGS) $$(go list ./... | grep -v -e /tests -e /bin -e /cmd -e /examples)
+
 test-e2e: test-e2e-postgres test-e2e-mysql test-e2e-clickhouse test-e2e-vertica
 
 test-e2e-postgres:

diff --git a/create_test.go b/create_test.go
@@ -11,6 +11,9 @@ import (
 
 func TestSequential(t *testing.T) {
  t.Parallel()
+ if testing.Short() {
+ t.Skip("skip long running test")
+ }
 
  dir := t.TempDir()
  defer os.Remove("./bin/create-goose") // clean up

diff --git a/fix_test.go b/fix_test.go
@@ -11,6 +11,9 @@ import (
 
 func TestFix(t *testing.T) {
  t.Parallel()
+ if testing.Short() {
+ t.Skip("skip long running test")
+ }
 
  dir := t.TempDir()
  defer os.Remove("./bin/fix-goose") // clean up

diff --git a/internal/migrationstats/migrationstats_test.go b/internal/migrationstats/migrationstats_test.go
@@ -8,6 +8,7 @@ import (
 )
 
 func TestParsingGoMigrations(t *testing.T) {
+ t.Parallel()
  tests := []struct {
  name string
  input string
@@ -38,6 +39,7 @@ func TestParsingGoMigrations(t *testing.T) {
 }
 
 func TestParsingGoMigrationsError(t *testing.T) {
+ t.Parallel()
  _, err := parseGoFile(strings.NewReader(emptyInit))
  check.HasError(t, err)
  check.Contains(t, err.Error(), "no registered goose functions")

diff --git a/lock/postgres_test.go b/lock/postgres_test.go
@@ -14,6 +14,7 @@ import (
 )
 
 func TestPostgresSessionLocker(t *testing.T) {
+ t.Parallel()
  if testing.Short() {
  t.Skip("skip long running test")
  }