Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

NEW_LINE bug in MZTabConstants #10

Closed
UKQIDA opened this issue Apr 24, 2017 · 1 comment
Closed

NEW_LINE bug in MZTabConstants #10

UKQIDA opened this issue Apr 24, 2017 · 1 comment
Assignees

Comments

@UKQIDA
Copy link

UKQIDA commented Apr 24, 2017

The "NEW_LINE" is defined in MZTabConstants.java as System.getProperty("line.separator").
When running MTDlineParserTest.java file on Windows system, if fails at line 211: assertEquals(msRun1.toString(), "MTD\tms_run[1]-location\tnull\n").
The reason is that System.getProperty("line.separator") on Windows is "\r\n". So the return value from msRun1.toString() is "MTD\tms_run[1]-location\tnull\r\n".

Change line 211 to assertEquals(msRun1.toString(), "MTD\tms_run[1]-location\tnull"+MZTabConstants.NEW_LINE); will fix the test file.

@nilshoffmann
Copy link
Collaborator

Fixed in #12

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants