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

Bug 137591

Summary: Duplicate wizards for creating a URL recording/test.
Product: z_Archived Reporter: Paul Slauenwhite <paulslau>
Component: TPTPAssignee: Paul Slauenwhite <paulslau>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P1 CC: jkubasta, kdsiefke, mddunn, nevicosi, paulslau, sluiman
Version: unspecifiedKeywords: plan
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on:    
Bug Blocks: 208110    

Description Paul Slauenwhite CLA 2006-04-19 16:00:48 EDT
Duplicate wizards for creating a URL recording/test.

There are two wizards for creating a URL recording/test (File >> New >> Other... >> Test >>):

Test from Recording
Recording >> HTTP Recording

The "HTTP Recording" wizard could be removed since the "Test from Recording" wizard provides the same functionality using the Generic Recorder Framework.  

Also, the "Test from Recording" wizard name is not intuitive or consistent with the TPTP URL Test wizard.  Consider using:

TPTP URL Test from an HTTP Recording
Comment 1 Mark D Dunn CLA 2006-04-28 15:48:43 EDT
Add Jeff as CC . 
Comment 2 Mark D Dunn CLA 2006-05-15 09:29:08 EDT
Jeff  - what do you think about removing this second option?  When you developed the GRF, was there a requirement to keep the old way of recording around for a certain amount of time.?
Comment 3 Jeff Nevicosi CLA 2006-05-15 11:09:11 EDT
There's a requirement for maintaining both wizard types until 5.0.

Comment 4 Paul Slauenwhite CLA 2006-05-29 16:49:08 EDT
We need to mark the necessary classes deprecated to initiate the removal process.
Comment 5 Mark D Dunn CLA 2006-05-31 09:01:54 EDT
Retarget for 4.2i4.  
Comment 6 Jeff Nevicosi CLA 2006-05-31 09:11:33 EDT
While Mark can double check, I'm fairly certain the old recorder classes have been marked as @deprecated since 4.1 (where the GRF was delivered as internal).
Comment 7 Mark D Dunn CLA 2006-06-05 10:03:24 EDT
Retarget to 4.2.1 .
Comment 8 Mark D Dunn CLA 2006-07-07 12:25:57 EDT
Retarget to 4.3.
Comment 9 Mark D Dunn CLA 2006-10-19 14:26:19 EDT
Reassign these Test.UI bugs/enhancements to Jerome.
Comment 10 Mark D Dunn CLA 2007-01-16 16:28:55 EST
Add estimate.  This needs to be deferred until 5.0 per Jeff's comment.  
Comment 11 Paul Slauenwhite CLA 2007-01-26 08:25:17 EST
Targeting to future since not containable in 4.4.
Comment 12 Paul Slauenwhite CLA 2007-01-26 08:59:13 EST
As discussed on this week's Test Project call (January 22, 2007), the Test Project will focus on existing P1 - P2/Blocker - Major and P1/Normal and test creation/automation defects (omitting defects dependant on outstanding features) in TPTP 4.4.  All other Test Project defects have been targeted to future.

If this defect has been targeted to future and you/originator feel it should be completed in 4.4, please provide the necessary reason as a reply to this comment or a post to the Test Project mailing list (tptp-test-tooling-dev@eclipse.org).  We will collectively triage and assess our resources to determine a case-by-case decision. 
Comment 13 Paul Slauenwhite CLA 2007-01-26 09:00:51 EST
As discussed on this week's Test Project call (January 22, 2007), the Test Project will focus on existing P1 - P2/Blocker - Major and P1/Normal and test creation/automation defects (omitting defects dependant on outstanding features) in TPTP 4.4.  All other Test Project defects have been targeted to future.

If this defect has been targeted to future and you/originator feel it should be completed in 4.4, please provide the necessary reason as a reply to this comment or a post to the Test Project mailing list (tptp-test-tooling-dev@eclipse.org).  We will collectively triage and assess our resources to determine a case-by-case decision. 
Comment 14 Paul Slauenwhite CLA 2007-01-26 09:03:09 EST
Correction:  The Test Project mailing list is tptp-testing-tools-dev@eclipse.org.
Comment 15 Paul Slauenwhite CLA 2007-10-11 13:19:51 EDT
Required for 4.5.
Comment 16 Paul Slauenwhite CLA 2008-01-07 09:03:38 EST
Deferring to I6.
Comment 17 Paul Slauenwhite CLA 2008-01-22 12:59:17 EST
Duwayne, I will do this defect as part of 208110.
Comment 18 Paul Slauenwhite CLA 2008-01-22 13:39:09 EST
In addition, we should remove the the import wizard for HTTP recordings (File > Import... >> Test >> HTTP Recording) since this function (e.g. create a test suite from an existing recording) is supported by the Generic Recording Framework.

