Update tests to support multiple orgs

Signed-off-by: Max Ehrlich <max.ehr@gmail.com>

pkg/apis/certmanager/validation/certificate_for_issuer.go

@@ -59,7 +59,7 @@ func ValidateCertificateForACMEIssuer(crt *v1alpha1.CertificateSpec, issuer *v1a
 		el = append(el, field.Invalid(specPath.Child("isCA"), crt.KeyAlgorithm, "ACME does not support CA certificates"))
 	}
 
-	if crt.Organization != "" {
+	if len(crt.Organization) != 0 {
 		el = append(el, field.Invalid(specPath.Child("organization"), crt.Organization, "ACME does not support setting the organization name"))
 	}
 

pkg/apis/certmanager/validation/certificate_for_issuer_test.go

@@ -136,7 +136,7 @@ func TestValidateCertificateForIssuer(t *testing.T) {
 		"acme certificate with organization set": {
 			crt: &v1alpha1.Certificate{
 				Spec: v1alpha1.CertificateSpec{
-					Organization: "shouldfailorg",
+					Organization: []string{"shouldfailorg"},
 					IssuerRef:    validIssuerRef,
 					ACME: &v1alpha1.ACMECertificateConfig{
 						Config: []v1alpha1.DomainSolverConfig{
@@ -155,7 +155,7 @@ func TestValidateCertificateForIssuer(t *testing.T) {
 				Namespace: defaultTestNamespace,
 			}),
 			errs: []*field.Error{
-				field.Invalid(fldPath.Child("organization"), "shouldfailorg", "ACME does not support setting the organization name"),
+				field.Invalid(fldPath.Child("organization"), []string{"shouldfailorg"}, "ACME does not support setting the organization name"),
 			},
 		},
 		"certificate with unspecified issuer type": {

pkg/apis/certmanager/validation/certificate_test.go

@@ -78,7 +78,7 @@ func TestValidateCertificate(t *testing.T) {
 				Spec: v1alpha1.CertificateSpec{
 					CommonName:   "testcn",
 					SecretName:   "abc",
-					Organization: "testorg",
+					Organization: []string{"testorg"},
 					IssuerRef:    validIssuerRef,
 				},
 			},