Bug 136186 - Test invocation reference does not work with the automated gui recorder
Summary: Test invocation reference does not work with the automated gui recorder
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P1 critical (vote)
Target Milestone: ---   Edit
Assignee: Liz Dancy CLA
QA Contact:
URL:
Whiteboard:
Keywords: plan
Depends on:
Blocks: 199852 296853
  Show dependency tree
 
Reported: 2006-04-11 16:09 EDT by amehrega CLA
Modified: 2016-05-05 10:28 EDT (History)
5 users (show)

See Also:
paulslau: review+


Attachments
Patch (8.91 KB, patch)
2007-05-25 12:43 EDT, Liz Dancy CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description amehrega CLA 2006-04-11 16:09:29 EDT
- Create a test suite with a test case
- Create a second test suite with another test case
- Create a third test suite that references the first and second test suite

When the third test suite is invoked it's expected for the runner to walk throught the behavior of the first and second test suite and invoke their test cases.
Comment 1 amehrega CLA 2006-10-09 14:17:11 EDT
Planning to defer to post 4.3
Comment 2 amehrega CLA 2006-10-20 15:41:46 EDT
Deferring to 4.4 according to Paul’s recent e-mail.
Comment 3 Paul Slauenwhite CLA 2007-01-15 11:32:56 EST
Updating original estimate based information provided by Liz.
Comment 4 Paul Slauenwhite CLA 2007-01-26 08:28:33 EST
Targeting to future since cannot be contained in 4.4.
Comment 5 Paul Slauenwhite CLA 2007-01-26 08:33:30 EST
Targeting to future since cannot be contained in 4.4.
Comment 6 Paul Slauenwhite CLA 2007-01-26 08:59:18 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 7 Paul Slauenwhite CLA 2007-01-26 09:00:57 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 8 Paul Slauenwhite CLA 2007-01-26 09:03:16 EST
Correction:  The Test Project mailing list is tptp-testing-tools-dev@eclipse.org.
Comment 9 Liz Dancy CLA 2007-02-05 12:31:32 EST
After further consideration, this is being included in the 4.4 plan. I have changed the target and priority to reflect this.
Comment 10 Liz Dancy CLA 2007-03-14 16:10:16 EDT
Updating target to 4.4i3 as 4.4i2 ends 03/16/07.
Comment 11 Liz Dancy CLA 2007-04-02 18:36:02 EDT
Cindy, is this what you were experiencing today? If so, please note that it is included in the 4.4 plan and so it should be completed in i3. Hopefully the workaround solved your problem from tonight.
Comment 12 Paul Slauenwhite CLA 2007-05-10 07:32:34 EDT
Increasing severity since this defect is impacting our users and internal test bucket.
Comment 13 Liz Dancy CLA 2007-05-22 09:27:11 EDT
Updated sizing as this was never implemented in the AGR.
Comment 14 Liz Dancy CLA 2007-05-25 12:39:40 EDT
Added test case 'TestSuiteReference' to the test suite 'Test.UI.AutoGUIRunner' in HEAD. The test suite covers the case outlined in the original description.
Comment 15 Liz Dancy CLA 2007-05-25 12:43:43 EDT
Created attachment 68797 [details]
Patch

Paul, please review this patch. Note that you need to include external Object mines when adding an external Test Suite so that the Object IDs can be found.
Comment 16 Liz Dancy CLA 2007-05-25 12:45:08 EDT
Updated hours worked. This took a lot of testing since there are so many cases to consider (Loops, combinations of test cases and suites etc.)
Comment 17 Paul Slauenwhite CLA 2007-05-25 13:30:12 EDT
(In reply to comment #15)
> Created an attachment (id=68797) [details]
> Patch
> 
> Paul, please review this patch. Note that you need to include external Object
> mines when adding an external Test Suite so that the Object IDs can be found. 
> 

Patch reviewed.

Please make a request to the Test Project mailing list for this defect. 

Once the fix is integrated, please add a comment to this defect containing the build ID of the driver used to verify the fix before marking the defect as fixed.

Please make sure the copyright dates are incremented, if necessary.


Comment 18 Paul Slauenwhite CLA 2007-05-25 13:49:33 EDT
(In reply to comment #15)
> Note that you need to include external Object
> mines when adding an external Test Suite so that the Object IDs can be found. 
> 
Would not the object minds be referenced by the invoked test suite and thus could reside in the invoked test suite?

Comment 19 Liz Dancy CLA 2007-05-25 14:47:57 EDT
It treats the Test suite invocation as an ExtendedTestInvocation which is associated with a 'Test' that can be either a case or a suite. I haven't pulled in Object mines automatically because of the potential for conflicts.
Comment 20 Liz Dancy CLA 2007-05-28 08:53:25 EDT
Patch checked in to HEAD. I will verify the fix in the next build before closing.
Comment 21 Liz Dancy CLA 2007-05-30 09:29:20 EDT
Fix verified in May29 build. Closing.
Comment 22 Paul Slauenwhite CLA 2007-06-02 14:52:24 EDT
Reporter: Please verify and close in preparation for shutting down the TPTP 4.4 release.  Thanks.
Comment 23 Liz Dancy CLA 2007-06-12 10:51:02 EDT
Verified in the 4.4i4 TP2 driver.
Comment 24 Liz Dancy CLA 2007-06-12 10:51:13 EDT
Closing
Comment 25 Paul Slauenwhite CLA 2008-11-27 16:32:07 EST
(In reply to comment #17)
> (In reply to comment #15)
> > Created an attachment (id=68797) [details] [details]
> > Patch
> > 
> > Paul, please review this patch. Note that you need to include external Object
> > mines when adding an external Test Suite so that the Object IDs can be found. 
> > 
> 
> Patch reviewed.

Marking as reviewed.