From 105f122c89b0e287c19de28a8654092a1564c0b4 Mon Sep 17 00:00:00 2001 From: labkey-jeckels Date: Fri, 22 Aug 2025 16:41:21 -0700 Subject: [PATCH 1/2] Issue 53672: List export / import roundtrip issue with long key name --- .../test/tests/list/ListArchiveExportTest.java | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/org/labkey/test/tests/list/ListArchiveExportTest.java b/src/org/labkey/test/tests/list/ListArchiveExportTest.java index 099af86d4b..fd394cbdbd 100644 --- a/src/org/labkey/test/tests/list/ListArchiveExportTest.java +++ b/src/org/labkey/test/tests/list/ListArchiveExportTest.java @@ -11,6 +11,7 @@ import org.labkey.test.categories.Daily; import org.labkey.test.categories.Hosting; import org.labkey.test.components.list.ManageListsGrid; +import org.labkey.test.pages.list.BeginPage; import org.labkey.test.params.FieldDefinition; import org.labkey.test.params.list.IntListDefinition; import org.labkey.test.util.ApiPermissionsHelper; @@ -71,10 +72,11 @@ private void doSetUp() throws IOException, CommandException _permissionHelper.addMemberToRole(_listUser, "Reader", PermissionsHelper.MemberType.user); } - private void createListWithData(String name, Map rowData) throws IOException, CommandException + private void createListWithData(String name, Map rowData) throws IOException, CommandException { var connection = createDefaultConnection(); - var listDef = new IntListDefinition(name, "RowId").setFields(List.of( + // Issue 53672: Use a long key column name + var listDef = new IntListDefinition(name, "keya123456789a123456789a123456789a123456789a123456789a123456789").setFields(List.of( new FieldDefinition("Shape", FieldDefinition.ColumnType.String), new FieldDefinition("Count", FieldDefinition.ColumnType.Integer))); var dataGenerator = listDef.create(connection, getCurrentContainerPath()); @@ -118,6 +120,14 @@ public void testExportListArchive() listsGrid.checkAllOnPage(); File listExport = listsGrid.exportSelectedLists(); Assert.assertTrue("Empty export file downloaded", listExport.length() > 0); + + // Issue 53672: Delete the lists and reimport, ensuring we have no errors + listsGrid.deleteSelectedLists(); + listsGrid = goToManageLists().getGrid(); + assertTextNotPresent(LIST_A, LIST_B); + listsGrid.clickImportArchive().setZipFile(listExport).clickImport(); + goToManageLists().getGrid(); + assertTextPresent(LIST_A, LIST_B); } @Override From 444fbcbfe27003f8f0b986cbf665e9eb3b1e26d2 Mon Sep 17 00:00:00 2001 From: labkey-jeckels Date: Mon, 25 Aug 2025 12:55:18 -0700 Subject: [PATCH 2/2] Fix SpecimenCustomizeTest, test mixed casing --- src/org/labkey/test/tests/list/ListArchiveExportTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/org/labkey/test/tests/list/ListArchiveExportTest.java b/src/org/labkey/test/tests/list/ListArchiveExportTest.java index fd394cbdbd..72a7be394a 100644 --- a/src/org/labkey/test/tests/list/ListArchiveExportTest.java +++ b/src/org/labkey/test/tests/list/ListArchiveExportTest.java @@ -76,7 +76,7 @@ private void createListWithData(String name, Map rowData) throws { var connection = createDefaultConnection(); // Issue 53672: Use a long key column name - var listDef = new IntListDefinition(name, "keya123456789a123456789a123456789a123456789a123456789a123456789").setFields(List.of( + var listDef = new IntListDefinition(name, "keyA123456789a123456789A123456789a123456789a123456789a123456789").setFields(List.of( new FieldDefinition("Shape", FieldDefinition.ColumnType.String), new FieldDefinition("Count", FieldDefinition.ColumnType.Integer))); var dataGenerator = listDef.create(connection, getCurrentContainerPath());