[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