So you're saying that we should not try to make this OS specific?<br><br>In the end, this is a test, not running code, which does work fine.<br><br>~Lincoln<br><br><div class="gmail_quote">On Tue, Mar 13, 2012 at 7:20 PM, Ivan St. Ivanov <span dir="ltr"><<a href="mailto:ivan.st.ivanov@gmail.com">ivan.st.ivanov@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks,<div><br></div><div>I was going to create a JIRA issue for this, but I didn't find where to click (probably I don't have permissions). Anyway... When I run the unit tests on Windows, I keep getting an error at JavaClassTest.testFormattingIsNotChanged. The issue is at this assertion:</div>
<div><br></div><div><div> assertTrue(output.contains(</div><div> "Telephone" + System.getProperty("line.separator") +</div><div> "{"</div><div> ));</div>
</div><div><br></div><div>On windows the line.separator system property resolves to \r\n. However, the output variable contains actually Telephone\n{. Which means that a \r is missing.</div><div><br></div><div>I hoped I can resolve this by myself. However, the output variable is initialized by the JavaClass's toString() method. Which is actually implemented in the AbstractJavaSource class. And there it uses the Document and TextEdit classes that come from org.eclipse.jface.text and org.eclipse.text.edits packages respectively. Which is outside Forge.</div>
<div><br></div><div>Does anybody have an idea how can we solve this? Of course we can change the test. But I'm afraid that the Forge-generated entities look really ugly on my machine. And I'm afraid that the reason is in this Eclipse library.</div>
<div><br></div><div>Cheers,</div><div>Ivan</div><div><br></div><div>P.S. Just for the record, I'm running a 64-bit Windows 7.</div>
<br>_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>"Simpler is better."<br>