[seam-dev] Seam 3: webbeans-logging/webbeans-logger

[Pete Muir]

What sort of unit test are you thinking of? The unit tests we do in  
Web Beans and the TCK run WB, so cause the injection to occur (do we  
need to provide a mock logger?!) - I though you used a similar thing  
in Seam...

On 29 May 2009, at 18:12, Dan Allen wrote:

> On Fri, May 29, 2009 at 10:00 AM, Pete Muir  
> <pmuir at bleepbleep.org.uk> wrote:
> Yeah, the split is un-obvious (esp due to naming), but we wanted a  
> module we could use inside the RI, that didn't have the producer  
> method on it....
>
> So I'm wondering, what should the standard signature of this  
> injection be? Always a field injection (as opposed to a constructor  
> injection)? What about access (package or private)?
>
> private @Logger Log log;
>
> The reason I ask about access (and constructor injection) is because  
> this could be the one pain in the side to unit testing a bean.  
> Package access just makes it easier to inject a stub. What would be  
> interesting is if the field could be seeded with a stub so that the  
> logger just works in a unit test.
>
> private @Logger Log log = new NoOpLogImpl();
>
> The injection would overwrite this value. Just an idea.
>
> -Dan
>
> -- 
> Dan Allen
> Senior Software Engineer, Red Hat | Author of Seam in Action
>
> http://mojavelinux.com
> http://mojavelinux.com/seaminaction
> http://in.relation.to/Bloggers/Dan
>
> NOTE: While I make a strong effort to keep up with my email on a daily
> basis, personal or other work matters can sometimes keep me away
> from my email. If you contact me, but don't hear back for more than  
> a week,
> it is very likely that I am excessively backlogged or the message was
> caught in the spam filters.  Please don't hesitate to resend a  
> message if
> you feel that it did not reach my attention.
> _______________________________________________
> seam-dev mailing list
> seam-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/seam-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/seam-dev/attachments/20090601/11779e9b/attachment.html