Function TestCleanOneOfMessage

pkg/parser/schema_errors_test.go:13–91

TestCleanOneOfMessage tests that oneOf error messages are simplified to plain English

(t *testing.T)

// TestCleanOneOfMessage tests that oneOf error messages are simplified to plain English
func TestCleanOneOfMessage(t *testing.T) {
	tests := []struct {
		name    string
		input   string
		wantNot []string // substrings that must NOT appear in output
		wantAny []string // at least one of these must appear in output
	}{
		{
			name: "engine typo removes got-string-want-object branch",
			input: "at '/engine': 'oneOf' failed, none matched\n" +
				"- at '/engine': value must be one of 'claude', 'codex', 'copilot', 'gemini'\n" +
				"- at '/engine': got string, want object",
			wantNot: []string{"oneOf", "got string, want object"},
			wantAny: []string{"value must be one of 'claude', 'codex', 'copilot', 'gemini'"},
		},
		{
			name: "permissions typo removes got-object-want-string branch",
			input: "at '/permissions': 'oneOf' failed, none matched\n" +
				"- at '/permissions': got object, want string\n" +
				"- at '/permissions/deployments': value must be one of 'read', 'write', 'none'",
			wantNot: []string{"oneOf", "got object, want string"},
			wantAny: []string{"value must be one of 'read', 'write', 'none'"},
		},
		{
			name:    "non-oneOf message is returned unchanged",
			input:   "value must be one of 'a', 'b', 'c'",
			wantNot: []string{"oneOf"},
			wantAny: []string{"value must be one of 'a', 'b', 'c'"},
		},
		{
			name: "nested path context preserved for sub-field errors",
			input: "at '/permissions': 'oneOf' failed, none matched\n" +
				"- at '/permissions': got object, want string\n" +
				"- at '/permissions/deployments': value must be one of 'read', 'write', 'none'",
			wantNot: []string{},
			wantAny: []string{"deployments"},
		},
		{
			name: "all type conflicts synthesizes plain-English message",
			input: "at '/x': 'oneOf' failed, none matched\n" +
				"- at '/x': got string, want object\n" +
				"- at '/x': got string, want array",
			// When all sub-errors are type conflicts, synthesize a plain-English message
			wantNot: []string{"oneOf", "got string, want object"},
			wantAny: []string{"expected object or array, got string"},
		},
		{
			name: "engine type conflict produces actionable message",
			input: "at '/engine': 'oneOf' failed, none matched\n" +
				"- at '/engine': got number, want string\n" +
				"- at '/engine': got number, want object",
			wantNot: []string{"oneOf", "got number, want string"},
			wantAny: []string{"expected string or object, got number", "Valid engine names", "copilot"},
		},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {