Revert "feat(conformance): validate implementation flags (#3715)" (#3920)

This reverts commit 564720a.

Author: LiorLieberman

conformance/conformance.go

@@ -72,14 +72,13 @@ func DefaultOptions(t *testing.T) suite.ConformanceOptions {
 	namespaceAnnotations := suite.ParseKeyValuePairs(*flags.NamespaceAnnotations)
 	conformanceProfiles := suite.ParseConformanceProfiles(*flags.ConformanceProfiles)
 
-	implementation, err := suite.ParseImplementation(
+	implementation := suite.ParseImplementation(
 		*flags.ImplementationOrganization,
 		*flags.ImplementationProject,
 		*flags.ImplementationURL,
 		*flags.ImplementationVersion,
 		*flags.ImplementationContact,
 	)
-	require.NoError(t, err, "error parsing implementation details")
 
 	return suite.ConformanceOptions{
 		AllowCRDsMismatch:          *flags.AllowCRDsMismatch,
@@ -93,7 +92,7 @@ func DefaultOptions(t *testing.T) suite.ConformanceOptions {
 		ExemptFeatures:             exemptFeatures,
 		ManifestFS:                 []fs.FS{&Manifests},
 		GatewayClassName:           *flags.GatewayClassName,
-		Implementation:             *implementation,
+		Implementation:             implementation,
 		Mode:                       *flags.Mode,
 		NamespaceAnnotations:       namespaceAnnotations,
 		NamespaceLabels:            namespaceLabels,

conformance/utils/suite/suite.go

@@ -21,7 +21,6 @@ import (
 	"errors"
 	"fmt"
 	"io/fs"
-	neturl "net/url"
 	"slices"
 	"sort"
 	"strings"
@@ -559,33 +558,14 @@ func (suite *ConformanceTestSuite) Report() (*confv1.ConformanceReport, error) {
 
 // ParseImplementation parses implementation-specific flag arguments and
 // creates a *confv1a1.Implementation.
-func ParseImplementation(org, project, url, version, contact string) (*confv1.Implementation, error) {
-	if org == "" {
-		return nil, errors.New("organization must be set")
-	}
-	if project == "" {
-		return nil, errors.New("project must be set")
-	}
-	if url == "" {
-		return nil, errors.New("url must be set")
-	}
-	if version == "" {
-		return nil, errors.New("version must be set")
-	}
-	if contact == "" {
-		return nil, errors.New("contact must be set")
-	}
-	if _, err := neturl.ParseRequestURI(url); err != nil {
-		return nil, errors.New("url is malformed")
-	}
-
-	return &confv1.Implementation{
+func ParseImplementation(org, project, url, version, contact string) confv1.Implementation {
+	return confv1.Implementation{
 		Organization: org,
 		Project:      project,
 		URL:          url,
 		Version:      version,
 		Contact:      strings.Split(contact, ","),
-	}, nil
+	}
 }
 
 // ParseConformanceProfiles parses flag arguments and converts the string to

conformance/utils/suite/suite_test.go

@@ -538,89 +538,3 @@ func namesToFeatureSet(names []string) FeaturesSet {
 	}
 	return featureSet
 }
-
-func TestParseImplementation(t *testing.T) {
-	testCases := []struct {
-		name        string
-		org         string
-		project     string
-		url         string
-		version     string
-		contact     string
-		expected    *confv1.Implementation
-		expectedErr error
-	}{
-		{
-			name:        "missing organization",
-			project:     "test-project",
-			url:         "https://example.com",
-			version:     "v1.0.0",
-			contact:     "[email protected]",
-			expectedErr: errors.New("organization must be set"),
-		},
-		{
-			name:        "missing project",
-			org:         "test-org",
-			url:         "https://example.com",
-			version:     "v1.0.0",
-			contact:     "[email protected]",
-			expectedErr: errors.New("project must be set"),
-		},
-		{
-			name:        "missing url",
-			org:         "test-org",
-			project:     "test-project",
-			version:     "v1.0.0",
-			contact:     "[email protected]",
-			expectedErr: errors.New("url must be set"),
-		},
-		{
-			name:        "missing version",
-			org:         "test-org",
-			project:     "test-project",
-			url:         "https://example.com",
-			contact:     "[email protected]",
-			expectedErr: errors.New("version must be set"),
-		},
-		{
-			name:        "missing contact",
-			org:         "test-org",
-			project:     "test-project",
-			url:         "https://example.com",
-			version:     "v1.0.0",
-			expectedErr: errors.New("contact must be set"),
-		},
-		{
-			name:        "malformed url",
-			org:         "test-org",
-			project:     "test-project",
-			url:         "invalid-url",
-			version:     "v1.0.0",
-			contact:     "[email protected]",
-			expectedErr: errors.New("url is malformed"),
-		},
-		{
-			name:    "valid input",
-			org:     "test-org",
-			project: "test-project",
-			url:     "https://example.com",
-			version: "v1.0.0",
-			contact: "[email protected],[email protected]",
-			expected: &confv1.Implementation{
-				Organization: "test-org",
-				Project:      "test-project",
-				URL:          "https://example.com",
-				Version:      "v1.0.0",
-				Contact:      []string{"[email protected]", "[email protected]"},
-			},
-		},
-	}
-
-	for _, tc := range testCases {
-		t.Run(tc.name, func(t *testing.T) {
-			result, err := ParseImplementation(tc.org, tc.project, tc.url, tc.version, tc.contact)
-			assert.Equal(t, tc.expected, result)
-			assert.Equal(t, tc.expectedErr, err)
-		})
-	}
-}

pkg/test/cel/grpcroute_test.go

@@ -341,8 +341,7 @@ func TestGRPCRouteRule(t *testing.T) {
 				}
 				return rules
 			}(),
-		},
-	}
+		}}
 
 	for _, tc := range tests {
 		t.Run(tc.name, func(t *testing.T) {