Skip to content

Commit 31abdff

Browse files
Require Java 25 (#2833)
Co-authored-by: labkey-tchad <[email protected]>
1 parent d6a8d90 commit 31abdff

File tree

3 files changed

+7
-13
lines changed

3 files changed

+7
-13
lines changed

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
apacheCommonsCsvVersion=1.14.1
22

3-
aspectjVersion=1.9.24
3+
aspectjVersion=1.9.25.1
44

55
assertjVersion=3.27.6
66

src/org/labkey/serverapi/reader/DataLoader.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -483,13 +483,6 @@ public void close() throws IOException
483483
{
484484
_closed = true;
485485
}
486-
487-
@Override
488-
protected void finalize() throws Throwable
489-
{
490-
super.finalize();
491-
// assert _closed; TODO: Uncomment to force all callers to close iterator.
492-
}
493486
}
494487
}
495488

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

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package org.labkey.test.tests.component;
22

33
import org.apache.commons.lang3.StringUtils;
4+
import org.apache.commons.lang3.mutable.MutableInt;
45
import org.json.JSONObject;
56
import org.junit.BeforeClass;
67
import org.junit.Ignore;
@@ -314,8 +315,8 @@ public void testDragFillSingleRow()
314315
// Get the various row heights before adding a value to the multiLine field.
315316
WebElement gridRow = Locator.tag("tr").findElements(testGrid).get(1);
316317
int rowHeightBefore = gridRow.getSize().height;
317-
var totalHeightBefore = new Object(){int size = 0; };
318-
Locator.tag("tr").findElements(testGrid).forEach(gr -> totalHeightBefore.size = totalHeightBefore.size + gr.getSize().height);
318+
var totalHeightBefore = new MutableInt(0);
319+
Locator.tag("tr").findElements(testGrid).forEach(gr -> totalHeightBefore.add(gr.getSize().height));
319320
WebElement topLeft = testGrid.setCellValue(0, FILL_STRING, stringValue);
320321

321322
testGrid.setCellValue(0, FILL_INT, intValue);
@@ -351,12 +352,12 @@ public void testDragFillSingleRow()
351352
testGrid.getColumnData(FILL_DATE));
352353

353354
// Check that pasting increased the size of all the rows.
354-
var totalHeightAfter = new Object(){int size = 0; };
355-
Locator.tag("tr").findElements(testGrid).forEach(gr -> totalHeightAfter.size = totalHeightAfter.size + gr.getSize().height);
355+
var totalHeightAfter = new MutableInt(0);
356+
Locator.tag("tr").findElements(testGrid).forEach(gr -> totalHeightAfter.add(gr.getSize().height));
356357

357358
checker().withScreenshot()
358359
.verifyTrue("The total height of all the rows should have increases after the paste.",
359-
totalHeightBefore.size + (3 * rowHeightBefore) >= totalHeightAfter.size);
360+
totalHeightBefore.intValue() + (3 * rowHeightBefore) >= totalHeightAfter.intValue());
360361
}
361362

362363
@Test

0 commit comments

Comments
 (0)