scalatest/scalatest-eclipse-plugin — pull request #11 by rlegendi (2012-08-03 07:09:20+00:00)
Title: Improved test failure display on the ScalaTest View

Hi all!

Hope this is the proper repo to submit any pull requests :-) I'm working on the [Specs2 integration through ScalaTest plugin](http://rlegendi.github.com/specs2-runner/) and got a feedback about [a minor issue](https://github.com/rlegendi/specs2-runner/issues/2). I thought the best approach would be to extend the ScalaTest plugin so everyone could benefit from the patch.

The issue is that if a test if failed, its reason is not reported in any way at the ScalaTest view.

The default Eclipse JUnit runner view for instance, looks like this:

![How JUnit View works](https://raw.github.com/rlegendi/specs2-runner/master/doc/issues/002/junit.png)

Now, the ScalaTest view had nothing similar:

![How the ScalaTest view worked so far](https://raw.github.com/rlegendi/specs2-runner/master/doc/issues/002/stacktrace.png)

So in this pull request I extended the view with the proper error message display that caused the test to fail:

![My new cool freature](https://raw.github.com/rlegendi/specs2-runner/master/doc/issues/002/ok_stacktrace.png)

I would also welcome any code review, because after many modifications I still feel the addition a bit clumsy [here](https://github.com/rlegendi/scalatest/pull/new/scalatest:master...improvedFailureDisplay#L2R86) and [here](https://github.com/rlegendi/scalatest/pull/new/scalatest:master...improvedFailureDisplay#L2R157). Thanks in advance!

I have also found an absolute library reference in the project classpath entries, changed it to relative (so after checking out the project it has no errors hopefully).

Thanks in advance!