Kent, can you confirm that these two wizards are no longer required by your consuming product?

Comment 19 Mark D Dunn CLA 2008-01-24 14:58:55 EST
I can confirm (at Kent's request) that these two wizards are no longer needed by our team.

1.  The Recording->HTTP Recording is the old way and is supplanted by the Generic Recorder Framework

2.  The File->Import...->HTTP Recording no longer works as intended and is obsolete.  The GRF provides the same functionality as intended by this wizard. 
Comment 20 Paul Slauenwhite CLA 2008-01-24 16:58:35 EST
(In reply to comment #19)
> I can confirm (at Kent's request) that these two wizards are no longer needed
> by our team.
> 
> 1.  The Recording->HTTP Recording is the old way and is supplanted by the
> Generic Recorder Framework
> 
> 2.  The File->Import...->HTTP Recording no longer works as intended and is
> obsolete.  The GRF provides the same functionality as intended by this wizard. 
> 

Thanks Mark and Kent.
Comment 21 Harm Sluiman CLA 2008-01-24 18:54:00 EST
Um, I don't follow the removal of the recording import function. 

The file formats are in effect API and we can't just remove the capability in a point release. 
Even if it is not working well, it did at one time (we claimed it did) and should be repaired. 
This sounds like related but not part of the orignal desire to reduce the redundancy in the wizards.
Comment 22 Paul Slauenwhite CLA 2008-01-25 07:12:05 EST
(In reply to comment #21)
> Um, I don't follow the removal of the recording import function. 
> 
> The file formats are in effect API and we can't just remove the capability in a
> point release. 

This removal is only for the wizard.  The HTTP recorder in the GRF still emits the same recording file formats.

> Even if it is not working well, it did at one time (we claimed it did) and
> should be repaired. 
> This sounds like related but not part of the orignal desire to reduce the
> redundancy in the wizards.
> 

It is part of the original intent to reduce redundancy since the GRF supports the exact same function.  That is, the user can create a URL test from an existing recording.  As with the duplicate wizards for creating a URL recording/test, this import wizard is duplicate function to the GRF.
Comment 23 Paul Slauenwhite CLA 2008-01-30 12:14:40 EST
Deferred from I5 to I6 with PMC approval.
Comment 24 Paul Slauenwhite CLA 2008-03-06 16:57:47 EST
The following files checked into CVS (HEAD):

org.eclipse.hyades.test.tools.ui.internal.resources.ToolsUiPluginResourceBundle.java
/org.eclipse.hyades.test.tools.ui/src/org/eclipse/hyades/test/tools/ui/internal/resources/messages.properties
/org.eclipse.hyades.test.tools.ui/META-INF/MANIFEST.MF
/org.eclipse.hyades.test.tools.ui/plugin.properties
/org.eclipse.hyades.test.tools.ui/plugin.xml

The the following deprecated package/classes (and the associated messages) of the org.eclipse.hyades.test.tools.ui
plug-in removed from CVS (HEAD) since they have been deprecated since TPTP 4.0:  

package: 
org.eclipse.hyades.execution.recorder.http.ui.actions 

classes:
org.eclipse.hyades.execution.recorder.http.ui.wizards.HttpRecorderFileSelectPage.java
org.eclipse.hyades.execution.recorder.http.ui.wizards.HttpRecorderLaunchWizard.java

Note: The package was also removed from the manifest file as exported packages.
Comment 25 Paul Slauenwhite CLA 2008-03-06 16:58:40 EST
In addition:

/org.eclipse.hyades.test.ui/plugin.properties
/org.eclipse.hyades.test.ui/plugin.xml
Comment 26 Paul Slauenwhite CLA 2008-06-10 10:00:05 EDT
Verified in TPTP-4.5.0-200806070100.  Closing.