Gherkin for export of application in topology #9887

sanketpathak · 2021-08-26T05:40:34Z

Epic: https://issues.redhat.com/browse/ODC-2148
Story: https://issues.redhat.com/browse/ODC-6280

Acceptance criteria:

Users can access an Export CTA from Topology to export resources in the selected namespace. Note that users will not be able to pick and choose which resources will be exported.

Checks required for approving Epic gherkin scripts PR:

Add @epic-number to the scenarios or feature file [e.g: @odc-xxx]
Add @to-do for automation possible scenarios
Add @regression or @smoke based on the importance of the scenario
Update the test scenarios count in Automation status confluence Report
Check for the linter issues by executing yarn run gherkin-lint on frontend folder [Skip epic number tags related linter issues]

invincibleJai · 2021-08-26T10:49:15Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+              And user has created a deployment workload
+             When user goes to Topology
+              And user clicks on Export Application button
+             Then user can see Export Application modal saying "Export of resources in aut-export-application has started."


it'll be a toast message

invincibleJai · 2021-08-26T10:50:36Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+        Scenario: Export Application is created: KC-02-TC02
+            Given user is at Topology
+              And Export Application is completed
+              And user clicks on download button


user will be shown with a toast message on export completion along with a link to download

this doesn't look right And user clicks on download button

makambalaji · 2021-08-30T06:05:48Z

/assign @makambalaji

makambalaji

Do we need "export-application" folder in Topology plugin folder?

makambalaji · 2021-08-30T12:10:16Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+            Given user has installed Gitops primer Operator
+              And user has created or selected namespace "aut-export-application"


Suggested change

Given user has installed Gitops primer Operator

And user has created or selected namespace "aut-export-application"

Given user has installed Gitops primer Operator

And user is at developer perspective

And user has created or selected namespace "aut-export-application"

makambalaji · 2021-08-30T12:10:47Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+
+        @smoke @to-do
+        Scenario: Export application button in topology: EA-02-TC01
+            Given user is at Developer perspective


Move it to BackGround section and remove it from here

makambalaji · 2021-08-30T12:11:23Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+        @smoke @to-do
+        Scenario: Export application button in topology: EA-02-TC01
+            Given user is at Developer perspective
+              And user has created a deployment workload


Provide the test data ""

makambalaji · 2021-08-30T12:11:33Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+        Scenario: Export application button in topology: EA-02-TC01
+            Given user is at Developer perspective
+              And user has created a deployment workload
+             When user goes to Topology


Suggested change

When user goes to Topology

When user navigates to Topology

makambalaji · 2021-08-30T12:11:55Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+
+        @regression @to-do
+        Scenario: Export Application modal when export application is in progress: EA-02-TC02
+            Given user is at Topology


Suggested change

Given user is at Topology

Given user is at Topology page

makambalaji · 2021-08-30T12:12:26Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+
+        @regression @to-do
+        Scenario: Export application button in empty topology view: EA-02-TC03
+            Given user is at Developer perspective


Remove this line and move it to BackGround section

makambalaji · 2021-08-30T12:12:54Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+        Scenario: Export application button in empty topology view: EA-02-TC03
+            Given user is at Developer perspective
+              And user has created or selected namespace "aut-export-applicatio-temp"
+             When user goes to Topology


Suggested change

When user goes to Topology

When user navigates to Topology page

makambalaji · 2021-08-30T12:13:09Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+
+        @regression @manual
+        Scenario: Export Application is created: EA-02-TC04
+            Given user is at Topology


Suggested change

Given user is at Topology

Given user is at Topology page

makambalaji · 2021-08-30T12:15:11Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+            Given user is at Topology
+              And Export Application is completed
+             Then user can see a toast message on export completion along with a link to download
+              And user can see a deployment workload for primer is created


In Given condition,
'User created deployment workload "<>"' is required and share that workload name here in this step for more clarity

No, this is a workload when the primer Job is finished the primer deployment is created

makambalaji · 2021-08-30T12:15:19Z

...topology/integration-tests/features/topology/export-application/export-of-applicaton.feature

+
+        @regression @manual
+        Scenario: Download the exported application: EA-02-TC05
+            Given user is at Topology


Suggested change

Given user is at Topology

Given user is at Topology page

sanketpathak · 2021-08-30T16:12:44Z

Yes, as the feature code is present in the Topology plugin so we need the export application tests in Topology at this moment

invincibleJai · 2021-08-31T13:03:44Z

/approve

cc @makambalaji

makambalaji · 2021-08-31T14:54:23Z

/lgtm

makambalaji · 2021-08-31T14:54:44Z

/label px-approved
/label docs-approved
/label qe-approved

openshift-ci · 2021-08-31T14:57:51Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: invincibleJai, makambalaji, sanketpathak

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

~~frontend/packages/topology/integration-tests/OWNERS~~ [invincibleJai,makambalaji]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci bot requested review from debsmita1 and gajanan-more August 26, 2021 05:41

openshift-ci bot added the component/topology Related to topology label Aug 26, 2021

invincibleJai reviewed Aug 26, 2021

View reviewed changes

sanketpathak force-pushed the gherkin-export-application branch 2 times, most recently from ad9ab39 to 68d5407 Compare August 27, 2021 12:48

openshift-ci bot assigned makambalaji Aug 30, 2021

makambalaji suggested changes Aug 30, 2021

View reviewed changes

sanketpathak force-pushed the gherkin-export-application branch from 68d5407 to 3d06b27 Compare August 30, 2021 16:09

gherkin for export of application in topology

acfa55e

sanketpathak force-pushed the gherkin-export-application branch from 3d06b27 to acfa55e Compare August 31, 2021 11:48

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 31, 2021

openshift-ci bot added px-approved Signifies that Product Support has signed off on this PR docs-approved Signifies that Docs has signed off on this PR qe-approved Signifies that QE has signed off on this PR lgtm Indicates that a PR is ready to be merged. labels Aug 31, 2021

openshift-merge-robot merged commit b9c5b6a into openshift:master Aug 31, 2021

spadgett added this to the v4.9 milestone Sep 4, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Gherkin for export of application in topology #9887

Gherkin for export of application in topology #9887

sanketpathak commented Aug 26, 2021 •

edited

invincibleJai Aug 26, 2021

sanketpathak Aug 26, 2021

invincibleJai Aug 26, 2021

sanketpathak Aug 26, 2021

invincibleJai Aug 26, 2021

sanketpathak Aug 27, 2021

makambalaji commented Aug 30, 2021

makambalaji left a comment

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021 •

edited

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

makambalaji Aug 30, 2021

sanketpathak Aug 30, 2021

sanketpathak commented Aug 30, 2021

invincibleJai commented Aug 31, 2021

makambalaji commented Aug 31, 2021

makambalaji commented Aug 31, 2021

openshift-ci bot commented Aug 31, 2021

		Given user has installed Gitops primer Operator
		And user has created or selected namespace "aut-export-application"

	When user goes to Topology
	When user navigates to Topology page

Gherkin for export of application in topology #9887

Gherkin for export of application in topology #9887

Conversation

sanketpathak commented Aug 26, 2021 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

makambalaji commented Aug 30, 2021

makambalaji left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sanketpathak Aug 30, 2021 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sanketpathak commented Aug 30, 2021

invincibleJai commented Aug 31, 2021

makambalaji commented Aug 31, 2021

makambalaji commented Aug 31, 2021

openshift-ci bot commented Aug 31, 2021

sanketpathak commented Aug 26, 2021 •

edited

sanketpathak Aug 30, 2021 •

edited