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

Bug 213219

Summary: TPTP test results are not reported correctly.
Product: z_Archived Reporter: Paul Slauenwhite <paulslau>
Component: TPTPAssignee: jkubasta
Status: CLOSED FIXED QA Contact:
Severity: major    
Priority: P1 Keywords: plan
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Paul Slauenwhite CLA 2007-12-17 14:46:39 EST
TPTP test results are not reported correctly.

There are some cases (e.g.
Platform.Communication.New_Agent_Controller_Application_Console.Windows_IAEM64T
in http://www.eclipse.org/tptp/test/reports/4.5_i4_TP1/index.html) that the
same test execution result appears more than one in the TPTP Test Report,
therefore corrupting the test pass results.  This is caused by the execution
result referencing more than one test suite with the same ID.  This is the
result of creating test suites by copying a *.testsuite file, which does not
update the ID of the copied test suite (see
https://bugs.eclipse.org/bugs/show_bug.cgi?id=166025).

To resolve this problem, the below files require following updates by the
Project Leads (or their delegates):

1) For each pair of test suites, determine which test suite was copied.
2) Create a new test suite to replace the copied test suite, including the same
test cases and test behavior.
3) Delete the copied test suite.
4) Update the test suite(s) (e.g. AllTests.test suite) referencing the
deleted/copied test suite by removing and re-adding the test invocation of the
deleted/copied test suite.
5) Regenerate the execution results for the deleted/copied test suite for the
current test pass (e.g. 4.5 I4 TP1 or TP2).

Platform:

org.eclipse.hyades.probekit.tests\test-resources\AllTests.testsuite and
org.eclipse.hyades.probekit.tests\test-resources\AllSmokeTests.testsuite

org.eclipse.hyades.probekit.tests\test-resources\Platform.Execution.Probekit.LaunchTypes.testsuite
and
org.eclipse.hyades.probekit.tests\test-resources\Platform.Execution.Probekit.LaunchTypes.JVMPI.testsuite

org.eclipse.hyades.use.cases\gui\AllTests.testsuite and
org.eclipse.hyades.use.cases\gui\AllSmokeTests.testsuite

org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform.UI.Report.testsuite
and
org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform.UI.Log.Report.testsuite

org.eclipse.hyades.use.cases\manual\AllTests.testsuite and
org.eclipse.hyades.use.cases\manual\AllSmokeTests.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Linux_IA64.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Linux_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Linux_IAEM64T.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Linux_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Windows_IA64.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Windows_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Windows_IAEM64T.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Application_Console.Windows_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Linux_IA64.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Linux_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Linux_IAEM64T.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Linux_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Windows_IA64.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Windows_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Windows_IAEM64T.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_Basic.Windows_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Linux_IA64.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Linux_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Linux_IAEM64T.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Linux_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Windows_IA64.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Windows_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Windows_IAEM64T.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.Communication.New_Agent_Controller_File_Transfer.Windows_IA32.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.UI.LogSets.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.UI.ImportMultiLogFiles.testsuite

org.eclipse.tptp.platform.common.ui.tests\junit\LocationValidatorTest.testsuite
and
org.eclipse.hyades.test.java.tests\junit\Test.JUnit.JUnitTestGeneration.testsuite
Comment 1 Paul Slauenwhite CLA 2007-12-19 06:44:09 EST
Can you include this in the 4.5 plan (P1 priority/plan keyword)?
Comment 2 jkubasta CLA 2008-02-27 15:19:15 EST
Changes checked in for all but the following, to date:

Platform:

org.eclipse.hyades.probekit.tests\test-resources\AllTests.testsuite and
org.eclipse.hyades.probekit.tests\test-resources\AllSmokeTests.testsuite

org.eclipse.hyades.use.cases\gui\AllTests.testsuite and
org.eclipse.hyades.use.cases\gui\AllSmokeTests.testsuite

org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform.UI.Report.testsuite
and
org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform.UI.Log.Report.testsuite

org.eclipse.hyades.use.cases\manual\AllTests.testsuite and
org.eclipse.hyades.use.cases\manual\AllSmokeTests.testsuite

org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.UI.LogSets.testsuite
and
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.UI.ImportMultiLogFiles.testsuite

org.eclipse.tptp.platform.common.ui.tests\junit\LocationValidatorTest.testsuite
and
org.eclipse.hyades.test.java.tests\junit\Test.JUnit.JUnitTestGeneration.testsuite
Comment 3 jkubasta CLA 2008-03-01 22:42:29 EST
org.eclipse.hyades.probekit.tests\test-resources\AllSmokeTests.testsuite
replaced with
org.eclipse.hyades.probekit.tests\test-resources\All_Smoke_Tests.testsuite

