Bug #15203
closedtest-all WEBrick - fixes needed for Azure Pipelines - three asserts, line endings
Description
Attached is a patch with fixes needed for the test-all WEBrick tests when run on Azure pipelines (vc12, vc14, mingw). These have not been needed for Appveyor builds, nor for local testing on mingw builds.
The changes involve line endings in three asserts.
Given that these haven't appeared before, I not sure what or where the fixes should be...
Thanks, Greg
Files
Updated by naruse (Yui NARUSE) almost 6 years ago
- Status changed from Open to Closed
Applied in changeset trunk|r64924.
Use binread because webrick also uses binread
Reported-by: MSP-Greg [Bug #15203]
Updated by naruse (Yui NARUSE) almost 6 years ago
I think it's because git on Azure Pipeline enables "Use CRLF" option.
Because of that those test files uses CRLF on the environment.
WEBrick reads file on binary mode, therefore responses are also CRLF text.
But the expect
data in those tests uses text mode read; this is the cause of this issue.
I fixed this by using binread on reading test expect data.