RE: Pico 2.4 release

View: New views
5 Messages — Rating Filter:   Alert me  

RE: Pico 2.4 release

by Michael Rimov :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Paul, (or anybody else :) )

If you could look at one issue for me before you release, I'd appreciate it.

As I was going through things, it turned out that
org.picocontainer.gems.behaviors.PooledTestCase test case methods were not
running (They were missing their @Test annotation)

However, now that I've restored the test cases, there are two that are failing.
It's just too late at night for me to go digging to figure out what's up.
They're currently tagged with @Ignore.

Other than that, I believe I've addressed Jörg's serialization concerns, and
I've integrated enough of gems in a unified manner that I'm happy with release
of Pico and Script.

I never got do any real docs for Persistence, nor have I been working with Web
yet, but that's fine, I'll work to get them next round.

                                                        -Mike



---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email



Re: RE: Pico 2.4 release

by Paul Hammant-3 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

> As I was going through things, it turned out that
> org.picocontainer.gems.behaviors.PooledTestCase test case methods  
> were not
> running (They were missing their @Test annotation)
>
> However, now that I've restored the test cases, there are two that  
> are failing.
> It's just too late at night for me to go digging to figure out  
> what's up.
> They're currently tagged with @Ignore.
>

Weird. They both pass in IDEA and fail from Maven/Command-line :-(




---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email



Re: RE: Pico 2.4 release

by Paul Hammant-3 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


On Jun 25, 2008, at 6:01 AM, Paul Hammant wrote:

>> As I was going through things, it turned out that
>> org.picocontainer.gems.behaviors.PooledTestCase test case methods  
>> were not
>> running (They were missing their @Test annotation)
>>
>> However, now that I've restored the test cases, there are two that  
>> are failing.
>> It's just too late at night for me to go digging to figure out  
>> what's up.
>> They're currently tagged with @Ignore.
>>
>
> Weird. They both pass in IDEA and fail from Maven/Command-line :-(

Hmm. Its related to the use of System.gc(); I think.  If you comment  
that line out (first test) then it fails in IDEA too

- Paul

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email



RE: RE: Pico 2.4 release

by Jörg Schaible :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


I might have a look later. IIRC those tests have been originally my stuff ...

Paul Hammant wrote:

> On Jun 25, 2008, at 6:01 AM, Paul Hammant wrote:
>
>>> As I was going through things, it turned out that
>>> org.picocontainer.gems.behaviors.PooledTestCase test case methods
>>> were not running (They were missing their @Test annotation)
>>>
>>> However, now that I've restored the test cases, there are two that
>>> are failing. It's just too late at night for me to go digging to
>>> figure out what's up. They're currently tagged with @Ignore.
>>>
>>
>> Weird. They both pass in IDEA and fail from Maven/Command-line :-(
>
> Hmm. Its related to the use of System.gc(); I think.  If you comment
> that line out (first test) then it fails in IDEA too
>
> - Paul
>
> ---------------------------------------------------------------------
> To unsubscribe from this list, please visit:
>
>     http://xircles.codehaus.org/manage_email


---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email



Re: RE: Pico 2.4 release

by Paul Hammant-3 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


>
> I might have a look later. IIRC those tests have been originally my  
> stuff ...
>

Fixed.  it was a missing @Before tag above a former setUp() method.

- Paul

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


LightInTheBox - Buy quality products at wholesale price