On Thu, Oct 23, 2008 at 9:44 AM, Eric Ayers <[EMAIL PROTECTED]> wrote:
> > > On Thu, Oct 23, 2008 at 9:21 AM, Miguel Méndez <[EMAIL PROTECTED]> wrote: > >> On Thu, Oct 23, 2008 at 8:42 AM, Eric Ayers <[EMAIL PROTECTED]> wrote: >> >>> Hi Miguel, >>> >>> Could you please review this patch? It is very small! >>> >>> I am currently running into issues with maps regression tests failing due >>> to some problems with the Maps JavaScript API. The Maps JavaScript API can >>> be configured to return the "current" version, either stable or >>> experimental. When you run into problems, it is helpful to know what >>> version your app is actually running so you can research the problem or >>> write a good bug report. This change adds Maps.getVersion() to return the >>> current Maps JavaScript API version and adds a unit test case for it. >>> >>> Also, when I sorted MapWidgetTest, a method moved. >>> >>> M maps/src/com/google/gwt/maps/client/Maps.java >> >> LG - The javadoc should probably be changed since this method will return >> the JS Maps API version even in cases where "current" scheme is not used. >> >> >>> >>> M maps/test/com/google/gwt/maps/client/MapWidgetTest.java >> >> LG with one nit: Ideally tests will not output to std:out unless there is >> an error or failure. So, consider removing the System.out.println during >> the test. >> > > Do you have an alternate suggestion for getting the version of the Maps API > from running the hosted mode tests? > > Seems like you would want to include it in the failure capture information of the assertions (exceptions thrown). > > > -- > Eric Z. Ayers - GWT Team - Atlanta, GA USA > http://code.google.com/webtoolkit/ > -- Miguel --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---