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

Bug 145070

Summary:	Test Log Viewer: Link to deployment is displayed when using temp deploy not named 'default'
Product:	z_Archived	Reporter:	Ashish Mathur <ashishkumarmathur>
Component:	TPTP	Assignee:	Paul Slauenwhite <paulslau>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P1	CC:	jkubasta, paulslau
Version:	unspecified	Keywords:	plan
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Ashish Mathur

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

2006-11-03 16:47:57 EST

Not containable in 4.3. Retargeting to 4.4.

Comment 2 Paul Slauenwhite

2007-01-17 09:23:15 EST

Sizing updated as provided by Bianaca.

Comment 3 Paul Slauenwhite

2007-01-26 08:25:20 EST

Targeting to future since not containable in 4.4.

Comment 4 Paul Slauenwhite

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

2007-01-26 09:03:30 EST

Correction:  The Test Project mailing list is tptp-testing-tools-dev@eclipse.org.

Comment 6 Kurtis Martin

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

2007-11-07 06:33:07 EST

Bulk update of the target field to reflect the change in iteration numbers.

Comment 8 Paul Slauenwhite

2007-12-10 10:19:53 EST

Deferring to I5.

Comment 9 Paul Slauenwhite

2008-01-07 11:48:57 EST

Deferring to I6.

Comment 10 Paul Slauenwhite

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

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

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

2009-11-23 10:38:14 EST

Correcting component.