Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 367422 - Canceling the TestExecutionJob can cause the client to freeze
Summary: Canceling the TestExecutionJob can cause the client to freeze
Status: CLOSED FIXED
Alias: None
Product: Jubula
Classification: Technology
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: Juno M5   Edit
Assignee: Markus Tiede CLA
QA Contact: Oliver Goetz CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-22 09:35 EST by Marc Steffens CLA
Modified: 2012-01-12 10:52 EST (History)
1 user (show)

See Also:

Attachments
Prevents Jubula client from freezing while canceling the TestExecutionJob (1.24 KB, patch)
2011-12-22 09:35 EST, Marc Steffens CLA 		markus.tiede: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Steffens CLA 2011-12-22 09:35:59 EST 
Created attachment 208733 [details]
Prevents Jubula client from freezing while canceling the TestExecutionJob

The current job handling of the TestExecutionJob in ClientTest can cause the client to freeze, if the CollectionMonitoringDataJob is running and the TestExecutionJob is canceled by the user. 

The CollectionMonitoringDataJob runs within the TestExecutionJob. The current implementation will cancel the TestExecutionJob, while the inner job is still running. This causes the client to freeze.

The attached patch will fix this problem. The user can still cancel the TestExecutionJob, but the client will wait for the CollectionMonitoringDataJob to finish. If the user is also canceling the CollectionMonitoringDataJob, then both jobs will be canceled and stopped correctly.
Comment 1 Marc Steffens CLA 2012-01-04 06:46:31 EST 
the corresponding GUIdancer bugzilla entry can be found under: https://bxapps.bredex.de/bugzilla/show_bug.cgi?id=68
Comment 2 Markus Tiede CLA 2012-01-09 07:41:27 EST 
The fix has been applied with changeset

http://git.eclipse.org/c/jubula/org.eclipse.jubula.core.git/commit/?id=16339a01b19424b22c64d3d4b91ab4c9e8373d80
Comment 3 Oliver Goetz CLA 2012-01-12 10:52:33 EST 
Fix verified manually.