Hi No, somebody put SomeInnerClass in container/openejb-core/src/test/java/org/apache/openejb/OpenEjbContainerTest.java as a final class then openejb tries to do what is needed to make this class injectable and it fails.
It is linked to my changes but not fine. So either we change OpenEjbContainer$Provider to tcheck runners before adding them as ManagedBean (container/openejb-core/src/main/java/org/apache/openejb/OpenEjbContainer.java) or we just remove the final. I'm for 1) (check). The check is basically if (notabstract and notfinal) then addManagedBean. We can go further in checks but think it is not needed now. Romain Manni-Bucau Twitter: @rmannibucau Blog: http://rmannibucau.wordpress.com/ LinkedIn: http://fr.linkedin.com/in/rmannibucau Github: https://github.com/rmannibucau 2014-05-14 5:49 GMT+02:00 Jean-Louis Monteiro <[email protected]>: > Hi, > > I think with Romain's patch today, it's fixed, isn't it? > > JLouis > > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Tue, May 13, 2014 at 3:35 PM, Thiago Veronezi <[email protected]>wrote: > >> Hi guys, >> >> I didn't manage to trace back what breaks the "OpenEjbContainerTest" test >> yet. >> >> If we change... >> >> public static final class SomeInnerClass { >> >> to... >> >> public static class SomeInnerClass { >> >> it works fine again. Please note that the last time someone touched the >> line above was back in 2013. The issue is somewhere else. >> >> Does it ring a bell for anyone? >> >> []s, >> Thiago. >>
