@@ -72,10 +72,14 @@ public class PlainResultFormatterTest { Issue issue = new IssueBuilder(file.toString(), 1, 1, "fatality").build(); issues.add(issue); runFormatter(file); - String expected = "foo/bar.js:1:1: fatality\n"; + // Build up the expected output in a cross-platform manner. + String nl = System.getProperty("line.separator"); + StringBuilder sb = new StringBuilder(file.toString()); + sb.append(":1:1: fatality"); + sb.append(nl); // NB: We use platform encoding here, as that's what we expect the // formatter to produce. - assertThat(out.toString(), is(expected)); + assertThat(out.toString(), is(sb.toString())); } } jslint4java-ant/src/test/java/com/googlecode/jslint4java/ant/PlainResultFormatterTest.java 399d9cfb93942f182ab1056bb35ea7ef9bdce62c Dominic Mitchell dom@happygiraffe.net http://github.com/happygiraffe/jslint4java/commit/790fba106db19181519664202829be30e91351bd 790fba106db19181519664202829be30e91351bd 2009-11-05T08:58:42-08:00 2009-11-05T08:58:42-08:00 issue 32: more cross-platform test fixes. I can't believe that I made this mistake directly below a test where I'd previously corrected it… 1b3ebdf296e4cb70583e5c5e5968cbb2291379e4 Dominic Mitchell dom@happygiraffe.net