Squashed commit of the following:

commit f58a5e9 Author: Igor Braginsky <ibragins@redhat.com> Date: Thu Jun 23 17:34:15 2022 +0300 Added local storage preservation functionality (kubev2v#965) commit 816a53c Author: Igor Braginsky <ibragins@redhat.com> Date: Mon Jun 20 16:13:06 2022 +0300 Postmigation cleanup now works with API (kubev2v#964) commit 1c3bf03 Author: Mike Turley <mike.turley@alum.cs.umass.edu> Date: Thu Jun 16 21:07:14 2022 -0400 Bug 2067708: Fix invalid-mapping check to allow mapping items by name or id (kubev2v#961) commit 99612fa Author: Mike Turley <mike.turley@alum.cs.umass.edu> Date: Thu Jun 16 13:51:51 2022 -0400 Upgrade @hot-loader/react-dom and use --legacy-peer-deps flag to unblock CI (kubev2v#962) commit c751e17 Author: Igor Braginsky <ibragins@redhat.com> Date: Mon Jun 6 12:53:40 2022 +0300 Made selector for cluster checkbox more generic (kubev2v#960) * As middle of selector there is flaky I amm searching for `Select` and name of the cluster * Update plan.ts commit ada12c6 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu Jun 2 13:58:59 2022 -0400 chore(deps): bump async from 2.6.3 to 2.6.4 (kubev2v#959) Bumps [async](https://github.com/caolan/async) from 2.6.3 to 2.6.4. - [Release notes](https://github.com/caolan/async/releases) - [Changelog](https://github.com/caolan/async/blob/v2.6.4/CHANGELOG.md) - [Commits](caolan/async@v2.6.3...v2.6.4) --- updated-dependencies: - dependency-name: async dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit d05cd3f Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu Jun 2 13:50:01 2022 -0400 chore(deps): bump ejs from 3.1.6 to 3.1.7 (kubev2v#958) Bumps [ejs](https://github.com/mde/ejs) from 3.1.6 to 3.1.7. - [Release notes](https://github.com/mde/ejs/releases) - [Changelog](https://github.com/mde/ejs/blob/main/CHANGELOG.md) - [Commits](mde/ejs@v3.1.6...v3.1.7) --- updated-dependencies: - dependency-name: ejs dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit 3786a80 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu Jun 2 13:17:20 2022 -0400 chore(deps): bump eventsource from 1.1.0 to 1.1.1 (kubev2v#957) Bumps [eventsource](https://github.com/EventSource/eventsource) from 1.1.0 to 1.1.1. - [Release notes](https://github.com/EventSource/eventsource/releases) - [Changelog](https://github.com/EventSource/eventsource/blob/master/HISTORY.md) - [Commits](EventSource/eventsource@v1.1.0...v1.1.1) --- updated-dependencies: - dependency-name: eventsource dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Mike Turley <mike.turley@alum.cs.umass.edu> commit e42c078 Author: Igor Braginsky <ibragins@redhat.com> Date: Thu Jun 2 01:04:52 2022 +0300 Did changes in test config files (kubev2v#956) commit 70e3063 Author: Igor Braginsky <ibragins@redhat.com> Date: Wed Jun 1 16:56:15 2022 +0300 Increased default timeouts for testing on slow VMs (kubev2v#955) commit 38b1f66 Author: Igor Braginsky <ibragins@redhat.com> Date: Thu May 26 17:30:56 2022 +0300 Added elements required for warm migration to config files (kubev2v#954) commit 0a7157f Author: Igor Braginsky <ibragins@redhat.com> Date: Tue May 17 23:56:57 2022 +0300 Added creation and removal of second network (kubev2v#953) Added creation and removal of project Updated tier0 tests to use those methods and make them more stable commit 545418c Author: Mike Turley <mike.turley@alum.cs.umass.edu> Date: Tue Apr 19 11:27:39 2022 -0400 Bug 2067708: Support source VMs, networks and datastores referenced either by id or name in plan and mapping objects (kubev2v#948) * Bug 2067708: Support source VMs, networks and datastores referenced either by id or name in plan and mapping objects * Adjust mock data to test changes commit 4141385 Author: Igor Braginsky <ibragins@redhat.com> Date: Wed Apr 13 12:47:01 2022 +0300 Separated RHV amd VMWare tier0 configs (kubev2v#950) Improved end-2-end RHV test Now after plan is edited - it is performed Few light changes commit 0f7150e Author: Igor Braginsky <ibragins@redhat.com> Date: Wed Apr 6 12:42:08 2022 +0300 Fixes for TIER0 tests (kubev2v#949) commit fa8f9e7 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu Mar 31 11:36:32 2022 -0400 chore(deps): bump minimist from 1.2.5 to 1.2.6 (kubev2v#947) Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6. - [Release notes](https://github.com/substack/minimist/releases) - [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6) --- updated-dependencies: - dependency-name: minimist dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit 6e7c0ef Author: Igor Braginsky <ibragins@redhat.com> Date: Wed Mar 23 22:26:52 2022 +0200 Added validation of hooks log for log inspector (kubev2v#946) commit 317d1ca Author: kpunwatk <kpunwatk@redhat.com> Date: Wed Mar 23 15:53:05 2022 +0530 modified: pkg/qe-tests/cypress/integration/models/providerRhv.ts (kubev2v#932) modified: pkg/qe-tests/cypress/integration/models/providerVmware.ts modified: pkg/qe-tests/cypress/integration/tests/Rhv/config_separate_mapping_rhv.ts new file: pkg/qe-tests/cypress/integration/tests/Rhv/editProvider.ts modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts new file: pkg/qe-tests/cypress/integration/tests/vmware/editProvider.ts modified: pkg/qe-tests/cypress/integration/types/types.ts modified: pkg/qe-tests/cypress/integration/models/providerRhv.ts modified: pkg/qe-tests/cypress/integration/models/providerVmware.ts modified: pkg/qe-tests/cypress/integration/tests/Rhv/config_separate_mapping_rhv.ts new file: pkg/qe-tests/cypress/integration/tests/Rhv/editProvider.ts modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts new file: pkg/qe-tests/cypress/integration/tests/vmware/editProvider.ts modified: pkg/qe-tests/cypress/integration/types/types.ts modified: pkg/qe-tests/cypress/integration/models/providerRhv.ts modified: pkg/qe-tests/cypress/integration/models/providerVmware.ts modified: pkg/qe-tests/cypress/integration/tests/Rhv/config_separate_mapping_rhv.ts new file: pkg/qe-tests/cypress/integration/tests/Rhv/editProvider.ts modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts new file: pkg/qe-tests/cypress/integration/tests/vmware/editProvider.ts modified: pkg/qe-tests/cypress/integration/types/types.ts Co-authored-by: Igor Braginsky <ibragins@redhat.com> commit 7ed0a74 Author: kpunwatk <kpunwatk@redhat.com> Date: Wed Mar 16 16:24:56 2022 +0530 modified: pkg/qe-tests/cypress/integration/tests/Rhv/config_separate_mapping_rhv.ts (kubev2v#941) modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts modified: pkg/qe-tests/cypress/integration/models/plan.ts modified: pkg/qe-tests/cypress/integration/tests/Rhv/config_separate_mapping_rhv.ts modified: pkg/qe-tests/cypress/integration/tests/tier0/tier0_tests_vmware.ts modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts Co-authored-by: Igor Braginsky <ibragins@redhat.com> commit b2cdce3 Author: kpunwatk <kpunwatk@redhat.com> Date: Wed Mar 16 15:51:54 2022 +0530 [QE] Infra for the edit mappings (kubev2v#943) modified: pkg/qe-tests/cypress/integration/models/mapping.ts modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts modified: pkg/qe-tests/cypress/integration/types/constants.ts modified: pkg/qe-tests/cypress/integration/views/mapping.view.ts modified: pkg/qe-tests/cypress/integration/models/mapping.ts modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts modified: pkg/qe-tests/cypress/integration/types/constants.ts modified: pkg/qe-tests/cypress/integration/views/mapping.view.ts modified: pkg/qe-tests/cypress/integration/models/mapping.ts modified: pkg/qe-tests/cypress/integration/tests/Rhv/config_separate_mapping_rhv.ts modified: pkg/qe-tests/cypress/integration/tests/vmware/config_separate_mapping.ts modified: pkg/qe-tests/cypress/integration/types/constants.ts modified: pkg/qe-tests/cypress/integration/views/mapping.view.ts commit 8087c64 Author: kpunwatk <kpunwatk@redhat.com> Date: Wed Mar 9 15:41:24 2022 +0530 modified: pkg/qe-tests/cypress/integration/models/providerRhv.ts (kubev2v#939) Co-authored-by: Igor Braginsky <ibragins@redhat.com> commit cebb68d Author: Igor Braginsky <ibragins@redhat.com> Date: Tue Mar 8 19:42:52 2022 +0200 Added some python code. (kubev2v#940) The task is to parse tarball that was downloaded by getLog and make sure that the content is valid commit 5495f56 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu Mar 3 14:19:40 2022 -0500 chore(deps): bump url-parse from 1.5.3 to 1.5.10 (kubev2v#923) Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.3 to 1.5.10. - [Release notes](https://github.com/unshiftio/url-parse/releases) - [Commits](unshiftio/url-parse@1.5.3...1.5.10) --- updated-dependencies: - dependency-name: url-parse dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit cf73624 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu Mar 3 12:16:35 2022 -0500 chore(deps): bump follow-redirects from 1.14.7 to 1.14.9 (kubev2v#937) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.9. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.14.7...v1.14.9) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit f2b3a31 Author: Mike Turley <mike.turley@alum.cs.umass.edu> Date: Thu Mar 3 10:52:15 2022 -0500 Fix build: update package-lock.json with missing changes from package.json, remove extra package files under qe-tests (kubev2v#935) * Fix build: update package-lock.json with missing changes from package.json * Delete extra package files under qe-tests commit 95f0052 Author: Mike Turley <mike.turley@alum.cs.umass.edu> Date: Wed Mar 2 13:06:05 2022 -0500 Bug 2059879: Invalidate cached secrets when patching a provider CR (kubev2v#933) commit 0851df9 Author: kpunwatk <kpunwatk@redhat.com> Date: Wed Mar 2 16:10:05 2022 +0530 [QE] Missing hooks step in edit plan method (kubev2v#929) modified: pkg/qe-tests/cypress/integration/models/plan.ts Co-authored-by: Igor Braginsky <ibragins@redhat.com> commit eb38ac7 Author: Mike Turley <mike.turley@alum.cs.umass.edu> Date: Tue Mar 1 12:37:27 2022 -0500 Bug 2049638: Address feedback from PR#928 (kubev2v#930)
mturley · Jun 23, 2022 · 2e080d7 · 2e080d7
1 parent 794c758
commit 2e080d7
Show file tree

Hide file tree

Showing 49 changed files with 6,297 additions and 861 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -62,6 +62,7 @@
     "css-loader": "^5.2.6",
     "css-minimizer-webpack-plugin": "^3.1.4",
     "cypress": "^9.1.1",
+    "cypress-localstorage-commands": "^2.1.0",
     "cypress-mochawesome-reporter": "^2.3.0",
     "cypress-multi-reporters": "^1.5.0",
     "eslint": "^8.6.0",
@@ -112,7 +113,7 @@
     "axios": "^0.21.2",
     "dayjs": "^1.10.5",
     "dayjs-ext": "^2.2.0",
-    "ejs": "^3.1.6",
+    "ejs": "^3.1.7",
     "express": "^4.17.1",
     "fast-deep-equal": "^3.1.3",
     "file-saver": "^2.0.5",

diff --git a/pkg/qe-tests/cypress/integration/models/mapping.ts b/pkg/qe-tests/cypress/integration/models/mapping.ts
@@ -1,11 +1,16 @@
 import { MappingData, MappingPeer } from '../types/types';
 import * as view from '../views/mapping.view';
-import { applyAction, clickByText, inputText, openSidebarMenu } from '../../utils/utils';
+import {
+  applyAction,
+  clickByText,
+  inputText,
+  openSidebarMenu,
+  selectFromDroplist,
+} from '../../utils/utils';
 import {
   button,
   selectSource,
   selectTarget,
-  selectProvider,
   create,
   mappings,
   createMapping,
@@ -16,20 +21,22 @@ import {
   SEC,
   addButton,
 } from '../types/constants';
-import { buttonNavLink, buttonModal, inputAttr, dataLabel } from '../views/mapping.view';
+import {
+  buttonNavLink,
+  buttonModal,
+  inputAttr,
+  dataLabel,
+  selectSourceProviderMenu,
+  selectTargetProviderMenu,
+} from '../views/mapping.view';
 
 export class Mapping {
-  protected selectProvider(providerName: string): void {
-    clickByText(button, selectProvider);
-    clickByText(button, providerName);
-  }
-
   protected selectSourceProvider(providerName: string): void {
-    this.selectProvider(providerName);
+    selectFromDroplist(selectSourceProviderMenu, providerName);
   }
 
   protected selectTargetProvider(providerName: string): void {
-    this.selectProvider(providerName);
+    selectFromDroplist(selectTargetProviderMenu, providerName);
   }
 
   protected selectInputByAttr(
@@ -98,7 +105,7 @@ export class Mapping {
 
   protected validateReady(mappingData: MappingData): void {
     const { name } = mappingData;
-    cy.contains(name)
+    cy.contains(name, { timeout: 120 * SEC })
       .parent(trTag)
       .within(() => {
         cy.get(dataLabel.status, { timeout: 600 * SEC }).should('contain.text', 'OK');

diff --git a/pkg/qe-tests/cypress/integration/models/plan.ts b/pkg/qe-tests/cypress/integration/models/plan.ts
@@ -71,6 +71,7 @@ export class Plan {
     openSidebarMenu();
     clickByText(navMenuPoint, migrationPLan);
   }
+
   protected fillName(name: string): void {
     inputText(planNameInput, name);
   }
@@ -105,7 +106,7 @@ export class Plan {
       //To select Openshift Virtualization Migration network in plan wizard
       cy.wait(2 * SEC);
       clickByText(button, differentNetwork);
-      cy.get(network).should('contain.text', ocpMigrationNetwork);
+      cy.get(network, { timeout: 120 * SEC }).should('contain.text', ocpMigrationNetwork);
       selectFromDroplist(network, podNetwork);
       confirm();
     }
@@ -122,12 +123,13 @@ export class Plan {
     } else if (providerData.type == providerType.rhv) {
       cluster = sourceClusterName;
     }
-    cy.contains('label', cluster, { timeout: 60 * SEC })
+    cy.contains('label', cluster, { timeout: 120 * SEC })
       .closest('.pf-c-tree-view__node-container')
       .within(() => {
         click(button);
       });
-    const selector = `[aria-label="Select Cluster ${sourceClusterName}"]`;
+    //As middle of text there is flaky I'm searching for `Select` and name of the cluster
+    const selector = `[aria-label^=Select][aria-label$=${sourceClusterName}]`;
     selectCheckBox(selector); //Added selectCheckBox function
     next();
   }
@@ -138,14 +140,15 @@ export class Plan {
     vmList.forEach((name) => {
       inputText(searchInput, name);
       click(selector);
-      cy.get(tdTag)
+      cy.get(tdTag, { timeout: 120 * SEC })
         .contains(name)
         .closest(trTag)
         .within(() => {
           selectCheckBox('input');
         });
     });
   }
+
   //Method to unselect VMs those are not needed
   protected unSelectVm(vmList: string[]): void {
     const selector = `[aria-label="search button for search input"]`;
@@ -248,7 +251,6 @@ export class Plan {
 
   protected finalReviewStep(planData: PlanData): void {
     const { name, sProvider, tProvider, namespace } = planData;
-    // const totalVmAmount = vmwareSourceVmList.length;
     this.reviewPlanName(name);
     this.reviewSourceProvider(sProvider);
     this.reviewTargetProvider(tProvider);
@@ -278,9 +280,7 @@ export class Plan {
       .contains(name)
       .closest(trTag)
       .within(() => {
-        cy.get(dataLabel.status).contains(planSuccessMessage, {
-          timeout: 3600 * SEC,
-        });
+        cy.get(dataLabel.status).contains(planSuccessMessage, { timeout: 3600 * SEC });
       });
   }
 
@@ -311,10 +311,10 @@ export class Plan {
       });
   }
   //Method for different Migaration plan step
-  protected waitForState(planStep: string): void {
+  protected waitForState(step: string): void {
     click(arrowDropDown); //click on dropdown arrow to see the plan steps box
     cy.get(dataLabel.step, { timeout: 200 * SEC })
-      .contains(planStep)
+      .contains(step)
       .closest(trTag)
       .within(() => {
         cy.get(dataLabel.elapsedTime, { timeout: 3600 * SEC })
@@ -350,7 +350,7 @@ export class Plan {
         .within(() => {
           cy.get('[data-label="Status"]', { timeout: 3600 * SEC }).should(
             'contain.text',
-            'Idle - Next incremental copy will begin in less than 1 minute'
+            'Idle - Next incremental copy will begin in'
           );
         });
     }
@@ -515,6 +515,7 @@ export class Plan {
     clickOnCancel();
     this.waitForCanceled(name);
   }
+
   //Method to click on Get Logs and Download logs
   getLogs(planData: PlanData): void {
     const { name } = planData;
@@ -523,6 +524,36 @@ export class Plan {
     clickByText(getlogsConfirmButton, getLogsButton);
     cy.wait(20 * SEC);
     this.run(name, downloadLogsButton);
+    this.validateLogs(planData);
+  }
+
+  // This method calls external python program that will take the downloaded archive and parse it
+  protected validateLogs(planData: PlanData): void {
+    const { name, vmList } = planData;
+    let vms = '';
+    let hook_sufix = '';
+    vmList.forEach((current_vm) => {
+      vms = vms + current_vm + ' ';
+    });
+    if (
+      planData.preHook.ansiblePlaybook ||
+      planData.postHook.ansiblePlaybook ||
+      planData.preHook.image ||
+      planData.postHook.image
+    ) {
+      hook_sufix = ' --hook true';
+    }
+    const command_line =
+      'python logInspector/main.py' +
+      ' -p ' +
+      name +
+      ' -v' +
+      vms +
+      '-f ~/Downloads/must-gather-plan ' +
+      name +
+      '.tar.gz' +
+      hook_sufix;
+    cy.exec(command_line).its('stdout').should('contain', 'PASSED');
   }
 
   duplicate(originalPlanData: PlanData, duplicatePlanData: PlanData): void {
@@ -564,6 +595,7 @@ export class Plan {
     this.networkMappingStep(duplicatePlanData);
     next();
     this.selectMigrationTypeStep(duplicatePlanData);
+    next();
     this.reviewSourceProvider(sProvider);
     this.reviewTargetProvider(tProvider);
     this.reviewTargetNamespace(namespace);

diff --git a/pkg/qe-tests/cypress/integration/models/providerRhv.ts b/pkg/qe-tests/cypress/integration/models/providerRhv.ts
@@ -11,13 +11,18 @@ import {
   dataLabel,
 } from '../views/providerRhv.view';
 import { providerMenu } from '../views/provider.view';
-import { removeButton, rhv, SEC, button, trTag } from '../types/constants';
+import {
+  removeButton,
+  rhv,
+  SEC,
+  button,
+  trTag,
+  editButton,
+  saveButton,
+  incorrectRhvHostname,
+} from '../types/constants';
 
 export class providerRhv extends Provider {
-  protected runWizard(providerData: RhvProviderData): void {
-    super.runWizard(providerData);
-  }
-
   protected fillName(name: string): void {
     inputText(instanceName, name);
   }
@@ -49,22 +54,49 @@ export class providerRhv extends Provider {
     cy.contains(name)
       .parent(trTag)
       .within(() => {
-        // Validating that provider is in `Ready` state
-        cy.get(dataLabel.status, { timeout: 600 * SEC }).should('have.text', 'Ready');
-        // Validating that endpoint is in proper format and contains proper URL
-        cy.get(dataLabel.endpoint).should('contain.text', `https://${hostname}/ovirt-engine/api`);
-        // Validating that amount of clusters is not empty and is not 0
-        cy.get(dataLabel.clusters).should('not.be.empty').should('not.contain.text', '0');
-        // Validating that amount of hosts is not empty and is not 0
-        cy.get(dataLabel.hosts).should('not.be.empty').should('not.contain.text', '0');
-        // Validating that amount of VMs is not empty and is not 0
-        cy.get(dataLabel.vms).should('not.be.empty');
-        // Validating that amount of networks is not empty and is not 0
-        cy.get(dataLabel.networks).should('not.be.empty').should('not.contain.text', '0');
-        // Validating that amount of storageDomains is not empty and is not 0
-        cy.get(dataLabel.storageDomains).should('not.be.empty').should('not.contain.text', '0');
+        if (hostname == incorrectRhvHostname) {
+          cy.get(dataLabel.status, { timeout: 600 * SEC }).should('have.text', 'Critical');
+        } else {
+          // Validating that provider is in `Ready` state
+          cy.get(dataLabel.status, { timeout: 600 * SEC }).should('have.text', 'Ready');
+          // Validating that endpoint is in proper format and contains proper URL
+          cy.get(dataLabel.endpoint).should('contain.text', `https://${hostname}/ovirt-engine/api`);
+          // Validating that amount of clusters is not empty and is not 0
+          cy.get(dataLabel.clusters).should('not.be.empty').should('not.contain.text', '0');
+          // Validating that amount of hosts is not empty and is not 0
+          cy.get(dataLabel.hosts).should('not.be.empty').should('not.contain.text', '0');
+          // Validating that amount of VMs is not empty and is not 0
+          cy.get(dataLabel.vms).should('not.be.empty');
+          // Validating that amount of networks is not empty and is not 0
+          cy.get(dataLabel.networks).should('not.be.empty').should('not.contain.text', '0');
+          // Validating that amount of storageDomains is not empty and is not 0
+          cy.get(dataLabel.storageDomains).should('not.be.empty').should('not.contain.text', '0');
+        }
       });
   }
+  protected runWizard(providerData: RhvProviderData): void {
+    const { name, hostname, username, password, cert } = providerData;
+    super.runWizard(providerData);
+    this.fillName(name);
+    this.fillHostname(hostname);
+    this.fillUsername(username);
+    this.fillPassword(password);
+    this.fillCaCert(cert);
+    click(addButtonModal);
+    cy.wait(2 * SEC);
+  }
+  //Edit method RHV
+  edit(providerData: RhvProviderData): void {
+    const { name, hostname, username, password, cert } = providerData;
+    providerRhv.openList();
+    applyAction(name, editButton);
+    this.fillHostname(hostname);
+    this.fillUsername(username);
+    this.fillPassword(password);
+    this.fillCaCert(cert);
+    clickByText(addButtonModal, saveButton);
+    this.populate(providerData);
+  }
 
   create(providerData: RhvProviderData): void {
     const { name, hostname, username, password, cert } = providerData;