test: honour -short in Docker-backed TestMain and cabana list-service fixture

TestMain scanned os.Args for a literal -test.short, but go test passes
-test.short=true, so beachcomber, conga, lagoon and lighthouse started a
Postgres container even under -short. Parse the flags and use
testing.Short() instead; skip cabana's newListService fixture under -short
like adminGorm already does.

modules/beachcomber/postgres_test.go

@@ -3,6 +3,7 @@ package beachcomber
 import (
 	"context"
 	"database/sql"
+	"flag"
 	"fmt"
 	"net/url"
 	"os"
@@ -42,13 +43,14 @@ func TestMain(m *testing.M) {
 	os.Exit(code)
 }
 
+// testShort reports whether -short was passed. TestMain runs before the
+// testing flags are parsed, and go test passes the flag as -test.short=true,
+// so parse the flags here instead of scanning os.Args.
 func testShort() bool {
-	for _, a := range os.Args {
+	if !flag.Parsed() {
-		if a == "-test.short" {
+		flag.Parse()
-			return true
-		}
 	}
-	return false
+	return testing.Short()
 }
 
 func startBeachcomberPostgres(ctx context.Context) error {

modules/cabana/query_test.go

@@ -269,6 +269,9 @@ var (
 
 func newListService(t *testing.T) (*service, *gorm.DB) {
 	t.Helper()
+	if testing.Short() {
+		t.Skip("requires testcontainers postgres")
+	}
 	listPGOnce.Do(func() {
 		ctx, cancel := context.WithTimeout(context.Background(), 2*time.Minute)
 		defer cancel()

modules/conga/postgres_test.go

@@ -3,6 +3,7 @@ package conga
 import (
 	"context"
 	"database/sql"
+	"flag"
 	"fmt"
 	"net/url"
 	"os"
@@ -45,13 +46,14 @@ func TestMain(m *testing.M) {
 	os.Exit(code)
 }
 
+// testShort reports whether -short was passed. TestMain runs before the
+// testing flags are parsed, and go test passes the flag as -test.short=true,
+// so parse the flags here instead of scanning os.Args.
 func testShort() bool {
-	for _, a := range os.Args {
+	if !flag.Parsed() {
-		if a == "-test.short" {
+		flag.Parse()
-			return true
-		}
 	}
-	return false
+	return testing.Short()
 }
 
 func startCongaPostgres(ctx context.Context) error {

modules/lagoon/postgres_test.go

@@ -3,6 +3,7 @@ package lagoon
 import (
 	"context"
 	"database/sql"
+	"flag"
 	"fmt"
 	"net/url"
 	"os"
@@ -39,13 +40,14 @@ func TestMain(m *testing.M) {
 	os.Exit(code)
 }
 
+// testShort reports whether -short was passed. TestMain runs before the
+// testing flags are parsed, and go test passes the flag as -test.short=true,
+// so parse the flags here instead of scanning os.Args.
 func testShort() bool {
-	for _, a := range os.Args {
+	if !flag.Parsed() {
-		if a == "-test.short" {
+		flag.Parse()
-			return true
-		}
 	}
-	return false
+	return testing.Short()
 }
 
 func startLagoonPostgres(ctx context.Context) error {

modules/lighthouse/postgres_test.go

@@ -3,6 +3,7 @@ package lighthouse
 import (
 	"context"
 	"database/sql"
+	"flag"
 	"fmt"
 	"net/url"
 	"os"
@@ -42,13 +43,14 @@ func TestMain(m *testing.M) {
 	os.Exit(code)
 }
 
+// testShort reports whether -short was passed. TestMain runs before the
+// testing flags are parsed, and go test passes the flag as -test.short=true,
+// so parse the flags here instead of scanning os.Args.
 func testShort() bool {
-	for _, a := range os.Args {
+	if !flag.Parsed() {
-		if a == "-test.short" {
+		flag.Parse()
-			return true
-		}
 	}
-	return false
+	return testing.Short()
 }
 
 func startLighthousePostgres(ctx context.Context) error {