Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[CHE 6] Update RenameProjectTest selenium test #7300

Merged
merged 1 commit into from
Nov 10, 2017
Merged

[CHE 6] Update RenameProjectTest selenium test #7300

merged 1 commit into from
Nov 10, 2017

Conversation

SkorikSergey
Copy link
Contributor

We need update RenameProjectTest selenium test:

  1. Cover use case described in issue A project has a simple folder type after renaming #6352 (project has a simple folder type after renaming) to ensure that functionality works properly.
  2. Check that the Rename Project form is opened by Shift+F6 keys, the project context menu and from the main menu.

What issues does this PR fix or reference?

#7290

added testing that the rename project dialog opened by menu and hot k…
6925304 
…eys, check that after renaming folder type is project
@benoitf benoitf added target/che6 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. labels Nov 10, 2017
dmytro-ndp
dmytro-ndp approved these changes Nov 10, 2017
View reviewed changes
Copy link
Contributor

@dmytro-ndp dmytro-ndp left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Please, don't forget to add into description of this PR the link to the twin PR into master.

@SkorikSergey SkorikSergey mentioned this pull request Nov 10, 2017
Merged
@SkorikSergey SkorikSergey merged commit 933fade into che6 Nov 10, 2017
@SkorikSergey SkorikSergey deleted the reworkRenameProjectTestChe6 branch November 10, 2017 12:31
@benoitf benoitf added this to the 6.0.0-M2 milestone Nov 10, 2017
@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 Nov 10, 2017
@SkorikSergey
Copy link
Contributor Author

SkorikSergey commented Nov 10, 2017 •
edited
Loading

Pull request for master branch merged #7291.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmytro-ndp dmytro-ndp dmytro-ndp approved these changes

@tolusha tolusha tolusha approved these changes

@vparfonov vparfonov Awaiting requested review from vparfonov

Assignees
No one assigned
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Milestone
6.0.0-M2
Development

Successfully merging this pull request may close these issues.
4 participants
@SkorikSergey @dmytro-ndp @tolusha @benoitf