solve rveiew

Author: camilamacedo86

test/e2e/features/recover.feature

@@ -143,9 +143,13 @@ Feature: Recover cluster extension from errors that might occur during its lifet
     And resource "deployment/test-operator" is available
     And resource "configmap/test-configmap" is available
     # Catalog deletion triggers automatic reconciliation via ClusterCatalog watch
-    # Wait for reconciliation to complete by checking the status condition
     When ClusterCatalog "test" is deleted
-    Then ClusterExtension reports Installed as True
+    # Wait for reconciliation to complete after catalog deletion by checking for Progressing=Retrying.
+    # This is necessary because Installed=True both before and after catalog deletion, so we need
+    # a condition that changes during reconciliation to prove that the fallback logic executed.
+    # The controller sets Reason=Retrying when falling back to the installed bundle (see handleResolutionError).
+    Then ClusterExtension reports Progressing as True with Reason Retrying
+    And ClusterExtension reports Installed as True
     And resource "deployment/test-operator" is available
     And resource "configmap/test-configmap" is available
 
@@ -222,8 +226,11 @@ Feature: Recover cluster extension from errors that might occur during its lifet
                 "olm.operatorframework.io/metadata.name": test-catalog
       """
     # Wait for reconciliation of the updated spec (config change should succeed without catalog)
-    And ClusterExtension is available
-    Then ClusterExtension reports Installed as True
+    # Check that the controller processed the spec update by waiting for Progressing=Retrying
+    # (which is set during fallback reconciliation when catalog is unavailable)
+    And ClusterExtension reports Progressing as True with Reason Retrying
+    Then ClusterExtension is available
+    And ClusterExtension reports Installed as True
 
   Scenario: Version upgrade does not proceed when catalog does not exist
     Given ServiceAccount "olm-sa" with needed permissions is available in ${TEST_NAMESPACE}