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

Bug 208110

Summary: Generic Recorder Wizard enhancement to filter the list of recorders
Product: z_Archived Reporter: Kent D Siefkes <kdsiefke>
Component: TPTPAssignee: Paul Slauenwhite <paulslau>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P1 CC: dmorris, jcayne, jkubasta, paulslau
Version: unspecifiedKeywords: plan
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
URL: http://www.eclipse.org/tptp/groups/Architecture/documents/features/hf_208110.htm
Whiteboard:
Bug Depends on: 137591, 139589, 206078, 217440, 221861    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
New plug-ins
none
New feature.
none
Patch (part 2) none

Description Kent D Siefkes CLA 2007-10-30 15:51:52 EDT
Add a new API or entry point in the Generic Recorder Wizard that instructs it to restrict the list of recorders that can be invoked to perform the recording.  The suggestion is to have it accept an array of IDs to be excluded from the list of contributed recorders that will be presented to the user.

This will allow vendors an ability to improve usability by providing a way to remove potential user confusion or prevent the user from going down a wrong path when multiple, perhaps similarly named recorders are all presented as valid choices for selection.
Comment 1 Paul Slauenwhite CLA 2007-11-15 14:30:46 EST
Updated Description Document based on AG review (http://wiki.eclipse.org/TPTP-Test-Tools-Design-Review-208110).
Comment 2 Paul Slauenwhite CLA 2007-11-26 12:08:03 EST
Retargetting to feature freeze iteration.
Comment 3 Paul Slauenwhite CLA 2008-01-22 12:58:05 EST
As part of this feature, we will remove the duplicate wizard for creating a URL recording/test to reduce confusion.
Comment 4 Paul Slauenwhite CLA 2008-03-06 09:05:44 EST
We also need to write enablement and TVT test cases for this new function (or modify existing test cases for modified/removed function) by March 14:

Writing translation verification test cases:  http://www.eclipse.org/tptp/home/documents/process/test/tvt.html

Writing enablement test cases:
http://www.eclipse.org/tptp/home/documents/process/test/enablement.html


Comment 5 Paul Slauenwhite CLA 2008-03-10 12:35:17 EDT
-Work item #1 from http://www.eclipse.org/tptp/groups/Architecture/documents/features/hf_208110.htm:

/org.eclipse.hyades.test.tools.ui/plugin.properties
/org.eclipse.hyades.test.tools.ui/plugin.xml

-Changed the name of the HTTP Recorder (and HTTP Recording) to URL Recorder (and URL Recording) including resource messages and documentation:

org.eclipse.hyades.execution.recorder.http.remote.CleanupObj.java
org.eclipse.hyades.execution.recorder.http.remote.RecorderX509TrustManager.java
org.eclipse.hyades.execution.recorder.http.local.HttpRecorderAgentEnvironmentAdapter.java
org.eclipse.hyades.execution.recorder.http.ui.preferences.HttpRecorderPreferencePage.java

/org.eclipse.hyades.test.core/src/org/eclipse/hyades/test/core/internal/resources/messages.properties
/org.eclipse.hyades.test.core/schema/Recorder.exsd

/org.eclipse.hyades.test.doc.user/concepts/cedittest.htm
/org.eclipse.hyades.test.doc.user/tasks/trecordtest.html

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/schema/HttpRecorderStartupPage.exsd
/org.eclipse.hyades.test.tools.ui/plugin.properties
/org.eclipse.hyades.test.tools.ui/plugin.xml

/org.eclipse.tptp.test.doc.isv/concepts/c_exnewtestrec.html
/org.eclipse.tptp.test.doc.isv/tasks/t_cretestrec.html
/org.eclipse.tptp.test.doc.isv/tasks/t_extgenrec.html
Comment 6 Paul Slauenwhite CLA 2008-03-11 14:09:17 EDT
Created attachment 92210 [details]
Patch
Comment 7 Paul Slauenwhite CLA 2008-03-11 14:10:21 EDT
Created attachment 92211 [details]
New plug-ins
Comment 8 Paul Slauenwhite CLA 2008-03-11 14:10:50 EDT
Created attachment 92212 [details]
New feature.
Comment 9 Paul Slauenwhite CLA 2008-03-11 14:11:40 EDT
-This enhancement required a substantial amount of refactoring to remove dead/duplicate/redundant code, simplify the URL Recorder preferences, and improve the readability of UI strings.

-[See patch+ new plug-ins/feature] Work item #2 from http://www.eclipse.org/tptp/groups/Architecture/documents/features/hf_208110.htm, including a new plug-ins (org.eclipse.tptp.test.recorders.url and org.eclipse.tptp.test.recorders.url.doc.user) and feature (org.eclipse.tptp.test.recorders.url).
Comment 10 Paul Slauenwhite CLA 2008-03-11 14:19:50 EDT
Joel, can you add the following plug-ins and feature to the TPTP Testing Tools  runtime and SDK packaging:

Feature:
org.eclipse.tptp.test.recorders.url

Plug-ins:
org.eclipse.tptp.test.recorders.url
org.eclipse.tptp.test.recorders.url.doc.user
Comment 11 Paul Slauenwhite CLA 2008-03-11 14:21:07 EDT
Patch and new plug-ins/feature checked into CVS (HEAD).

Duwayne, please test using the next build.
Comment 12 Paul Slauenwhite CLA 2008-03-11 14:31:32 EDT
(In reply to comment #9)

The documentation for the URL Recorder has been isolated to the org.eclipse.tptp.test.recorders.url.doc.user.  A file was added and the existing URL Test type docuemtnation was refactored for creating a URL Test manually without a recording.

Comment 13 Joel Cayne CLA 2008-03-11 16:52:40 EDT
org.eclipse.tptp.test.recorders.url added to the build.
Comment 14 Paul Slauenwhite CLA 2008-03-12 06:48:29 EDT
Created attachment 92286 [details]
Patch (part 2)

This patch includes the deprecated/internal files that were moved to the new org.eclipse.tptp.test.recorders.url plug-in and deleted from their original plug-in.
Comment 15 Paul Slauenwhite CLA 2008-03-13 12:55:14 EDT
Test cases added to /org.eclipse.hyades.test.URLTest.tests/manual/basic_tests/Test.Agents.Recorder.testsuite  to test an installation with this new feature removed/disabled.
Comment 16 Paul Slauenwhite CLA 2008-03-13 12:58:29 EDT
(In reply to comment #14)
> Created an attachment (id=92286) [details]
> Patch (part 2)
> 
> This patch includes the deprecated/internal files that were moved to the new
> org.eclipse.tptp.test.recorders.url plug-in and deleted from their original
> plug-in.  
> 

In addition, the src-recorder-http source folder from org.eclipse.hyades.test.tools.ui.
Comment 17 Paul Slauenwhite CLA 2008-03-13 15:13:17 EDT
Design, implementation, documentation, test cases, and build/infrastructure complete.  

Testing was completed on the TPTP-4.5.0-200803130100 driver.  

New plug-ins/feature added to the TPTP Project Matrix.  

Finally, all of the dependent defects have been resolved.

Joel, can you please regenerate the TPTP Project Matrix?

Duwayne, can you please test the TPTP-4.5.0-200803130100 or later driver?
Comment 18 Paul Slauenwhite CLA 2008-11-27 19:23:14 EST
Approval request(s) no longer required.
Comment 19 Paul Slauenwhite CLA 2009-02-25 11:58:43 EST
Closing by default since not closed by the originator in the 7+ months since
being resolved.  

Please reopen if the issue is still present in the latest TPTP release or the
resolution is not correct.