Skip to content

Commit 3faa1ef

Browse files
authored
FIx EditableGridTest failures (#2838)
1 parent 26543f5 commit 3faa1ef

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/org/labkey/test/components/ui/grids/EditableGrid.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -215,6 +215,7 @@ public EditableGrid shiftSelectRange(int start, int end)
215215
throw new NoSuchElementException("there is no selection column for grid");
216216

217217
var checkBoxes = Locator.tag("tr").child("td")
218+
.child(Locator.byClass("table-cell-content"))
218219
.child(Locator.tagWithAttribute("input", "type", "checkbox"))
219220
.findElements(elementCache().table);
220221
getWrapper().scrollIntoView(checkBoxes.get(start)); // Make sure the header isn't in the way

src/org/labkey/test/tests/component/EditableGridTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -459,12 +459,13 @@ public void testShiftClick()
459459
Tests the scenario where a row is selected, then another, and another are shift-selected
460460
expects the range-bump to redefine the selected range
461461
*/
462-
@Test public void testShiftSelect_bumpSelect()
462+
@Test public void testShiftSelectMultipleTimes()
463463
{
464464
EditableGrid testGrid = goToEditableGrid(PASTING_SAMPLE_TYPE);
465465
testGrid.addRows(15);
466466

467467
Locator boxes = Locator.tag("tr").child("td")
468+
.child(Locator.byClass("table-cell-content"))
468469
.child(Locator.tagWithAttribute("input", "type", "checkbox"));
469470
var checkBoxes = boxes.findElements(testGrid);
470471
scrollIntoView(checkBoxes.get(2), false);

0 commit comments

Comments
 (0)