Bug 300016

Summary: kaddressbook thememanagertest hangs
Product: kaddressbook Reporter: Michael Palimaka <kensington>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: johu, tokoe
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Backtrace after killing hanging test

Description Michael Palimaka 2012-05-14 16:04:42 UTC
thememanagertest in grantlee/tests hangs.

Reproducible: Always

Steps to Reproduce:
1. cmake . -DKDE4_BUILD_TESTS=true
2. make -j5
3. ctest



Firstly, the themepath is hard-coded to /mnt/archive/src/kde-trunk/src/kdepim/kaddressbook/grantlee/tests/themes/.

Then, a couple of comparisons fail, where the 'actual' result is correct.

Finally, the test hangs.
Comment 1 Michael Palimaka 2012-05-14 16:10:22 UTC
Created attachment 71093 [details]
Backtrace after killing hanging test
Comment 2 Christophe Giboudeaux 2012-05-14 22:15:44 UTC
Git commit af701cd232ed03c69d9a0952b7a89a20e2380101 by Christophe Giboudeaux.
Committed on 15/05/2012 at 00:14.
Pushed by cgiboudeaux into branch 'master'.

Fix hanging test

M  +9    -6    kaddressbook/grantlee/tests/thememanagertest.cpp

http://commits.kde.org/kdepim/af701cd232ed03c69d9a0952b7a89a20e2380101
Comment 3 Michael Palimaka 2012-05-15 16:42:53 UTC
Is it OK if I backport that commit to KDE/4.8?
Comment 4 Christophe Giboudeaux 2012-05-15 19:56:33 UTC
Git commit 1c838ba3d762efd2323a3e0b7298de08acc4238e by Christophe Giboudeaux.
Committed on 15/05/2012 at 00:14.
Pushed by cgiboudeaux into branch 'KDE/4.8'.

Fix hanging test
(cherry picked from commit af701cd232ed03c69d9a0952b7a89a20e2380101)

M  +9    -6    kaddressbook/grantlee/tests/thememanagertest.cpp

http://commits.kde.org/kdepim/1c838ba3d762efd2323a3e0b7298de08acc4238e
Comment 5 Christophe Giboudeaux 2012-05-15 19:56:44 UTC
Voila