[Nepomuk] Review Request 112134: Tests: robustify the sandbox

Commit Hook null at kde.org
Sun Sep 1 03:13:16 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112134/
-----------------------------------------------------------

(Updated Sept. 1, 2013, 3:13 a.m.)


Status
------

This change has been marked as submitted.


Review request for Nepomuk and Vishesh Handa.


Description
-------

Tests: robustify the sandbox

Running the nepomuk tests always timed out for me. I found that this was
because the nepomukserver was failing to start. There are two problems
here:

1) if the sandbox fails to start, it should just fail, not
timeout.

2) nepomukserver should not fail to start.

This patch fixes 1) by adding return value checking to the sandbox
script; if kdeinit4 & friends fail to start the test fails instantly

2) turned out to be because the nepomukserver could not
connect to the (local) X server. To fix this, I changed the script to be
started through xvfb-run, which starts a virtual X server for it to
connect to.

Finally, start the nepomukserver we just built, rather than the
installed one.


Diffs
-----

  autotests/lib/tools/runNepomukTest.sh.in 2eac3659043e47ebdaa8555061e6a8bbbea98511 
  autotests/lib/tools/reallyRunNepomukTest.sh.in PRE-CREATION 
  autotests/lib/tools/nepomuk-sandbox-end.sh.in e331cb04cce444ab582de9f3f731821792b116f3 
  autotests/lib/tools/nepomuk-sandbox-begin.sh.in 1517dc99cd7c4247eb4e328100a4d3100979e563 
  autotests/lib/tools/CMakeLists.txt 2e249628cbe161a059887dc231184aa82e1bc6a4 

Diff: http://git.reviewboard.kde.org/r/112134/diff/


Testing
-------

Only impacts tests. Makes them not timeout for me


Thanks,

Simeon Bird

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20130901/44df7b04/attachment.html>


More information about the Nepomuk mailing list