Skip to content

GitHub Issue #159: AssayExportImportTest fix to use assay designer UI to add transform script - #3139

Open
cnathe wants to merge 1 commit into
release26.8-SNAPSHOTfrom
26.8_fb_assayExportTest159
Open

GitHub Issue #159: AssayExportImportTest fix to use assay designer UI to add transform script#3139
cnathe wants to merge 1 commit into
release26.8-SNAPSHOTfrom
26.8_fb_assayExportTest159

Conversation

@cnathe

@cnathe cnathe commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Rationale

The changes for this issue https://github.com/LabKey/internal-issues/issues/159 require that assay transform scripts live in the LabKey container's @scripts directory. This PR fixes the AssayExportImportTest to upload the script to the expected location using the assay designer UI.

Related Pull Requests

Changes

  • AssayExportImportTest to upload transform script to @script dir using assay designer UI

@cnathe cnathe self-assigned this Aug 3, 2026
@cnathe
cnathe requested a review from labkey-tchad August 3, 2026 21:05

@labkey-tchad labkey-tchad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does the Java remote API need an update for this?

@cnathe

cnathe commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

Does the Java remote API need an update for this?

I don't think so. The SaveProtocolCommand in the java API could still use the setProtocolTransformScripts helper if it was providing a file path within the container's @scripts. So it should still work under those conditions. However, any usages that were just referring to a different path will stop working (this is the only usage I found in the tests).

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.

2 participants