Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 342570

Summary: Packages in Test-Import-Package are not sorted when formatting the template.mf
Product: [RT] Virgo Reporter: Martin Lippert <mlippert>
Component: toolingAssignee: Leo Dos Santos <leo.dos.santos>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: eclipse, mlippert, ric.klaren
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Mac OS X   
Whiteboard:
Bug Depends on:    
Bug Blocks: 368782    

Description Martin Lippert CLA 2011-04-12 10:13:53 EDT 
Consider this stripped down template.mf:

--snip--
Bundle-SymbolicName: eu.educator.supervision
Bundle-Name: eu.educator.supervision
Bundle-Vendor: Educator
Bundle-ManifestVersion: 2
Bundle-Version: 1.0.0
Excluded-Exports: eu.educator.supervision.service.impl,
eu.educator.supervision.domain.repository,
eu.educator.supervision.domain.repository.impl
Import-Package: org.springframework.transaction.interceptor,
eu.educator.core.controller.repository.v2
Test-Import-Package: org.jmock.syntax,
org.hamcrest
Excluded-Imports: eu.educator.supervision.service.impl,
eu.educator.supervision.domain.repository,
eu.educator.supervision.domain.repository.impl
--snip--

Note the Test-Import-Package header. When I open this template.mf in STS go to the text view and press ctrl-shift-f the Test-Import-Package contents are not reordered, the others are.

(This issue was moved from the old dm server tooling issue tracker: https://issuetracker.springsource.com/browse/STS-876)
Comment 1 Miles Parker CLA 2012-01-16 18:17:34 EST 
It looks like Test-Import-Package is not supported as a MANIFEST item at all, which explains why it can't be supported. If it *should* be supported, please re-open with edited description to reflect that.
Comment 2 Leo Dos Santos CLA 2012-01-17 16:36:18 EST 
A TEMPLATE.MF can be used to generate separate product and test manifests for a particular bundle (think MANIFEST.MF & TEST.MF). In this case the Test-Import-Package header is a bundlor header; a hint to generate an Import-Package header with the given packages to the TEST.MF. You can get an idea of how this works in BundlorProjectBuilder.java

I'm reopening this because dm Server Tools was generally pretty good about supporting bundlor and dm Server specific manifest headers and we should continue to maintain that for Virgo IDE
Comment 3 Miles Parker CLA 2012-01-17 17:10:06 EST 
Cool. Perhaps we need to actually enlarge task? Because currently you don't get auto-complete, etc. either.