Bug 145980 - Link with Editor broken in Test Perspective.
Summary: Link with Editor broken in Test Perspective.
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P1 major (vote)
Target Milestone: ---   Edit
Assignee: Jerome Gout CLA
QA Contact:
URL:
Whiteboard:
Keywords: plan
: 153589 161326 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-06-08 07:53 EDT by Paul Slauenwhite CLA
Modified: 2016-05-05 10:29 EDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Slauenwhite CLA 2006-06-08 07:53:41 EDT
Link with Editor broken in Test Perspective.

Using the TPTP-4.2.0-200606080100 build, selecting the Link with Editor button does not highlight the associated file in the Test Navigator when the file is open.
Comment 1 Jerome Gout CLA 2006-10-19 12:36:13 EDT
*** Bug 161326 has been marked as a duplicate of this bug. ***
Comment 2 Navid Mehregani CLA 2006-10-19 14:14:57 EDT
Why is this defect targetted to future?  If the Test team isn't planing to fix this, they should remove the button altogether.  What is the reason for keeping this button if it has no purpose? I believe this is a serious usability issue and shouldn't be treated as a normal defect.
Comment 3 Jerome Gout CLA 2006-10-20 09:50:49 EDT
We need to change slightly the behavior of this button action in order to imitate the JDT 'link to editor' action.

The bring to top event of an editor should select and reveal its element in the test navigator.

FYI, at the moment the selection of an element in the test navigator brings to top the open editor of this element.
Comment 4 Paul Slauenwhite CLA 2006-10-20 15:24:47 EDT
Jerome, please increase the priority to ensure it is completed for 4.4.
Comment 5 Jerome Gout CLA 2007-01-16 08:19:00 EST
*** Bug 129228 has been marked as a duplicate of this bug. ***
Comment 6 Jerome Gout CLA 2007-01-16 08:51:08 EST
*** Bug 153589 has been marked as a duplicate of this bug. ***
Comment 7 Paul Slauenwhite CLA 2007-01-17 09:44:07 EST
Has the priority for this defect been recently reviewed given its severity?  If not, please take a few minutes and reevaluate the priority.
Comment 8 Kurtis Martin CLA 2007-01-24 10:53:01 EST
Test Navigator link with editor only works for BaseEditorPart.  The implementation of Link With Editor should be more generic to work with any editor who's input can be adapted to an IFile.  This is similar to the behavior of the Eclipse Resource Navigator.  We have made use of the testNavigatorFileProxyFactory extension to add files of type .trcmxmi to the TestNavigator.  This file has it's own custom editor associated with it.  The editor is not of type BaseEditorPart.  Hence, Link With Editor doesn't work in the Test Navigator.  This would probably be common for users of the testNavigatorFileProxyFactory extension since these are untyped test assets which don't have associated Hyades editors.  

I also notice this issue for .html files and .execution files which are shown in the Test Navigator but don't use BaseEditorPart.  So that means "Link With Editor" doesn't work for Reports or Test Log Viewer.

It seems like work is being done in this area, so I think it should be implemented in such a way that doesn't restrict itself to BaseEditorPart.  I have limited experience with such implementation but I found an article which describes how to register a part listener with the workbench page to achieve "Link With Editor" behavior.  Here's the article:

http://eclipse.pookzilla.net/2006/10/link-to-editor.php

Also, I want to mention the Test Navigator correctly brings an open editor to the front when its corresponding ProxyNode is selected.  This even works for my editor.  So, that aspect of link to editor works fine.  It's just the other direction that doesn't work.
Comment 9 Paul Slauenwhite CLA 2007-01-26 08:45:10 EST
Will be included in the 4.4 plan.  

Please target to a 4.4 iteration.
Comment 10 Paul Slauenwhite CLA 2007-03-23 16:13:34 EDT
Assigning target.
Comment 11 Jerome Gout CLA 2007-05-03 11:53:09 EDT
Fixed.

Editor input is no longer needed to implement IFileEditorInput to be in sync with the test navigator if it is adaptable in IFile. (input.getAdapter(IFile.class) != null)

Html nodes in the test navigator are not handled by link to editor since the input is not adaptable in IFile (this is also the case in the resource navigator).
Comment 12 Paul Slauenwhite CLA 2007-06-02 14:51:59 EDT
Reporter: Please verify and close in preparation for shutting down the TPTP 4.4 release.  Thanks.
Comment 13 Navid Mehregani CLA 2007-06-03 22:45:32 EDT
Verified with the candidate driver.
Comment 14 Kurtis Martin CLA 2007-06-04 08:55:29 EDT
I have reviewed the related 4.4 code changes Jerome made to TestNavigator.  They look good and take care of the issue I raised in comment #8.  Thanks!
Comment 15 Paul Slauenwhite CLA 2007-06-25 07:34:09 EDT
Closing since verified by originator.