[picocontainer-dev] Simple failing test case for trunk

Konstantin Priblouda <kpriblouda-/[email protected]>
Newsgroups gmane.comp.java.picocontainer.devel
Message-ID <[email protected]>
Hi all, 
Just discovered ( while hacking in branch ) 
that this simple test case fails miserably in trunk:
-------------%<------------      
    public void testThatStringIsAddedOnlyOnce() {
        MutablePicoContainer container =
createPicoContainer(null);
        
        container.addComponent("foo bar");
       
assertEquals(1,container.getComponentAdapters().size());
    }
---------------%<--------------

Is this intendet behavour to add string value twice? 
( first string/string , then String.class/string ) 

Fix is easy though: 
DPC @ 336 - convert to return statement. 
and does not break available tests.
Shall I commit it?  Or do I miss something important? 




----[ Konstantin Pribluda http://www.pribluda.de ]----------------
JTec quality components: http://www.pribluda.de/projects/


      ____________________________________________________________________________________
Never miss a thing.  Make Yahoo your home page. 
http://www.yahoo.com/r/hs

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

    http://xircles.codehaus.org/manage_email
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.