feat(test): add --timeout flag to t/ runner and make test

Add a configurable per-package test timeout flag to the t/ runner.
Previously the timeout was hardcoded to 30m (or 180m with --race),
which caused the 21million/live test to time out on slower machines.

Usage:
  make test TIMEOUT=90m
  cd t && ./t --suite=all --timeout=60m

Defaults remain unchanged: 30m normal, 180m with --race. An explicit
--timeout overrides both.

Author: mlwelles

CONTRIBUTING.md

@@ -154,6 +154,7 @@ make test-upgrade       # Upgrade tests
 
 # Use variables for more control
 make test TAGS=integration2 PKG=systest/vector
+make test TIMEOUT=90m          # Override per-package timeout (default: 30m)
 ```
 
 Run `make help` to see all available targets and variables.

Makefile

@@ -98,7 +98,7 @@ else
 endif
 else
 	@echo "Running test suite: $(or $(SUITE),all)"
-	$(MAKE) -C t test args="--suite=$(or $(SUITE),all) $(if $(PKG),--pkg=\"$(PKG)\") $(if $(TEST),--test=\"$(TEST)\")"
+	$(MAKE) -C t test args="--suite=$(or $(SUITE),all) $(if $(PKG),--pkg=\"$(PKG)\") $(if $(TEST),--test=\"$(TEST)\") $(if $(TIMEOUT),--timeout=$(TIMEOUT))"
 endif
 
 .PHONY: test-all
@@ -205,6 +205,7 @@ help: ## Show available targets and variables
 	@echo "  TAGS      Go build tags - bypasses t/ runner (e.g., make test TAGS=integration2)"
 	@echo "  PKG       Limit to specific package (e.g., make test PKG=systest/export)"
 	@echo "  TEST      Run specific test function (e.g., make test TEST=TestGQLSchema)"
+	@echo "  TIMEOUT   Per-package test timeout (e.g., make test TIMEOUT=60m). Default: 30m"
 	@echo "  FUZZ      Enable fuzz testing (e.g., make test FUZZ=1)"
 	@echo "  FUZZTIME  Fuzz duration per package (e.g., make test FUZZ=1 FUZZTIME=60s)"
 	@echo ""
@@ -223,4 +224,5 @@ help: ## Show available targets and variables
 	@echo "  make test TAGS=upgrade PKG=acl TEST=TestACL           # specific upgrade test"
 	@echo "  make test FUZZ=1 PKG=dql FUZZTIME=30s                 # fuzz dql package for 30s"
 	@echo "  make test SUITE=systest PKG=systest/backup/filesystem # systest for backup pkg"
+	@echo "  make test TIMEOUT=90m                                  # all suites with 90m timeout"
 	@echo "  make test-benchmark PKG=posting                       # benchmark posting package"

TESTING.md

@@ -335,6 +335,7 @@ For more control, pass variables to `make test`:
 | `TAGS`     | Go build tags - bypasses t/ runner | `make test TAGS=integration2`   |
 | `PKG`      | Limit to specific package          | `make test PKG=systest/export`  |
 | `TEST`     | Run specific test function         | `make test TEST=TestGQLSchema`  |
+| `TIMEOUT`  | Per-package test timeout           | `make test TIMEOUT=90m`         |
 | `FUZZ`     | Enable fuzz testing                | `make test FUZZ=1`              |
 | `FUZZTIME` | Fuzz duration per package          | `make test FUZZ=1 FUZZTIME=60s` |
 
@@ -578,15 +579,16 @@ cd t && go build .
 
 ### Key Flags
 
-| Flag          | Description                                                        |
-| ------------- | ------------------------------------------------------------------ |
-| `--suite=X`   | Select test suite(s): all, ldbc, load, unit, systest, vector, core |
-| `--pkg=X`     | Run specific package                                               |
-| `--test=X`    | Run specific test function                                         |
-| `-j=N`        | Concurrency (default: 1)                                           |
-| `--keep`      | Keep cluster running after tests                                   |
-| `-r`          | Remove all test containers                                         |
-| `--skip-slow` | Skip slow packages                                                 |
+| Flag            | Description                                                        |
+| --------------- | ------------------------------------------------------------------ |
+| `--suite=X`     | Select test suite(s): all, ldbc, load, unit, systest, vector, core |
+| `--pkg=X`       | Run specific package                                               |
+| `--test=X`      | Run specific test function                                         |
+| `--timeout=X`   | Per-package timeout (e.g. 60m, 2h). Default: 30m (180m with --race)|
+| `-j=N`          | Concurrency (default: 1)                                           |
+| `--keep`        | Keep cluster running after tests                                   |
+| `-r`            | Remove all test containers                                         |
+| `--skip-slow`   | Skip slow packages                                                 |
 
 ---
 

t/t.go

@@ -92,7 +92,9 @@ var (
 	tmp               = pflag.String("tmp", "", "Temporary directory used to download data.")
 	downloadResources = pflag.BoolP("download", "d", true,
 		"Flag to specify whether to download resources or not")
-	race = pflag.Bool("race", false, "Set true to build with race")
+	race        = pflag.Bool("race", false, "Set true to build with race")
+	testTimeout = pflag.String("timeout", "",
+		"Timeout for each test package (e.g. 60m, 2h). Defaults to 30m (180m with --race).")
 	skip = pflag.String("skip", "",
 		"comma separated list of packages that needs to be skipped. "+
 			"Package Check uses string.Contains(). Please check the flag carefully")
@@ -423,12 +425,17 @@ func sanitizeFilename(pkg string) string {
 func runTestsFor(ctx context.Context, pkg, prefix string, xmlFile string) error {
 	args := []string{"gotestsum", "--junitfile", xmlFile, "--format", "standard-verbose", "--max-fails", "1", "--",
 		"-v", "-failfast", "-tags=integration"}
-	if *race {
+	switch {
+	case *testTimeout != "":
+		args = append(args, "-timeout", *testTimeout)
+	case *race:
 		args = append(args, "-timeout", "180m")
+	default:
+		args = append(args, "-timeout", "30m")
+	}
+	if *race {
 		// Todo: There are few race errors in tests itself. Enable this once that is fixed.
 		// args = append(args, "-race")
-	} else {
-		args = append(args, "-timeout", "30m")
 	}
 
 	if *count > 0 {