test: add ToMap round-trip test for expression and float rejection for timeout-minutes

Copilot · pelikhan · web-flow · commit 188e95638bce · 2026-04-01T12:30:59.000Z
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/f80cd78a-1d25-4f2b-8c69-166d9dc00adb Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
diff --git a/pkg/workflow/frontmatter_types_test.go b/pkg/workflow/frontmatter_types_test.go
@@ -158,6 +158,17 @@ func TestParseFrontmatterConfig(t *testing.T) {
 		}
 	})
 
+	t.Run("rejects timeout-minutes as float", func(t *testing.T) {
+		frontmatter := map[string]any{
+			"timeout-minutes": 5.5,
+		}
+
+		_, err := ParseFrontmatterConfig(frontmatter)
+		if err == nil {
+			t.Error("expected error for float timeout-minutes, got nil")
+		}
+	})
+
 	t.Run("rejects timeout-minutes as free-form string", func(t *testing.T) {
 		frontmatter := map[string]any{
 			"timeout-minutes": "not-an-expression",
@@ -494,6 +505,25 @@ func TestFrontmatterConfigBackwardCompatibility(t *testing.T) {
 		}
 	})
 
+	t.Run("round-trip preserves timeout-minutes expression", func(t *testing.T) {
+		frontmatter := map[string]any{
+			"name":            "test-workflow",
+			"engine":          "copilot",
+			"timeout-minutes": "${{ inputs.timeout }}",
+		}
+
+		config, err := ParseFrontmatterConfig(frontmatter)
+		if err != nil {
+			t.Fatalf("parse error: %v", err)
+		}
+
+		reconstructed := config.ToMap()
+
+		if reconstructed["timeout-minutes"] != "${{ inputs.timeout }}" {
+			t.Errorf("timeout-minutes expression mismatch: got %v (%T)", reconstructed["timeout-minutes"], reconstructed["timeout-minutes"])
+		}
+	})
+
 	t.Run("round-trip preserves labels", func(t *testing.T) {
 		frontmatter := map[string]any{
 			"name":   "test-workflow",
