-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Selenium: Update javadocs to actual state in the "CodenvyEditor" class #9115
Conversation
/** | ||
* Waits until editor's tab with specified {@code nameOfFile} is closed. | ||
* | ||
* @param nameOfFile visible name of the editor's tab |
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.
did you mean title of tab?
@@ -596,7 +624,9 @@ private void openGoToLineFormAndSetCursorToPosition(int positionLine, int positi | |||
askForValueDialog.waitFormToClose(); | |||
} | |||
|
|||
/** launch code assistant with ctrl+space keys and wait container is open */ | |||
/** | |||
* Launches code assistant by "ctrl" + "space" keys combination and waits until container is open. |
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.
is opened? (passive voice)
@@ -657,7 +687,7 @@ public void waitMarkerInPosition(MarkerLocator markerLocator, int position) { | |||
expectedNumberOfActiveLine(position); | |||
} | |||
|
|||
/** Wait for no Git change markers in the opened editor. */ | |||
/** Waits for no Git change markers in the opened editor. */ |
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.
"No Git Change" markers
is clearer.
@@ -670,7 +700,8 @@ public void waitNoGitChangeMarkers() { | |||
} | |||
|
|||
/** | |||
* Wait for Git insertion marker in the opened editor. | |||
* Waits for Git insertion marker between {@code startLine} and {@code endLine} including this two |
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.
it could be written simpler: between the specified fromLine, inclusive, and toLine, inclusive.
public String getAllVisibleTextFromProposition() { | ||
waitPropositionAssistContainer(); | ||
return seleniumWebDriverHelper.waitVisibilityAndGetText(propositionContainer); | ||
} | ||
|
||
/** wait assist proposition container is open */ | ||
/** Waits until assist proposition container is open. */ |
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.
is opened (Passive voice)
*/ | ||
public void waitActiveTabFileName(String nameOfFile) { | ||
seleniumWebDriverHelper.waitPresence(By.xpath(format(ACTIVE_TAB_FILE_NAME, nameOfFile))); | ||
} | ||
|
||
/** check that files have been closed. (Check disappears all text areas and tabs) */ | ||
/** Checks that all files have been closed. (Checks disappears all text areas and tabs) */ |
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.
disappears > disappearance of
@@ -1139,7 +1226,7 @@ public void selectLineAndDelete() { | |||
typeTextIntoEditor(DELETE.toString()); | |||
} | |||
|
|||
/** Deletes current line with Ctrl+D */ | |||
/** Deletes current editor's line with Ctrl+D. */ |
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.
with Ctrl+D
> by Ctrl+D keys pressing
* | ||
* @param position position of the breakpoint | ||
* @param position number of line in which breakpoint should be placed |
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.
in which
> where
} | ||
|
||
/** | ||
* select defined implementation in the 'Implementation' form | ||
* Selects defined {@code itemName} in the 'Implementation' form |
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.
defined
> specified
* Select tab by index of editor window after split | ||
* Selects editor's tab by specified {@code index} and {@code tabName}. | ||
* | ||
* <p>Note! Is used for split editor. |
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.
Is used for split editor.
is not clear, as for me.
* | ||
* @param selectedElement is the selected element into editor | ||
* @param selectedElement visible text in the editor on which context click should be perform |
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.
be performed
(Passive Voice)
@@ -1854,18 +2006,18 @@ public void openContextMenuOnElementInEditor(String selectedElement) { | |||
waitContextMenu(); | |||
} | |||
|
|||
/** wait context menu form is open */ | |||
/** Waits until context menu form is open. */ |
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.
is opened
What does this PR do?
Update javadocs to actual state in the "CodenvyEditor" class.
What issues does this PR fix or reference?
Issue: #9085
Release Notes
Docs PR