Bug 19626 - [JUnit] (post 2.0) Display method icons in TestCase wizard
Summary: [JUnit] (post 2.0) Display method icons in TestCase wizard
Status: RESOLVED WONTFIX
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: JDT-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: investigate
Depends on:
Blocks:
 
Reported: 2002-06-07 13:21 EDT by Eric Nickell CLA
Modified: 2003-08-07 07:16 EDT (History)
0 users

See Also:


Attachments
fix (1.39 KB, patch)
2002-12-08 11:45 EST, Sebastian Davids CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Nickell CLA 2002-06-07 13:21:53 EDT
When using the JUnit TestCase wizard, one ends up at a dialog box labeled, "Test
Methods / Select methods for which test method stubs should be created".

It would be helpful to show the same icons for the various methods that are
shown in the outline view.  (Example: In situations where I'm constructing an
abstract class test hierarchy, I usually only test public methods which are not
overriding the superclass.)
Comment 1 Erich Gamma CLA 2002-06-07 14:00:27 EDT
sigh, missed this one, but cannot fix for 2.0
Comment 2 Sebastian Davids CLA 2002-12-08 11:45:13 EST
Created attachment 2709 [details]
fix

Implementation notes:

uses the internal AppearanceAwareLabelProvider -- I did not want to copy code.

New Public API?
Comment 3 Erich Gamma CLA 2002-12-14 17:07:24 EST
applied patch - we should surface the more sophisticated label provider as 
well. 
However, I'd prefer to not surface the concrete type, suggest a factory method 
in ILabelProvider JavaUI.getStandardLabelProvider()
Comment 4 Martin Aeschlimann CLA 2003-01-29 13:17:10 EST
Not so easy as this would mean that 30 flags would become API as well.
Moving to jdt-inbox for reconsideration.
Comment 5 Erich Gamma CLA 2003-02-20 18:05:17 EST
no further action planned
Comment 6 Eric Nickell CLA 2003-02-24 13:55:22 EST
*sigh*

Thanks for trying.