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

Bug 314315

Summary: Test log possess the same id as its test suite
Product: z_Archived Reporter: Ben Chen <chenbs>
Component: TPTPAssignee: Paul Slauenwhite <paulslau>
Status: CLOSED WONTFIX QA Contact: Kathy Chan <kathy>
Severity: normal    
Priority: P1 CC: paulslau
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Ben Chen CLA 2010-05-25 13:43:31 EDT
Build Identifier: TPTP-4.7.0-201005222133

found while running
Test.UI.TestNavigator_bugzilla_232089

Reproducible: Always

Steps to Reproduce:
1.Create java project, and a TPTP test(say, TPTP Junit test)
2.Execute the test and specify the log to be put under the same project
3.Open both assets with system editor(winzip), check the resource file for their ids and verify that the ids are the same
Comment 1 Paul Slauenwhite CLA 2010-05-25 14:06:34 EDT
See /org.eclipse.hyades.test.ui.navigator.tests/manual/Test.UI.TestNavigator_bugzilla_232089.testsuite.
Comment 2 Paul Slauenwhite CLA 2010-05-25 14:42:26 EDT
This is caused by the execution result being created with the ID of the referring test suite.  See org.eclipse.hyades.loaders.common.ExecutionContext.ExecutionContext(TPFTestSuite, String, String, String, boolean, TPFDeployment, boolean, boolean):

rootResult = new ExecutionResultData(getTestSuite(), suite.getId());

Since this fix constitutes an API change that may impact TPTP/consuming product function and there is no apparent impact to TPTP/consuming products, we will not fix this symptom.
Comment 3 Ben Chen CLA 2010-05-25 14:44:22 EDT
(In reply to comment #2)
> This is caused by the execution result being created with the ID of the
> referring test suite.  See
> org.eclipse.hyades.loaders.common.ExecutionContext.ExecutionContext(TPFTestSuite,
> String, String, String, boolean, TPFDeployment, boolean, boolean):
> 
> rootResult = new ExecutionResultData(getTestSuite(), suite.getId());
> 
> Since this fix constitutes an API change that may impact TPTP/consuming product
> function and there is no apparent impact to TPTP/consuming products, we will
> not fix this symptom.

As such, closed