New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gherkin for integration of tekton hub with pipeline builder #9747
Gherkin for integration of tekton hub with pipeline builder #9747
Conversation
cc3151e
to
49e1293
Compare
49e1293
to
78f385f
Compare
@@ -27,7 +27,9 @@ Feature: Create the pipeline from builder page | |||
Scenario Outline: Create a pipeline with series tasks: P-02-TC02 | |||
Given user is at Pipeline Builder page | |||
When user enters pipeline name as "<pipeline_name>" | |||
And user selects "<task_name>" from Task drop down | |||
And user clicks on Add task | |||
And user searches "<task_name>" in quick search bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And user searches "<task_name>" in quick search bar | |
And user searches "<cluster_task_name>" in quick search bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
@@ -69,7 +75,9 @@ Feature: Create the pipeline from builder page | |||
Scenario Outline: Create pipeline with "<resource_type>" as resource type from pipeline builder page: P-02-TC05 | |||
Given user is at Pipeline Builder page | |||
When user enters pipeline name as "<pipeline_name>" | |||
And user selects "<task_name>" from Task drop down | |||
And user clicks on Add task | |||
And user searches "<task_name>" in quick search bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add option will be shown only for already installed clusterTask or namespace task through tektonhub. so change this in all the Add
button scenarios to make it more readable.
And user searches "<task_name>" in quick search bar | |
And user searches "<cluster_task_name>" in quick search bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
And user searches "kn" in quick search bar | ||
And user clicks on "Add" in "kn" task | ||
And user adds a task in series | ||
And user scrolls over to the newly added task |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And user scrolls over to the newly added task | |
And user hovers over the newly added task |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
And user adds a task in series | ||
And user scrolls over to the newly added task | ||
And user clicks on delete icon | ||
Then user can see the taks in series gets removed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then user can see the taks in series gets removed | |
Then user can see the task in series gets removed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected
Given user is at Pipeline Builder page | ||
When user enters pipeline name as "<pipeline_name>" | ||
And user clicks on Add task | ||
And user searches and select "<task_name>" in the list of items based on the provider="tekton" in quick search bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we change like this ?
And user searches and select "<task_name>" in the list of items based on the provider="tekton" in quick search bar | |
And user searches and select "<task_name>" in the list of items based on the "tekton" provider in quick search bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
And user clicks on Add task | ||
And user searches and select "<task_name>" in the list of items based on the provider="tekton" in quick search bar | ||
And user clicks on "Install and Add" in "<task_name>" task | ||
And user clicks Create button on Pipeline Builder page |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And user clicks Create button on Pipeline Builder page | |
And user should see the loading node until the installation is complete | |
And user should see the Create button enabled after installation | |
And user clicks Create button on Pipeline Builder page |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
78f385f
to
e0b2a0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
And user clicks on Add task | ||
And user searches "openshift-client" in quick search bar | ||
And user changes version to latest | ||
And user clicks on "Updated and Add" button |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And user clicks on "Updated and Add" button | |
And user clicks on "Update and Add" button |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see you have added tektonHub scenarios
- Install and Add
- update and Add
Shall we also include Add
flow, for already installed tektonhub or a ClusterTask ?
e0b2a0f
to
3415fad
Compare
@karthikjeeyar Updated the pr with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@regression @to-do @odc-4315 | ||
Scenario Outline: Create a pipeline with TektonHub task present in cluster from pipeline builder page: P-02-TC010 | ||
Given user is at Pipeline Builder page | ||
And user has installed tekton hub "<task_name>" kn in cluster |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And user has installed tekton hub "<task_name>" kn in cluster | |
And user has installed tekton hub "<task_name>" task in cluster |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
And user clicks on "Update and Add" button | ||
And user clicks Create button on Pipeline Builder page | ||
Then user will be redirected to Pipeline Details page with header name "<pipeline_name>" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sanketpathak Don't you miss the Examples section here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Edited the name part
3415fad
to
28a1b22
Compare
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gajanan-more, karthikjeeyar, psrna, 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:
Approvers can indicate their approval by writing |
As this does not change feature code |
Epic: https://issues.redhat.com/browse/ODC-4315
Story: https://issues.redhat.com/browse/ODC-5972
Acceptance criteria:
Checks required for approving Epic gherkin scripts PR:
yarn run gherkin-lint
on frontend folder [Skip epic number tags related linter issues]