Bug 90723 - jdt.junit.runtime is a special case
Summary: jdt.junit.runtime is a special case
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 blocker (vote)
Target Milestone: ---   Edit
Assignee: JDT-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-07 16:55 EDT by Wassim Melhem CLA
Modified: 2005-04-08 03:22 EDT (History)
2 users (show)

See Also:


Attachments
Patch for the jdt.junit.runtime project (1.40 KB, patch)
2005-04-07 16:58 EDT, Wassim Melhem CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Wassim Melhem CLA 2005-04-07 16:55:47 EDT
The plugin org.eclipse.jdt.junit runtime is a special case.

It is exempt from the JARd plugin format.

It requires a special treatment when it comes to adopting a manifest.mf file.

Reason:
Eclipse 3.x supports running 2.x Plugin Junit tests.  If you recall, in 2.1, 
Plugin Junit testing was not part of the sdk.
Therefore, what we do is implicitly append the org.eclipse.jdt.junit.runtime 
and org.eclipse.pde.junit.runtime plugins to the list of the plugins to run 
when running 2.x Junit tests.

The 2.x runtime does not understand JARd plugins, therefore, jdt.junit.runtime 
must NOT be JARd

Also, the 2.x runtime does not understand manifest.mf.  Therefore, the 
plugin.xml for said plugin must remain intact so that it could be read by the 
2.x runtime.  The plugin must still have a manifest.mf so that it could be 
read by the 3.x runtime.

So the plugin.xml for jdt.junit.runtime will contain deprecated/unused 
elements (as per the 3.0 standards) since it has both a full 
plugin.xml/manifest.mf.  To not show deprecation warnings on your plugin.xml, 
the attached patch contains project preferences for the Plugin manifest 
validator.
Comment 1 Wassim Melhem CLA 2005-04-07 16:58:00 EDT
Created attachment 19665 [details]
Patch for the jdt.junit.runtime project
Comment 2 Dani Megert CLA 2005-04-08 03:22:45 EDT
Thanks for the info and the patch.
Fixed in HEAD.
Available in builds > 20050408.