Thanks for looking into this.

A couple of questions/comments before I give the LGTM..


http://gwt-code-reviews.appspot.com/1224801/diff/1/2
File user/src/com/google/gwt/junit/client/GWTTestCase.java (right):

http://gwt-code-reviews.appspot.com/1224801/diff/1/2#newcode445
user/src/com/google/gwt/junit/client/GWTTestCase.java:445: // Check to
see if gwt.devjar is null. If so, notify the
Is this system property required?

http://gwt-code-reviews.appspot.com/1224801/diff/1/2#newcode448
user/src/com/google/gwt/junit/client/GWTTestCase.java:448:
testResult.addError(this, new Throwable("No GWT devjar property found.
Did you mean to run as a GWT JUnit Test instead of a JUnit Test?"));
Suggestion:

"No GWT devjar property found. Did you mean to run as a GWT JUnit Test
instead of a JUnit Test?" -->

"The gwt.devjar property has not been set. If you are using the Google
Plugin for Eclipse, you should be running this test as a 'GWT JUnit
Test', not a plain 'JUnit Test'."

http://gwt-code-reviews.appspot.com/1224801/show

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to