Bug 145070 - Test Log Viewer: Link to deployment is displayed when using temp deploy not named 'default'
Summary: Test Log Viewer: Link to deployment is displayed when using temp deploy not n...
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P1 normal (vote)
Target Milestone: ---   Edit
Assignee: Paul Slauenwhite CLA
QA Contact:
URL:
Whiteboard:
Keywords: plan
Depends on:
Blocks:
 
Reported: 2006-06-02 10:59 EDT by Ashish Mathur CLA
Modified: 2016-05-05 10:30 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ashish Mathur CLA 2006-06-02 10:59:13 EDT
The Test Log Viewer displays a link to the deployment used to run the test. In cases where the run use the default temporary deployment named 'default', it hides the link. So far so good.

However, the temporary deployment may not always be called 'default' and is certainly the case of my product. I would rather the check be made to see if the deployment object is associated with an eResource which will determine if the deployment is temporary or persisted.

I have talked to Bianca nad she agrees.
Comment 1 Bianca Jiang CLA 2006-11-03 16:47:57 EST
Not containable in 4.3. Retargeting to 4.4.
Comment 2 Paul Slauenwhite CLA 2007-01-17 09:23:15 EST
Sizing updated as provided by Bianaca.
Comment 3 Paul Slauenwhite CLA 2007-01-26 08:25:20 EST
Targeting to future since not containable in 4.4.
Comment 4 Paul Slauenwhite CLA 2007-01-26 09:01:12 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 5 Paul Slauenwhite CLA 2007-01-26 09:03:30 EST
Correction:  The Test Project mailing list is tptp-testing-tools-dev@eclipse.org.
Comment 6 Kurtis Martin CLA 2007-05-15 16:01:46 EDT
I'm in the same boat as the original submitter of this bugzilla.  It is desired for the check to call createDeploymentLinks() to make sure deployment.eResource() != null.  This is in class ExecutionResultDetailsPart, method setFormInput.

I would like to add that even if not dealing with temporary deployments, this is important in case the deployment file has been deleted.  Using base TPTP, if you delete a deploy and open a test that used it.  There will still be a hyperlink deployment icon shown in Test Log Viewer.  If you click it, an editor opens showing an Exception has occurred.  This will always happen if deployment.eResource() == null.  So, it's better to check for this condition and not show the deployment link.
Comment 7 Paul Slauenwhite CLA 2007-11-07 06:33:07 EST
Bulk update of the target field to reflect the change in iteration numbers.
Comment 8 Paul Slauenwhite CLA 2007-12-10 10:19:53 EST
Deferring to I5.
Comment 9 Paul Slauenwhite CLA 2008-01-07 11:48:57 EST
Deferring to I6.
Comment 10 Paul Slauenwhite CLA 2008-03-31 12:14:05 EDT
Deferring to I7 as discussed by PMC/AG (http://wiki.eclipse.org/TPTP-AG-20080331#Test_Project).
Comment 11 Paul Slauenwhite CLA 2008-04-11 11:47:13 EDT
org.eclipse.hyades.test.ui.forms.base.ExecutionResultDetailsPart.java checked in into CVS (HEAD).
Comment 12 Paul Slauenwhite CLA 2009-02-25 11:58:58 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.
Comment 13 Paul Slauenwhite CLA 2009-11-23 10:38:14 EST
Correcting component.