RFR: Expect carriage return to be part of lines when testing on Windows.

Jorn Vernee via github.com duke at openjdk.java.net
Fri Aug 23 08:47:20 UTC 2019


One more patch to address failing `vcs` tests on Windows.

The tests in `RepositoryTest` use `Files::write` to create some test files, but this also means the files will have Windows line endings (i.e. CRLF). When these are parsed by the skara tooling the LF part is dropped as a delimiter, but the CR part ends up being part of the line. This is all correct and as expected, since patch files are supposed to have unix line endings, so the carriage return ends up as part of the content.

The problem comes when testing what is written to the file. The tests are not expecting the Windows line endings to be there, so a lot of them fail.

This PR changes the tests to expect the Windows line endings as well. Or rather, it tries to stick what's in the system line separator (minus a trailing '\n', since that is dropped as a delimiter) at the end of the checked line.

With this patch, I have all the `vcs` tests passing locally.

----------------

Commits:
 - eec50c80:	Expect carriage return to be part of lines when testing on Windows.

Pull request:
https://git.openjdk.java.net/skara/pull/74

Webrev:
https://webrevs.openjdk.java.net/skara/74/webrev.00

Patch:
https://git.openjdk.java.net/skara/pull/74.diff

Fetch command:
git fetch https://git.openjdk.java.net/skara pull/74/head:pull/74


More information about the skara-dev mailing list