Bug 472248 - JaCoCo - Expert AUT-configuration part: reset on Test Suite start checkbox state is not persisted
Summary: JaCoCo - Expert AUT-configuration part: reset on Test Suite start checkbox st...
Status: CLOSED FIXED
Alias: None
Product: Jubula (Archived)
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 blocker (vote)
Target Milestone: 4.0   Edit
Assignee: Marvin Mueller CLA
QA Contact: Oliver Goetz CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 470373
  Show dependency tree
 
Reported: 2015-07-09 03:07 EDT by Markus Tiede CLA
Modified: 2016-05-19 03:39 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Tiede CLA 2015-07-09 03:07:54 EDT
Within the expert AUT-configuration part changes to the "reset on Test Suite start" checkbox state is not persisted.
Comment 1 Bela Bara CLA 2015-12-01 04:26:50 EST
I couldn't reproduce the bug, checked on Windows and Linux too. Could you share the steps how to reproduce?
Comment 2 Alexandra Schladebeck CLA 2015-12-01 04:31:36 EST
I couldn't reproduce this in the embedded or oracle databases. Assigning to Markus for comment / steps to reproduce.
Comment 3 Markus Tiede CLA 2015-12-02 04:35:55 EST
I'm also not able to reproduce this - resolving accordingly.
Comment 4 Oliver Goetz CLA 2015-12-09 05:10:06 EST
Closed due to comment 1 - 3
Comment 5 Marvin Mueller CLA 2016-01-15 08:37:03 EST
I can reproduce this issue and i wonder why this is working sometimes and in other times not.
Looking into the code[1] is telling us that the selection state of the checkbox is never set! As a result i have found this, i will also contribute a fix for this.

[1] git.eclipse.org/c/jubula/org.eclipse.jubula.core.git/tree/org.eclipse.jubula.client.ui.rcp/src/org/eclipse/jubula/client/ui/rcp/widgets/autconfig/AutConfigComponent.java#n1551
Comment 6 Eclipse Genie CLA 2016-01-15 08:40:11 EST
New Gerrit change created: https://git.eclipse.org/r/64467
Comment 7 Alexandra Schladebeck CLA 2016-01-22 06:50:42 EST
As discussed, should be pushed to master.
Comment 9 Marvin Mueller CLA 2016-01-26 02:25:09 EST
Fixed see Comment 8
Comment 10 Oliver Goetz CLA 2016-02-23 05:44:14 EST
Verified