This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 92363 - Add Test Module dependency creates another test-type markup tag
Summary: Add Test Module dependency creates another test-type markup tag
Status: VERIFIED FIXED
Alias: None
Product: apisupport
Classification: Unclassified
Component: Project (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: Tomas Danek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-11 17:29 UTC by pzajac
Modified: 2007-01-15 15:11 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description pzajac 2007-01-11 17:29:06 UTC 
[200701101900]
I tried add test dependency to apisupport/project on org.openide.filesystems. It
deletes all test dependencies. There is no undo (only cvs revert). The use case
is reproducible on other project types with the same test dependencies.
Comment 1 Marian Mirilovic 2007-01-12 10:06:11 UTC 
Actually they are not deleted, the problem is that new <test-type>..</test-type>
markup tag is created after existing one, this caused to remove nodes from the
Projects view ;(

generated (broken) project.xml
================================
   <test-type>
      <name>unit</name>
      <test-dependency>
         <code-name-base>org.openide.filesystems</code-name-base>
         <compile-dependency/>
      </test-dependency>
      <test-dependency>
         <code-name-base>org.openide.awt</code-name-base>
         <compile-dependency/>
      </test-dependency>
      <test-dependency>
         <code-name-base>org.openide.util</code-name-base>
         <compile-dependency/>
      </test-dependency>
   </test-type>
   <test-type>
      <name>unit</name>
      <test-dependency>
         <code-name-base>org.openide.dialogs</code-name-base>
         <compile-dependency/>
      </test-dependency>
   </test-type>
</test-dependencies>
================================

Tomas, this needs to be fixed ASAP ...
Comment 2 Tomas Danek 2007-01-12 11:23:54 UTC 
yes, this was already reproduced, see issue 91080.
Comment 3 Tomas Danek 2007-01-12 13:27:35 UTC 
Checking in ProjectXMLManager.java;
/cvs/apisupport/project/src/org/netbeans/modules/apisupport/project/ProjectXMLManager.java,v
 <--  ProjectXMLManager.java
new revision: 1.55; previous revision: 1.54
done
Comment 4 pzajac 2007-01-12 14:31:20 UTC 
and test?
Comment 5 pzajac 2007-01-12 16:11:48 UTC 
Tomas promised to write test on monday.

Verified.
Comment 6 Tomas Danek 2007-01-15 14:47:44 UTC 
I added test as I promised:-)

Checking in ProjectXMLManagerTest.java;
/cvs/apisupport/project/test/unit/src/org/netbeans/modules/apisupport/project/ProjectXMLManagerTest.java,v
 <--  ProjectXMLManagerTest.java
new revision: 1.37; previous revision: 1.36
done
Comment 7 pzajac 2007-01-15 15:11:02 UTC 
Great!