Skip to content

Selenium: cover #8674 issue in selenium test#10006

Merged
SkorikSergey merged 6 commits intomasterfrom
selenium-8674
Jun 12, 2018
Merged

Selenium: cover #8674 issue in selenium test#10006
SkorikSergey merged 6 commits intomasterfrom
selenium-8674

Conversation

@SkorikSergey
Copy link
Copy Markdown
Contributor

What does this PR do?

This PR adds covering an usecase from #8674 issue for tracking regression to CreateAndDeleteProjectsTest selenium test.

What issues does this PR fix or reference?

#9770

@SkorikSergey SkorikSergey added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/task Internal things, technical debt, and to-do tasks to be performed. team/che-qe labels Jun 12, 2018
@SkorikSergey SkorikSergey requested a review from vparfonov as a code owner June 12, 2018 09:30
projectSourcePage.selectSample(CONSOLE_JAVA_SIMPLE);
projectSourcePage.clickOnAddProjectButton();
projectSourcePage.clickOnAddOrImportProjectButton();
projectSourcePage.selectSample(WEB_JAVA_SPRING);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it seems we are doing it twice.
Could you, please, leave a comment in the code about why we need to import the project and to select the web java spring sample again?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the second creation 'web-java-spring-1' project will be created. I will add comment about it.

@SkorikSergey SkorikSergey merged commit 5a2a5bd into master Jun 12, 2018
@SkorikSergey SkorikSergey deleted the selenium-8674 branch June 12, 2018 12:17
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jun 12, 2018
@benoitf benoitf added this to the 6.7.0 milestone Jun 12, 2018
hbhargav pushed a commit to hbhargav/che that referenced this pull request Dec 5, 2018
@svor svor mentioned this pull request Sep 8, 2021
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/task Internal things, technical debt, and to-do tasks to be performed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants