remove unnecessary fields and methods, fix syntax errors

eclipse-che · Sep 22, 2017 · 6140007 · 6140007
1 parent 3273f82
commit 6140007
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 24 deletions.
diff --git a/...um/che-selenium-test/src/main/java/org/eclipse/che/selenium/pageobject/CodenvyEditor.java b/...um/che-selenium-test/src/main/java/org/eclipse/che/selenium/pageobject/CodenvyEditor.java
@@ -38,7 +38,6 @@
 import java.util.stream.Collectors;
 import org.eclipse.che.selenium.core.SeleniumWebDriver;
 import org.eclipse.che.selenium.core.action.ActionsFactory;
-import org.eclipse.che.selenium.core.constant.TestMenuCommandsConstants;
 import org.eclipse.che.selenium.core.utils.WaitUtils;
 import org.openqa.selenium.By;
 import org.openqa.selenium.Keys;
@@ -61,8 +60,6 @@ public class CodenvyEditor {
 
   public static final String CLOSE_ALL_TABS = "gwt-debug-contextMenu/closeAllEditors";
 
-  @Inject private Menu menu;
-
   public static final class EditorContextMenu {
     public static final String REFACTORING = "contextMenu/Refactoring";
     public static final String REFACTORING_MOVE = "contextMenu/Refactoring/Move";
@@ -514,19 +511,6 @@ public void setCursorToDefinedLineAndChar(int positionLine, int positionChar) {
     waitSpecifiedValueForLineAndChar(positionLine, positionChar);
   }
 
-  public void setCursorToDefinedLineAndCharByMenu() {
-    menu.runCommand(
-        TestMenuCommandsConstants.Run.RUN_MENU, TestMenuCommandsConstants.Run.DEBUG_CONFIGURATION);
-  }
-
-  /**
-   * set cursor in specified position
-   *
-   * @param positionLine is the specified number line
-   * @param positionChar is the specified number char
-   */
-  public void setCursorToDefinedLineAndCharByMenu(int positionLine, int positionChar) {}
-
   /** launch code assistant with ctrl+space keys and wait container is open */
   public void launchAutocompleteAndWaitContainer() {
     loader.waitOnClosed();

diff --git a/selenium/che-selenium-test/src/main/java/org/eclipse/che/selenium/pageobject/Consoles.java b/selenium/che-selenium-test/src/main/java/org/eclipse/che/selenium/pageobject/Consoles.java
@@ -75,7 +75,7 @@ public Consoles(SeleniumWebDriver seleniumWebDriver, Loader loader) {
   }
 
   @FindBy(css = CONSOLE_PANEL_DRUGGER_CSS)
-  WebElement consolesPanelDrugger;
+  WebElement consolesPanelDrag;
 
   @FindBy(id = PROCESSES_TAB)
   WebElement processesTab;
@@ -310,9 +310,8 @@ public void clickOnMaximizePanelIcon() {
    *
    * @param xoffset offset in pixels for shifting
    */
-  public void drugConsolesInDefinePosition(int xoffset) {
-    WebElement drugger =
-        redrawDriverWait.until(ExpectedConditions.visibilityOf(consolesPanelDrugger));
-    new Actions(seleniumWebDriver).dragAndDropBy(drugger, xoffset, xoffset).perform();
+  public void dragConsolesInDefinePosition(int xoffset) {
+    WebElement drag = redrawDriverWait.until(ExpectedConditions.visibilityOf(consolesPanelDrag));
+    new Actions(seleniumWebDriver).dragAndDropBy(drag, xoffset, xoffset).perform();
   }
 }
diff --git a/...rc/main/java/org/eclipse/che/selenium/pageobject/plugins/JavaTestRunnerPluginConsole.java b/...rc/main/java/org/eclipse/che/selenium/pageobject/plugins/JavaTestRunnerPluginConsole.java
@@ -43,7 +43,6 @@ public class JavaTestRunnerPluginConsole extends Consoles {
       "//div[contains(@id,'gwt-uid')]//div[@style='color: red;']";
   private static final String METHODS_MARKED_AS_IGNORED =
       "//div[contains(@id,'gwt-uid')]//div[@style='text-decoration: line-through; color: yellow;']";
-  private static final String TEST_RESULT_TREE_ITEM = "gwt-debug-test-tree-navigation-panel";
 
   private static final String TEST_RESULT_NAVIGATION_TREE = "gwt-debug-test-tree-navigation-panel";
 
@@ -188,7 +187,7 @@ private List<String> getAllMetodsWithDefinedStatus(List<WebElement> definedMetho
   }
 
   /**
-   * get taxt from the test result tree. Mote! This method represent only text from test result tree
+   * get text from the test result tree. Mote! This method represent only text from test result tree
    * without styles and formatting
    *
    * @return text representation of results of the test result tree widget

diff --git a/...org/eclipse/che/selenium/testrunner/JavaTestPluginJuinit4CheckRunSuitesAndScopesTest.java b/...org/eclipse/che/selenium/testrunner/JavaTestPluginJuinit4CheckRunSuitesAndScopesTest.java
@@ -74,7 +74,7 @@ public void prepareTestProject() throws Exception {
     projectExplorer.waitItem(JUNIT4_PROJECT);
     runCompileCommandByPallete(compileCommand);
     notifications.waitProgressPopupPanelClose();
-    consoles.drugConsolesInDefinePosition(-100);
+    consoles.dragConsolesInDefinePosition(-100);
   }
 
   @Test

diff --git a/selenium/che-selenium-test/src/test/resources/suites/CheSuite.xml b/selenium/che-selenium-test/src/test/resources/suites/CheSuite.xml
@@ -546,6 +546,7 @@
             </class>
             <class name="org.eclipse.che.selenium.testrunner.JavaTestPluginJunit4Test"/>
             <class name="org.eclipse.che.selenium.testrunner.JavaTestPluginTestNgTest"/>
+            <class name="org.eclipse.che.selenium.testrunner.JavaTestPluginJuinit4CheckRunSuitesAndScopesTest"/>
             <class name="org.eclipse.che.selenium.preferences.CheckErrorsWarningsTabTest">
                 <methods>
                     <!-- unstable -->