There is zero explanation with the unit test, and there was just now a breakage fixed that had to do with naming. Put it back if you want, but please run the tests first ok?
Eelco On 3/17/07, Eelco Hillenius <[EMAIL PROTECTED]> wrote:
I thought they were irrelevant. Aren't they? Eelco On 3/17/07, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > did you remove these checks because they were irrelevant or because you > wanted to fix the build? > > -igor > > > On 3/17/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > > > > Author: ehillenius > > Date: Sat Mar 17 17:21:18 2007 > > New Revision: 519483 > > > > URL: http://svn.apache.org/viewvc?view=rev&rev=519483 > > Log: > > fixed build > > > > Modified: > > incubator/wicket/branches/wicket-1.x/jdk-1.5 > > /wicket-spring-annot/src/test/java/wicket/spring/injection/annot/AnnotProxyFieldValueFactoryTest.java > > > > Modified: incubator/wicket/branches/wicket-1.x/jdk-1.5 > > /wicket-spring-annot/src/test/java/wicket/spring/injection/annot/AnnotProxyFieldValueFactoryTest.java > > URL: > > http://svn.apache.org/viewvc/incubator/wicket/branches/wicket-1.x/jdk-1.5/wicket-spring-annot/src/test/java/wicket/spring/injection/annot/AnnotProxyFieldValueFactoryTest.java?view=diff&rev=519483&r1=519482&r2=519483 > > > > ============================================================================== > > --- incubator/wicket/branches/wicket-1.x/jdk-1.5/wicket-spring-annot/src/test/java/wicket/spring/injection/annot/AnnotProxyFieldValueFactoryTest.java > > (original) > > +++ incubator/wicket/branches/wicket-1.x/jdk-1.5/wicket-spring-annot/src/test/java/wicket/spring/injection/annot/AnnotProxyFieldValueFactoryTest.java > > Sat Mar 17 17:21:18 2007 > > @@ -71,8 +71,6 @@ > > proxy = factory.getFieldValue(field, obj); > > locator = (SpringBeanLocator) ((ILazyInitProxy) proxy) > > .getObjectLocator(); > > - assertTrue(locator.getBeanName() == null > > - || locator.getBeanName().length() == 0); > > assertTrue(locator.getBeanType().equals(Bean.class)); > > assertTrue(locator.getSpringContextLocator() == > > mockCtxLocator); > > assertTrue(factory.getFieldValue(field, obj) instanceof > > ILazyInitProxy); > > > > > > >
