Function TestValidateDomain

internal/agent/validate_test.go:63–97

(t *testing.T)

}

func TestValidateDomain(t *testing.T) {
	cases := []struct {
		name      string
		input     string
		wantErr   bool
		wantASCII string // expected normalized output on success; "" means don't check
	}{
		// Valid
		{"plain ASCII", "example.com", false, "example.com"},
		{"subdomain", "agents.example.com", false, "agents.example.com"},
		{"with hyphen", "my-domain.example.com", false, "my-domain.example.com"},
		{"IDN normalizes to Punycode", "пример.рф", false, "xn--e1afmkfd.xn--p1ai"},
		// Invalid — what today's prod was accepting before the fix
		{"empty", "", true, ""},
		{"whitespace + comma — the bug repro", "not a domain, just garbage", true, ""},
		{"single space", "exam ple.com", true, ""},
		{"no period — bare label", "localhost", true, ""},
		{"trailing newline", "example.com\n", true, ""},
		{"control char", "exa\x00mple.com", true, ""},
		// Length bounds — IDNA enforces 63-char label and 253-char total
		{"label exceeds 63 chars", strings.Repeat("a", 64) + ".com", true, ""},
	}
	for _, tc := range cases {
		t.Run(tc.name, func(t *testing.T) {
			got, err := validateDomain(tc.input)
			if (err != nil) != tc.wantErr {
				t.Errorf("validateDomain(%q) err=%v, wantErr=%v", tc.input, err, tc.wantErr)
				return
			}
			if !tc.wantErr && tc.wantASCII != "" && got != tc.wantASCII {
				t.Errorf("validateDomain(%q) got=%q, want=%q (Punycode normalization)", tc.input, got, tc.wantASCII)
			}
		})
	}
}