revert the unwanted test changes

Signed-off-by: NAVEENA S <nas@redhat.com>

Author: nas438

test/openshift/e2e/ginkgo/fixture/clusterserviceversion/fixture.go

@@ -6,20 +6,13 @@ import (
 	//lint:ignore ST1001 "This is a common practice in Gomega tests for readability."
 	. "github.com/onsi/gomega" //nolint:all
 	olmv1alpha1 "github.com/operator-framework/api/pkg/operators/v1alpha1"
-	"github.com/redhat-developer/gitops-operator/test/openshift/e2e/ginkgo/fixture"
 	"github.com/redhat-developer/gitops-operator/test/openshift/e2e/ginkgo/fixture/utils"
 	"k8s.io/client-go/util/retry"
 	"sigs.k8s.io/controller-runtime/pkg/client"
 )
 
 // Update will update a ClusterServiceVersion CR. Update will keep trying to update object until it succeeds, or times out.
 func Update(obj *olmv1alpha1.ClusterServiceVersion, modify func(*olmv1alpha1.ClusterServiceVersion)) {
-	if fixture.EnvNonOLM() || fixture.EnvLocalRun() || fixture.EnvCI() {
-		// Skipping CSV update as operator is not managed via OLM in these environments.
-		// In CI environment, the operator is managed via Subscription rather than direct CSV access.
-		return
-	}
-
 	k8sClient, _ := utils.GetE2ETestKubeClient()
 
 	err := retry.RetryOnConflict(retry.DefaultRetry, func() error {

test/openshift/e2e/ginkgo/sequential/1-085_validate_dynamic_plugin_installation_test.go

@@ -46,11 +46,6 @@ var _ = Describe("GitOps Operator Sequential E2E Tests", func() {
 				return
 			}
 
-			if fixture.EnvCI() {
-				Skip("Skipping CSV-based test in CI environment where operator is managed via Subscription")
-				return
-			}
-
 			// Find CSV
 			var csv *olmv1alpha1.ClusterServiceVersion
 			var csvList olmv1alpha1.ClusterServiceVersionList

test/openshift/e2e/ginkgo/sequential/1-121-valiate_resource_constraints_gitopsservice_test.go

@@ -86,11 +86,6 @@ var _ = Describe("GitOps Operator Sequential E2E Tests", func() {
 		})
 
 		It("validates that GitOpsService can take in custom resource constraints", func() {
-			if fixture.EnvCI() {
-				Skip("Skipping CSV-based test in CI environment where operator is managed via Subscription")
-				return
-			}
-
 			csv := getCSV(ctx, k8sClient)
 			Expect(csv).ToNot(BeNil())
 			defer func() { Expect(fixture.RemoveDynamicPluginFromCSV(ctx, k8sClient)).To(Succeed()) }()
@@ -172,11 +167,6 @@ var _ = Describe("GitOps Operator Sequential E2E Tests", func() {
 		})
 
 		It("validates that GitOpsService can update resource constraints", func() {
-			if fixture.EnvCI() {
-				Skip("Skipping CSV-based test in CI environment where operator is managed via Subscription")
-				return
-			}
-
 			csv := getCSV(ctx, k8sClient)
 			Expect(csv).ToNot(BeNil())
 			defer func() { Expect(fixture.RemoveDynamicPluginFromCSV(ctx, k8sClient)).To(Succeed()) }()
@@ -248,11 +238,6 @@ var _ = Describe("GitOps Operator Sequential E2E Tests", func() {
 		})
 
 		It("validates gitops plugin and backend can have different resource constraints", func() {
-			if fixture.EnvCI() {
-				Skip("Skipping CSV-based test in CI environment where operator is managed via Subscription")
-				return
-			}
-
 			csv := getCSV(ctx, k8sClient)
 			Expect(csv).ToNot(BeNil())
 			defer func() { Expect(fixture.RemoveDynamicPluginFromCSV(ctx, k8sClient)).To(Succeed()) }()