myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dennis Byrne" <den...@dbyrne.net>
Subject Re: FactoryFinder.releaseFactories() - was [continuum] BUILD FAILURE: API
Date Tue, 21 Feb 2006 05:20:59 GMT
John,

Do you guys have anything fancy for ThreadLocal references of FacesContext?  Did you set this
to null per TestSuite? per TestCase? per test method ?

Dennis Byrne

>-----Original Message-----
>From: John Fallows [mailto:john.r.fallows@gmail.com]
>Sent: Tuesday, February 21, 2006 12:07 AM
>To: 'MyFaces Development'
>Subject: Re: FactoryFinder.releaseFactories() - was [continuum] BUILD FAILURE: API
>
>fyi - IIRC we tackled this problem slightly differently in the ADF Faces
>codebase.
>
>In an effort to fully isolate anything that might be keyed by
>ContextClassLoader (including FactoryFinder internal state), we created a
>trivial wrapper ClassLoader to provide a unique ContextClassLoader in
>setUp() and restore it back to the original in tearDown().
>
>Kind Regards,
>John Fallows.
>
>On 2/20/06, Sean Schofield <sean.schofield@gmail.com> wrote:
>>
>> > The call is required in the setUp() method to make things work correctly
>> on
>> > the *first* test, when you have the MyFaces implementation in the
>> classpath
>> > of the tests.  Calling it in tearDown() doesn't hurt anything, but
>> protects
>> > test developers who try to subvert the JUnit test lifecycle stuff.
>>
>> Exactly.  I don't think its necessary in the teardown but it won't
>> hurt.  Its *defnitely* needed in the setup for the reason you
>> mentioned.
>>
>> > Craig
>>
>> Sean
>>
>
>
>
>--
>http://apress.com/book/bookDisplay.html?bID=10044
>Author: Pro JSF and Ajax: Building Rich Internet Components, Apress
>



Mime
View raw message