Bug 289288 - Proposed refactoring of MWEDebuggerLauncher, MWELaunchShortcut, and MWELaunchDelegate to offer more flexibility (extensibility) and usability
Summary: Proposed refactoring of MWEDebuggerLauncher, MWELaunchShortcut, and MWELaunch...
Status: NEW
Alias: None
Product: EMFT
Classification: Modeling
Component: MWE (show other bugs)
Version: unspecified   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 289287
Blocks:
  Show dependency tree
 
Reported: 2009-09-12 14:58 EDT by Alexander Nyßen CLA
Modified: 2009-09-12 15:01 EDT (History)
0 users

See Also:

Attachments
Patch comprising the described refactorings. (35.59 KB, patch)
2009-09-12 14:58 EDT, Alexander Nyßen CLA 		no flags Details | Diff
View All 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 Alexander Nyßen CLA 2009-09-12 14:58:06 EDT 
User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; de; rv:1.9.0.4) Gecko/2008102920 Firefox/3.0.4
Build Identifier: 

The current MWE launch mechanism is not extensible and is inversive regarding the classpath settings of the project containing the mwe workflow to launch.

I have attached a patch providing a complete refactoring of the MWE launch mechanism (MWELaunchShortcut, MWELaunchDelegate, and MWEDebuggerLauncher) that overcomes these problems. In detail, MWELaunchDelegate was completely refactored to dynamically calculate the required runtime classpath to pass to the WorkflowRunner (so the plug-in containing the workflow file does not have to specify those classes within its local classpath). The implementation allows allows to extend MWELaunchDelegate, so custom launch configurations can be derived (e.g. to perform instrumentation of any launched workflow to gather coverage information). The proposed patch is related to the refactoring of the WorkflowRunner class I have proposed by bug #289287. 

Reproducible: Always
Comment 1 Alexander Nyßen CLA 2009-09-12 14:58:44 EDT 
Created attachment 147049 [details]
Patch comprising the described refactorings.