Hi.
>> What do you guys think?
>
> Sounds good to me.
>
> It would be nice if the comparison would be somewhat smart, so that it
> won't flag false positives on "pointless" changes, like the order of
> import, indentation, etc, but I guess doing something like that would be
> difficult.
I don't think we really need that smartness. I think code model must be
just made deterministic.
* Since code model is configured within the test, indentation is
predictable/deterministic. Well, if the way classes are indented
changes, test target sources must be updated, but I don't think it's
that huge problem.
* If order of imports is non-deterministic (HashMaps and so on), it
should be made so.
I'm really sure making code model deterministic makes a lot of sence.
Tests and SVN diffs will profit from it. And it's not hard. Let's just
watch our own tests and user reports. If an indeterministic behaviour is
noticed/reported, we'll just fix it.
Bye.
/lexi