Skip to content

Use OS-specific file separator for expected file paths#2536

Merged
labkey-tchad merged 3 commits intorelease25.7-SNAPSHOTfrom
25.7_fb_fileSeparatorFix
Jul 3, 2025
Merged

Use OS-specific file separator for expected file paths#2536
labkey-tchad merged 3 commits intorelease25.7-SNAPSHOTfrom
25.7_fb_fileSeparatorFix

Conversation

@labkey-tchad
Copy link
Member

Rationale

The files in this grid use OS file separators. First time running this test on Windows revealed the test needs to account for that.

Values in the 'File Upload Test' column not as expected. expected:<[ sampletype/csv_sample.csv,  sampletype/sample.txt,  sampletype/xls_sample.xls,  sampletype/tsv_sample.tsv,  sampletype/targz_sample.tar.gz]> but was:<[ sampletype\csv_sample.csv,  sampletype\sample.txt,  sampletype\xls_sample.xls,  sampletype\tsv_sample.tsv,  sampletype\targz_sample.tar.gz]>

Related Pull Requests

Changes

  • Use OS-specific file separator for expected file paths

@labkey-tchad labkey-tchad requested review from a team and labkey-chrisj and removed request for a team July 3, 2025 16:52
@labkey-tchad
Copy link
Member Author

Verifying on TeamCity

@labkey-tchad labkey-tchad merged commit 39f09ec into release25.7-SNAPSHOT Jul 3, 2025
6 checks passed
@labkey-tchad labkey-tchad deleted the 25.7_fb_fileSeparatorFix branch July 3, 2025 21:29
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