tomee-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jean-Louis MONTEIRO <jeano...@gmail.com>
Subject Re: Cobertura error
Date Mon, 08 Oct 2012 10:15:43 GMT
That's against the spec, so ignoring it by default even if the interface is
empty does not seem relevant IMO.
That's why I proposed to ignore it on OpenEJB instead of having to specify
the local interface for each bean class even if there is only one interface
in the application (IE. only for cobertura with tests).

Adding a property to set in maven surefire plugin is simple and eases end
user life.
Any other thoughts?

JLouis




2012/10/8 Romain Manni-Bucau <rmannibucau@gmail.com>

> no need IMO
>
> would you want to implement it?
>
> http://cobertura.sourceforge.net/sample/net.sourceforge.cobertura.coveragedata.HasBeenInstrumented.html
>
> *Romain Manni-Bucau*
> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> *Blog: **http://rmannibucau.wordpress.com/*<
> http://rmannibucau.wordpress.com/>
> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> *Github: https://github.com/rmannibucau*
>
>
>
>
> 2012/10/8 Jean-Louis MONTEIRO <jeanouii@gmail.com>
>
> > Thanks Romain but I would prefer to make it optional.
> > Something like -Dopenejb.skip.cobertura
> >
> > 2012/10/8 Romain Manni-Bucau <rmannibucau@gmail.com>
> >
> > > done https://issues.apache.org/jira/browse/OPENEJB-1915
> > >
> > > *Romain Manni-Bucau*
> > > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> > > *Blog: **http://rmannibucau.wordpress.com/*<
> > > http://rmannibucau.wordpress.com/>
> > > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> > > *Github: https://github.com/rmannibucau*
> > >
> > >
> > >
> > >
> > > 2012/10/8 Jean-Louis MONTEIRO <jeanouii@gmail.com>
> > >
> > > > Maybe, we could filter Cobertura interfaces during scanning and
> > > application
> > > > validation to avoid such errors and such workarounds.
> > > > WDYT?
> > > >
> > > > JLouis
> > > >
> > > > 2012/10/8 amber <amber63@hotmail.fr>
> > > >
> > > > > hi Romain,
> > > > >
> > > > > exact, same error (got some hidden classes ;) )
> > > > >
> > > > > Last version give me better traces : GRAVE - FAIL ... FooDAOImpl:
> > > > >  Ejb
> > > > > [FooDAOImpl] implements too many interfaces:
> [my.company.dao.FooDAO,
> > > > > net.sourceforge.cobertura.coveragedata.HasBeenInstrumented]. Use
> > > > > @Local(<interface>) or keep a single interface.
> > > > >
> > > > > After checking all my @stateless classes and adding @Local
> > > > > (<interface>.class) , Cobertura works fine :
> > > > >
> > > > > ...
> > > > > @Local (IFoo.class)
> > > > > @Stateless
> > > > > public class Foo implements IFoo
> > > > > ...
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > thanks again for your help :)
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > View this message in context:
> > > > >
> > > >
> > >
> >
> http://openejb.979440.n4.nabble.com/Cobertura-error-tp4657791p4657852.html
> > > > > Sent from the OpenEJB User mailing list archive at Nabble.com.
> > > > >
> > > >
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message