Well, in this case I think that's acceptable because this test is not really intended to test brace positioning. This is to test generation functionality.

On Tue, May 22, 2012 at 7:26 PM, Thomas Frühbeck <fruehbeck@aon.at> wrote:
as Lincoln already mentioned the problem is mainly a formating issue.
There are to sides to the problem:
  - pure indentation changes: \t to "  "
  - change in positioning of braces: Java style to C# style
The Code Conventions for Java Programming Language is very clear about
that, so this is kinda question of taste.

One possibility would be to normalize all whitespace like
        content.replaceAll("(\r|\n|\\s)+"," ")
but then the idea of the test would become somewhat absurd, because this
is, what the test should test.
And then it would _not_ solve the braces positioning issue :-/

Thomas
_______________________________________________
forge-dev mailing list
forge-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/forge-dev



--
Lincoln Baxter, III
http://ocpsoft.org
"Simpler is better."