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

Bug 218550

Summary: Test Reports contain no data.
Product: z_Archived Reporter: Paul Slauenwhite <paulslau>
Component: TPTPAssignee: Paul Slauenwhite <paulslau>
Status: CLOSED FIXED QA Contact:
Severity: blocker    
Priority: P1 CC: apnan, dmorris, newboya1988
Version: unspecifiedFlags: apnan: review+
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 218578    
Bug Blocks:    
Attachments:
Description Flags
Sample project and incorrect reports.
none
JScrib Patch none

Description Paul Slauenwhite CLA 2008-02-11 15:02:24 EST
Test Reports contain no data.

Using the 4.5.0-200802100400 driver, Test Reports (JScrib or BIRT) contain no data when generating on a valid test suite and execution result.
Comment 1 Paul Slauenwhite CLA 2008-02-11 15:03:08 EST
Estimated sizing.
Comment 2 Paul Slauenwhite CLA 2008-02-11 15:03:59 EST
Created attachment 89445 [details]
Sample project and incorrect reports.
Comment 3 Paul Slauenwhite CLA 2008-02-11 17:36:12 EST
The issue with the JScrib reports is a regression caused by one of the performamnce improvments introduced in 4.5 I5 defect 190453 in the rg.eclipse.hyades.models.common.util.ExecutionUtil.findExecutionResults(List)  method (see https://bugs.eclipse.org/bugs/show_bug.cgi?id=190453#c10).  

Since this method will be rewritten or replaced under defect https://bugs.eclipse.org/bugs/show_bug.cgi?id=180469, the fix for this defect will be to back out the performamnce improvments introduced in 4.5 I5 defect 190453 in the rg.eclipse.hyades.models.common.util.ExecutionUtil.findExecutionResults(List)  method (see 'JScrib Patch').
Comment 4 Paul Slauenwhite CLA 2008-02-11 17:36:32 EST
Created attachment 89459 [details]
JScrib Patch
Comment 5 Paul Slauenwhite CLA 2008-02-11 18:06:43 EST
Alex, can you review this patch?
Comment 6 Alex Nan CLA 2008-02-11 18:34:12 EST
Patch reviewed. 
Comment 7 Paul Slauenwhite CLA 2008-02-11 18:37:04 EST
PMC appreoved and patch checked-in to CVS (HEAD).
Comment 8 Paul Slauenwhite CLA 2008-02-19 13:18:16 EST
Hours for verifying latest BIRT/DTP builds.
Comment 9 Paul Slauenwhite CLA 2008-02-20 10:16:21 EST
Moving to I6 since I5 has already been dropped to Ganymede.
Comment 10 Paul Slauenwhite CLA 2008-02-21 07:43:55 EST
Hours for verifying latest BIRT/DTP builds.
Comment 11 Paul Slauenwhite CLA 2008-02-22 15:09:20 EST
Fixed using TPTP M5 and 

birt-report-framework-2_3_0-N20080222.zip
dtp-sdk_1.6M6_20080222.zip
Comment 12 Paul Slauenwhite CLA 2008-02-22 15:09:45 EST
Closing.
Comment 13 Paul Slauenwhite CLA 2008-02-22 15:10:13 EST
(In reply to comment #6)
> Patch reviewed. 
> 

Thanks.  Please set the review flag to '+'.