Skip to content

Commit

Permalink
Renaming AbstractTable.getSelectetItems
Browse files Browse the repository at this point in the history
Signed-off-by: Ondrej Dockal <odockal@redhat.com>
  • Loading branch information
odockal authored and rawagner committed Feb 20, 2018
1 parent b7d743a commit e9b4330
Show file tree
Hide file tree
Showing 6 changed files with 20 additions and 6 deletions.
Expand Up @@ -104,7 +104,7 @@ public String addVariable(String name , String value , boolean overwriteIfExists
}
new OkButton(variableEntryShell).click();
new WaitWhile(new ShellIsAvailable(variableEntryShell));
return new DefaultTable(this).getSelectetItems().get(0).getText();
return new DefaultTable(this).getSelectedItems().get(0).getText();
}

/**
Expand Down
Expand Up @@ -57,7 +57,7 @@ public boolean isUseHTMLTeplate(){
* @return selected HTML template
*/
public String getHTMLTemplate(){
return new DefaultTable(this).getSelectetItems().get(0).getText();
return new DefaultTable(this).getSelectedItems().get(0).getText();
}


Expand Down
Expand Up @@ -149,5 +149,13 @@ public interface Table extends Control<org.eclipse.swt.widgets.Table>, Reference
*
* @return list of selected table items
*/
List<TableItem> getSelectedItems();

/**
* @deprecated As of 2.1.0 because of typo in method name.
* Use {@link #getSelectedItems()} instead.
*
*/
@Deprecated
List<TableItem> getSelectetItems();
}
Expand Up @@ -263,7 +263,7 @@ public String getHeader(int index) {
* @see org.eclipse.reddeer.swt.api.Table#getSelectetItems()
*/
@Override
public List<TableItem> getSelectetItems(){
public List<TableItem> getSelectedItems(){
LinkedList<TableItem> result = new LinkedList<TableItem>();
for (TableItem tableItem : getItems()){
if (tableItem.isSelected()){
Expand All @@ -272,4 +272,10 @@ public List<TableItem> getSelectetItems(){
}
return result;
}

@Override
@Deprecated
public List<TableItem> getSelectetItems(){
return getSelectedItems();
}
}
Expand Up @@ -56,7 +56,7 @@ public void teardown() {
public void selectFileTypeTest() {
prefPage.selectFileType("*.html");
DefaultTable defaultTable = new DefaultTable();
List<TableItem> selectetItems = defaultTable.getSelectetItems();
List<TableItem> selectetItems = defaultTable.getSelectedItems();

assertEquals("Only one item should be selected", 1, selectetItems.size());
assertTrue("*.html file type should be selected", selectetItems.get(0).getText().equals("*.html"));
Expand Down
Expand Up @@ -197,7 +197,7 @@ public void testGetIndexOfNonexistingHeader() {
public void testGetSelectedItemsSingle() {
org.eclipse.reddeer.swt.api.Table table = new DefaultTable();
table.select(2);
List<org.eclipse.reddeer.swt.api.TableItem> selectedTableItems = table.getSelectetItems();
List<org.eclipse.reddeer.swt.api.TableItem> selectedTableItems = table.getSelectedItems();
int numSelected = selectedTableItems.size();
assertTrue("Only one Table Item has to be selected but " + numSelected + " are.", numSelected == 1);
org.eclipse.reddeer.swt.api.TableItem expectedSelectedItem = table.getItem(2);
Expand All @@ -210,7 +210,7 @@ public void testGetSelectedItemsSingle() {
public void testGetSelectedItemsMulti() {
org.eclipse.reddeer.swt.api.Table table = new DefaultTable();
table.select(1, 3);
List<org.eclipse.reddeer.swt.api.TableItem> selectedTableItems = table.getSelectetItems();
List<org.eclipse.reddeer.swt.api.TableItem> selectedTableItems = table.getSelectedItems();
int numSelected = selectedTableItems.size();
assertTrue("Two Table Item has to be selected but " + numSelected + " are.", numSelected == 2);
org.eclipse.reddeer.swt.api.TableItem expectedSelectedItem = table.getItem(1);
Expand Down

0 comments on commit e9b4330

Please sign in to comment.