[jira] Closed: (LOOM-52) Build fails due to test failure for ConfigurationMergerTestCase.testAddChild()

jira-yCVjj/[email protected] Mon, 26 Jan 2004 21:30:59 -0500 (EST)
Newsgroups gmane.comp.java.jcontainer.interest
Message-ID <[email protected]>
Message:

   The following issue has been closed.

   Resolver: Peter Donald
       Date: Mon, 26 Jan 2004 9:30 PM

Fixed by making sure the order that children are added is the order that they appear in with merged afte base configs
---------------------------------------------------------------------
View the issue:
  http://jira.codehaus.org/secure/ViewIssue.jspa?key=LOOM-52

Here is an overview of the issue:
---------------------------------------------------------------------
        Key: LOOM-52
    Summary: Build fails due to test failure for ConfigurationMergerTestCase.testAddChild()
       Type: Bug

     Status: Closed
   Priority: Major
 Resolution: FIXED

 Original Estimate: Unknown
 Time Spent: Unknown
  Remaining: Unknown

    Project: loom
 Components: 
             Engine

   Assignee: Peter Donald
   Reporter: Ryan Hoegg

    Created: Thu, 22 Jan 2004 11:20 PM
    Updated: Mon, 26 Jan 2004 9:30 PM

Description:
I did some debugging and it looks like the problem is the order of child elements in the merged Configuration; I looked into ConfigurationUtil in DNA and it definitely does care what order child Configurations are in. 
To get my version to build I just reversed kid1 and kid2, but we should probably either rethink the merged child order or kick this to DNA.  The Configuration comparison is in ConfigurationUtil.equals


---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira