Skip to content

Conversation

@labkey-tchad
Copy link
Member

Rationale

This has been failing intermittently on TeamCity. If the randomly generated measure field name ends with a ':' the data does not import.

java.lang.AssertionError: Expect graphical measures to be as imported for Decimal: Meas.ure+@}👾],^안< ;]И%и=안};?$/&}~,. %()=+-[] |*`'":;\<>?!@#^и И안はΠ⋯∑(|-],`/?~`(は$/&}~,. %()=+-[] |*`'":;\<>?!@#^и И안はΠ⋯∑. '?}%'@?: expected:<[0.1, 0.11, 0.12, 0.13, , 0.21, 0.22, 0.23, 0.3, 0.31, 0.32, 0.33, 0.1, 0.11, 0.12, 0.13, , 0.21, 0.22, 0.23, 0.3, 0.31, 0.32, 0.33]> but was:<[, , , , , , , , , , , , , , , , , , , , , , , ]>
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:120)
        at org.labkey.test.util.DeferredErrorCollector.lambda$5(DeferredErrorCollector.java:248)
        at org.labkey.test.util.DeferredErrorCollector.wrapAssertion(DeferredErrorCollector.java:188)
        at org.labkey.test.util.DeferredErrorCollector.verifyEquals(DeferredErrorCollector.java:248)
        at org.labkey.test.tests.biologics.plates.PlateAssayImportTest.testVerifyGraphicalVsTabularImport(PlateAssayImportTest.java:197)

Related Pull Requests

Changes

  • Handle multiple special chars in plate based assay measure field

Copy link
Contributor

@cnathe cnathe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change looks good. I also pulled down the change locally and tested it via the PlateAssayImportTest.testVerifyGraphicalVsTabularImport test with a measure name that ends in a colon.

@labkey-tchad labkey-tchad merged commit 60d028a into release25.11-SNAPSHOT Dec 24, 2025
7 checks passed
@labkey-tchad labkey-tchad deleted the 25.11_fb_plateImportParsing branch December 24, 2025 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants