Function TestSpec_PublicAPI_UpdateFieldInFrontmatter

pkg/cli/spec_test.go:355–404

TestSpec_PublicAPI_UpdateFieldInFrontmatter validates the documented frontmatter field update. Spec: "Sets a field in frontmatter YAML"

(t *testing.T)

// TestSpec_PublicAPI_UpdateFieldInFrontmatter validates the documented frontmatter field update.
// Spec: "Sets a field in frontmatter YAML"
func TestSpec_PublicAPI_UpdateFieldInFrontmatter(t *testing.T) {
	tests := []struct {
		name          string
		content       string
		fieldName     string
		fieldValue    string
		wantErr       bool
		checkContains string
	}{
		{
			name:          "updates existing field",
			content:       "---\ndescription: old description\n---\n\n# Content",
			fieldName:     "description",
			fieldValue:    "new description",
			wantErr:       false,
			checkContains: "new description",
		},
		{
			name:          "adds new field when absent",
			content:       "---\nengine: copilot\n---\n\n# Content",
			fieldName:     "description",
			fieldValue:    "my workflow",
			wantErr:       false,
			checkContains: "my workflow",
		},
		{
			// SPEC_AMBIGUITY: The README spec says "Sets a field in frontmatter YAML" without
			// specifying the error-path for content without frontmatter. The implementation
			// creates a new frontmatter block in this case rather than returning an error.
			name:          "creates frontmatter when content has none",
			content:       "# Just markdown with no frontmatter",
			fieldName:     "description",
			fieldValue:    "value",
			wantErr:       false,
			checkContains: "value",
		},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			result, err := cli.UpdateFieldInFrontmatter(tt.content, tt.fieldName, tt.fieldValue)
			if tt.wantErr {
				assert.Error(t, err, "UpdateFieldInFrontmatter should return error for %q", tt.name)
				return
			}
			require.NoError(t, err, "UpdateFieldInFrontmatter should not error for %q", tt.name)
			assert.Contains(t, result, tt.checkContains, "result should contain updated value for %q", tt.name)
		})
	}
}

// TestSpec_PublicAPI_SetFieldInOnTrigger validates the documented on: trigger field update.
// Spec: "Sets a field inside the on: trigger block"