fix: Module deletion (#3922)

* ucomment test

* fix module delete

* improve tests

Author: dbadura

src/components/KymaModules/providers/KymaModuleProvider.js

@@ -69,7 +69,8 @@ export function KymaModuleContextProvider({
     singularName: 'Kyma',
     pluralKind: 'Kymas',
     resource: kymaResourceState,
-    initialUnchangedResource: initialUnchangedResource,
+    initialResource: initialUnchangedResource,
+    updateInitialResource: setInitialUnchangedResource,
     createUrl: resourceUrl,
     afterCreatedFn: () =>
       notification.notifySuccess({

tests/integration/support/commands.js

@@ -168,6 +168,7 @@ Cypress.Commands.add(
       clearSearch = true,
       checkIfResourceIsRemoved = true,
       selectSearchResult = false,
+      searchInPlainTableText = false,
     } = options;
 
     cy.wait(500)
@@ -183,7 +184,15 @@ Cypress.Commands.add(
         .click();
     }
 
-    cy.checkItemOnGenericListLink(resourceName);
+    if (searchInPlainTableText) {
+      //  TODO: Modules in tests are unmannaged and text is not in ui5-text component
+      cy.get('ui5-table-row')
+        .find('ui5-table-cell')
+        .contains(resourceName)
+        .should('be.visible');
+    } else {
+      cy.checkItemOnGenericListLink(resourceName);
+    }
 
     cy.get('ui5-button[data-testid="delete"]').click();
 
@@ -198,19 +207,19 @@ Cypress.Commands.add(
         cy.contains('ui5-toast', /deleted/).should('be.visible');
       }
 
-      if (clearSearch) {
-        cy.get('ui5-input[id="search-input"]:visible')
-          .find('input')
-          .wait(1000)
-          .clear();
-      }
-
       if (checkIfResourceIsRemoved) {
         cy.get('ui5-table')
           .contains(resourceName)
           .should('not.exist');
       }
     }
+
+    if (clearSearch) {
+      cy.get('ui5-input[id="search-input"]:visible')
+        .find('input')
+        .wait(1000)
+        .clear();
+    }
   },
 );
 

tests/integration/tests/kyma-cluster/test-kyma-modules.spec.js

@@ -239,29 +239,31 @@ context('Test Kyma Modules views', () => {
       .should('not.be.exist');
   });
 
-  // Uncomment after adding local KLM
-  // it('Test deleting Modules from List and Details', { retries: 3 }, () => {
-  //   cy.deleteFromGenericList('Module', 'eventing');
+  it('Test deleting Modules from List and Details', { retries: 3 }, () => {
+    cy.deleteFromGenericList('Module', 'eventing', {
+      searchInPlainTableText: true,
+    });
 
-  //   cy.get('ui5-input[id="search-input"]:visible')
-  //     .find('input')
-  //     .wait(1000)
-  //     .type('api-gateway');
-
-  //   cy.get('ui5-table-row')
-  //     .contains('api-gateway')
-  //     .click();
-
-  //   cy.deleteInDetails('Module', 'api-gateway', true);
-
-  //   cy.wait(20000);
-
-  //   cy.get('ui5-input[id="search-input"]:visible')
-  //     .find('input')
-  //     .clear();
-
-  //   cy.get('ui5-table')
-  //     .contains('ui5-illustrated-message', 'No modules')
-  //     .should('be.visible');
-  // });
+    // Uncomment after adding local KLM
+    // cy.get('ui5-input[id="search-input"]:visible')
+    //   .find('input')
+    //   .wait(1000)
+    //   .type('api-gateway');
+    //
+    // cy.get('ui5-table-row')
+    //   .contains('api-gateway')
+    //   .click();
+    //
+    // cy.deleteInDetails('Module', 'api-gateway', true);
+    //
+    // cy.wait(20000);
+    //
+    // cy.get('ui5-input[id="search-input"]:visible')
+    //   .find('input')
+    //   .clear();
+    //
+    // cy.get('ui5-table')
+    //   .contains('ui5-illustrated-message', 'No modules')
+    //   .should('be.visible');
+  });
 });