Comment 4 jkubasta CLA 2008-03-01 23:06:53 EST
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.UI.ImportMultiLogFiles.testsuite
replaced with
org.eclipse.hyades.use.cases\manual\Profiling_and_Logging\Platform.UI.Import_Multi_LogFiles.testsuite
Comment 5 jkubasta CLA 2008-03-02 09:58:04 EST
org.eclipse.hyades.use.cases\gui\AllSmokeTests.testsuite
replaced with
org.eclipse.hyades.use.cases\gui\All_Smoke_Tests.testsuite.  Additionally, the following tests were omitted from the new file:
1. CodeCoverage
2. AllSmokeTests


Comment 6 jkubasta CLA 2008-03-02 10:48:30 EST
org.eclipse.hyades.use.cases\gui\AllSmokeTests.testsuite was replaced with
org.eclipse.hyades.use.cases\gui\All_Smoke_Tests.testsuite
Comment 7 jkubasta CLA 2008-03-02 10:49:44 EST
Paul, please tell me how I move the testcases in org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform.UI.Log.Report.testsuite into the new testcase
Comment 8 jkubasta CLA 2008-03-02 11:08:11 EST
org.eclipse.hyades.use.cases\manual\AllSmokeTests.testsuite replaced with
org.eclipse.hyades.use.cases\manual\All_Smoke_Tests.testsuite
Comment 9 jkubasta CLA 2008-03-02 11:12:35 EST
Paul, org.eclipse.hyades.test.java.tests\junit\Test.JUnit.JUnitTestGeneration.testsuite appears to be a copy of org.eclipse.tptp.platform.common.ui.tests\junit\LocationValidatorTest.testsuite
. Please update
Comment 10 Paul Slauenwhite CLA 2008-03-03 15:56:06 EST
(In reply to comment #7)
> Paul, please tell me how I move the testcases in
> org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform.UI.Log.Report.testsuite
> into the new testcase
> 

1) Create a new AGR test suite.
2) Create new test cases in that AGR test suite (note, you need to start the AGR recorder and immediately stop it to create a new AGR test case).
3) Copy the object mine.
4) Copy the test case macro.
5) Save.
Comment 11 Paul Slauenwhite CLA 2008-03-03 16:01:20 EST
(In reply to comment #9)
> Paul,
> org.eclipse.hyades.test.java.tests\junit\Test.JUnit.JUnitTestGeneration.testsuite
> appears to be a copy of
> org.eclipse.tptp.platform.common.ui.tests\junit\LocationValidatorTest.testsuite
> . Please update
> 

org.eclipse.hyades.test.java.tests\junit\Test.JUnit.JUnitTestGeneration.testsuite has since been updated. 

However, I did notice that org.eclipse.tptp.platform.common.ui.tests\junit\LocationValidatorTest.testsuite does not have an ID property.  Can you recreate the test suite again?  Note, you can import the JUnit Test class (File >> Import... >> Test >> JUnit tests to TPTP).
Comment 12 jkubasta CLA 2008-04-16 20:01:48 EDT
org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform.UI.Log.Report.testsuite replaced with org.eclipse.hyades.use.cases\gui\Profiling_and_Logging\Report\Platform_UI_Log_Report.testsuite
Comment 13 jkubasta CLA 2008-04-16 22:49:18 EDT
org.eclipse.tptp.platform.common.ui.tests\junit\LocationValidatorTest.testsuite replaced with org.eclipse.tptp.platform.common.ui.tests\junit\Location_Validator_Test.testsuite
Comment 14 jkubasta CLA 2008-04-16 22:51:13 EDT
Paul, please verify when you have a chance
Comment 15 Paul Slauenwhite CLA 2008-05-01 09:41:40 EDT
The test suites that were updated now do not appear in the I7 TP report (http://www.eclipse.org/tptp/test/reports/4.5_i7_TP1/index.html).  When a new test it created, it needs to be added to the root test suite(s) (e.g. AllTests.testsuite, AllSmoketests.testsuite) for that plug-in (see http://www.eclipse.org/tptp/home/documents/process/TPTP_Testing_Strategy.html#2.0) to be included in the test reports.   
Comment 16 jkubasta CLA 2008-05-12 19:14:18 EDT
resolved as far as I can tell. Please reopen if you disagree
Comment 17 Paul Slauenwhite CLA 2008-06-10 11:58:08 EDT
Verified in http://www.eclipse.org/tptp/test/reports/4.5_i8_TP2/index.html.  Closing.