Hi,
Anyone?
To clarify, it's not my test that is not running. I didn't manage to run any
tests.
Is it possible, through ant, to execute a single test suite ?
How do you launch your tests ? are you doing the complete ant test each
time ?
Thanks,
Antoine.
2011/3/8 Antoine DESSAIGNE
Some tests require that the sources (gwt-dev/src, gwt-dev/test,
gwt-dev/super, gwt-user/src, etc.) be on the classpath.
Alternatively, you can indeed run only a few tests with ant, setting the
gwt.junit.testcase.includes property (it defaults to **/*Suite.class),
something like (from the user
Thinking a bit more about it, this fix is IMO not enough: it will only
resolve properties that have not yet been resolved, which means in the
case of two methods returning the same object, if the second method
modifies some properties of the object before returning it and those
properties have
Thanks a lot !
Now it's working fine :)
2011/3/10 Thomas Broyer t.bro...@gmail.com
Some tests require that the sources (gwt-dev/src, gwt-dev/test,
gwt-dev/super, gwt-user/src, etc.) be on the classpath.
Alternatively, you can indeed run only a few tests with ant, setting the
http://gwt-code-reviews.appspot.com/1369809/diff/1/user/src/com/google/gwt/touch/client/TouchScroller.java
File user/src/com/google/gwt/touch/client/TouchScroller.java (right):
http://gwt-code-reviews.appspot.com/1369809/diff/1/user/src/com/google/gwt/touch/client/TouchScroller.java#newcode173
LGTM. With a nit.
http://gwt-code-reviews.appspot.com/1369810/diff/1/user/src/com/google/gwt/validation/client/impl/NodeImpl.java
File user/src/com/google/gwt/validation/client/impl/NodeImpl.java
(right):
How about http://gwt-code-reviews.appspot.com/1374804 instead to allow
developer control over the classloader instead?
--
Bob Vawter
Google Web Toolkit Team
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Opps. Fixed.
http://gwt-code-reviews.appspot.com/1369810/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Revision: 9835
Author: fabio...@google.com
Date: Thu Mar 10 03:46:43 2011
Log: add ie9 user.agent with fallback binding to ie8. (in preparation for
ie9 support).
Review at http://gwt-code-reviews.appspot.com/1369808
http://code.google.com/p/google-web-toolkit/source/detail?r=9835
Modified:
Revision: 9836
Author: b...@google.com
Date: Thu Mar 10 05:45:01 2011
Log: Extract ServiceLayer.getDomainClassLoader() to ensure consistent use
of ClassLoaders within the service layer.
Issue 6092.
Review at http://gwt-code-reviews.appspot.com/1374804
Patch by: bobv
Review by: t.broyer
Revision: 9837
Author: ncha...@google.com
Date: Thu Mar 10 05:54:28 2011
Log: Support paths for iterables without a key or an index.
[JSR 303 TCK Result] 116 of 257 (45.14%) Pass with 14 Failures and 9 Errors.
Review at http://gwt-code-reviews.appspot.com/1369810
Review by: rchan...@google.com
http://gwt-code-reviews.appspot.com/1369809/diff/1/user/src/com/google/gwt/touch/client/TouchScroller.java
File user/src/com/google/gwt/touch/client/TouchScroller.java (right):
http://gwt-code-reviews.appspot.com/1369809/diff/1/user/src/com/google/gwt/touch/client/TouchScroller.java#newcode162
I've taken out the hacks, and backfilled missing logic for processMetas
and modified computeScriptBase(). It turns out the old linker was
stomping 'base' with the original url to the gadget spec passed to the
url query string
This updated patch now works with the sample apps in the gwt-gadgets
LGTM, committing with minor cleanups.
http://gwt-code-reviews.appspot.com/1364801/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Revision: 9838
Author: t.bro...@gmail.com
Date: Thu Mar 10 07:36:40 2011
Log: Address issue 6076 where composite paths were not recorded correctly.
http://gwt-code-reviews.appspot.com/1364801/
Patch by: t.broyer
Review by: bobv
http://code.google.com/p/google-web-toolkit/source/detail?r=9838
Revision: 9839
Author: t.bro...@gmail.com
Date: Thu Mar 10 07:55:44 2011
Log: Address issue 6066 where RequestFactoryEditorDriver doesn't collect
composite paths.
http://gwt-code-reviews.appspot.com/1363801
Patch by: t.broyer
Review by: bobv
Revision: 9840
Author: rj...@google.com
Date: Thu Mar 10 09:08:32 2011
Log: Copy/paste detected in NumberLabelParser :-)
It leaks DateTimeFormat from DateLabelParser.
I've added couple statements into test and fixed problem.
Submitting for scheg...@google.com
Review at
EditorTest refers to a SimpleFooBarOnlyEditor. Which patch defines that
type?
http://gwt-code-reviews.appspot.com/1371801/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1370808/diff/1010/gadgets/src/com/google/gwt/gadgets/linker/computeGadgetsBase.js
File gadgets/src/com/google/gwt/gadgets/linker/computeGadgetsBase.js
(right):
On 2011/03/10 20:49:41, bobv wrote:
EditorTest refers to a SimpleFooBarOnlyEditor. Which patch defines
that type?
It was there before r9839
http://code.google.com/p/google-web-toolkit/source/detail?r=9839
See also my mail Re: Severe regressions with editor overhaul at r9666
from Mon, Feb 28,
It was there before r9839
http://code.google.com/p/google-web-toolkit/source/detail?r=9839
That type had been renamed in one of the other patches. I'm
submitting the patches out-of-order relative to how they were written,
so I'll just fix it up.
--
Bob Vawter
Google Web Toolkit Team
--
Reviewers: jlabanca, rjrjr, jat,
Description:
IE9 support (BETA).
known issues:
IE9 devmode throws non fatal js exception.
Please review this at http://gwt-code-reviews.appspot.com/1383801/
Affected files:
M dev/core/src/com/google/gwt/dev/shell/StandardRebindOracle.java
M
John L will need to give the real review, but one nit:
http://gwt-code-reviews.appspot.com/1383801/diff/1/dev/core/src/com/google/gwt/dev/shell/StandardRebindOracle.java
File dev/core/src/com/google/gwt/dev/shell/StandardRebindOracle.java
(right):
It seems the issue you are raising is to give a better warning message. I
will file an issue and address this next week.
On Mar 10, 2011 5:40 PM, rj...@google.com wrote:
John L will need to give the real review, but one nit:
http://gwt-code-reviews.appspot.com/1370808/diff/1010/gadgets/src/com/google/gwt/gadgets/linker/installGadgetsScript.js
File gadgets/src/com/google/gwt/gadgets/linker/installGadgetsScript.js
(right):
Reviewers: rjrjr,
Description:
Clean up pesky UiBinder warnings
Please review this at http://gwt-code-reviews.appspot.com/1380802/
Affected files:
M user/src/com/google/gwt/uibinder/elementparsers/InterpreterPipe.java
M user/src/com/google/gwt/uibinder/elementparsers/UiChildParser.java
http://gwt-code-reviews.appspot.com/1355802/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
I realized the code generator didn't support the new KeyGenerator
annotation, and some things were in the rebind package that should have
been in server.
http://gwt-code-reviews.appspot.com/1355802/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
28 matches
Mail list logo