Skip to content

Commit c56d27f

Browse files
committed
Issue 54062: Strip folder name from displayed name a file field
1 parent d91e464 commit c56d27f

File tree

4 files changed

+9
-12
lines changed

4 files changed

+9
-12
lines changed

src/org/labkey/test/tests/AttachmentFieldTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,9 @@ public void testFileFieldInSampleType()
9191

9292
clickAndWait(Locator.tagWithText("a", "S1"));
9393
clickAndWait(Locator.tagWithClass("a", "labkey-text-link").withText("edit"));
94-
waitForElement(Locator.tagContainingText("div", "sampletype/jpg_sample.jpg"));
94+
waitForElement(Locator.tagContainingText("div", "jpg_sample.jpg"));
9595
// Issue 53200: Update form incorrectly shows that a file is not available
96-
assertTextNotPresent("sampletype/jpg_sample.jpg (unavailable)");
96+
assertTextNotPresent("jpg_sample.jpg (unavailable)");
9797
clickButton("Cancel");
9898

9999
log("Verifying view in browser works");

src/org/labkey/test/tests/FileAttachmentColumnTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -613,7 +613,7 @@ private void validateAssayRun(String assayName, String folderPath, String runNam
613613
checker().withScreenshot("unexpected_run_file_links")
614614
.wrapAssertion(()-> Assertions.assertThat(runFileTexts.stream().map(String::trim).toList())
615615
.as("expect complete run files")
616-
.containsOnly(String.format("assaydata%s%s", File.separatorChar, runFile.getName()))
616+
.containsOnly(runFile.getName())
617617
.hasSize(5));
618618
}
619619

@@ -637,7 +637,7 @@ private void validateDatasetData(String datasetName, String folderPath, List<Fil
637637
}
638638
else
639639
{
640-
Optional<WebElement> optionalFileLink = Locator.linkContainingText(String.format("datasetdata%s%s", File.separatorChar, file.getName()))
640+
Optional<WebElement> optionalFileLink = Locator.linkContainingText(file.getName())
641641
.findOptionalElement(dataRegionTable);
642642
checker().withScreenshot("unexpected_file_state")
643643
.awaiting(Duration.ofSeconds(2),

src/org/labkey/test/tests/InlineImagesAssayTest.java

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -182,10 +182,7 @@ public final void testAssayInlineImages() throws Exception
182182
customizeView.addColumn(FieldKey.fromParts("Run", "RowId"));
183183
customizeView.addColumn(FieldKey.fromParts("Run", "Protocol", "RowId"));
184184
customizeView.applyCustomView();
185-
var protocolId = list.getDataAsText(0, "Run/Protocol/RowId");
186-
var runId = list.getDataAsText(0, "Run/RowId");
187-
String helpJpgFilePath = String.format("AssayId_%s%sRunId_%s%s%s", protocolId, File.separatorChar,
188-
runId, File.separatorChar, HELP_JPG_FILE.getName());
185+
String helpJpgFilePath = HELP_JPG_FILE.getName();
189186

190187
log("Validate that two links to this image file are now present.");
191188
assertElementPresent("Did not find the expected number of icons for images for " + PNG01_FILE.getName() + " from the runs.", Locator.xpath("//img[contains(@title, '" + PNG01_FILE.getName() + "')]"), 3);
@@ -206,8 +203,8 @@ public final void testAssayInlineImages() throws Exception
206203

207204
log("Validate that the 'File' (last) column is as expected.");
208205
assertEquals("Values in 'File' column not exported as expected [" + exportedFile.getName() + "]",
209-
Arrays.asList("Batch File Field", "assaydata" + File.separator + XLS_FILE.getName(), "assaydata" + File.separator + XLS_FILE.getName(), "assaydata" + File.separator + XLS_FILE.getName()),
210-
ExcelHelper.getColumnData(workbook.getSheetAt(workbook.getActiveSheetIndex()), 7));
206+
Arrays.asList("Batch File Field", XLS_FILE.getName(), XLS_FILE.getName(), XLS_FILE.getName()),
207+
ExcelHelper.getColumnData(workbook.getSheetAt(workbook.getActiveSheetIndex()), 7));//
211208
}
212209

213210
log("Remove the 'File' (last) column from the batch and see that things still work.");
@@ -286,7 +283,7 @@ private void validateExcelExport(File exportedFile, Workbook workbook, String he
286283

287284
exportedColumn = ExcelHelper.getColumnData(sheet, 5);
288285
assertEquals("Values in 'File' column not exported as expected [" + exportedFile.getName() + "]",
289-
Arrays.asList("Run File Field", "assaydata" + File.separator + PNG01_FILE.getName(), "assaydata" + File.separator + PNG01_FILE.getName(), "assaydata" + File.separator + PNG01_FILE.getName()),
286+
Arrays.asList("Run File Field", PNG01_FILE.getName(), PNG01_FILE.getName(), PNG01_FILE.getName()),
290287
exportedColumn);
291288

292289
}

src/org/labkey/test/tests/core/admin/AllowedFileExtensionTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -383,7 +383,7 @@ public void testAllowedFileExtensionsInSampleType()
383383
{
384384
fieldMap = Map.of("Name", String.format("S-%d", i), stFileField, fileMap.get(allowedType).getAbsolutePath());
385385
sampleTypeHelper.insertRow(fieldMap);
386-
expectedValues.add(String.format(" sampletype%s%s", File.separator, fileMap.get(allowedType).getName()));
386+
expectedValues.add(String.format(" %s", fileMap.get(allowedType).getName()));
387387
i++;
388388
}
389389

0 commit comments

Comments
 (0)