Commit ac429148 authored by Sebastien Robin's avatar Sebastien Robin

disable xdpyinfo for now since it is not installed

parent caba9c9d
...@@ -81,11 +81,18 @@ class Xvfb: ...@@ -81,11 +81,18 @@ class Xvfb:
stdout=null, stderr=null, close_fds=True) stdout=null, stderr=null, close_fds=True)
# try to check if X screen is available # try to check if X screen is available
time.sleep(5) time.sleep(5)
return
# XXX xdpyinfo is not installed yet. Is this checking really needed ?
# If it is required, we have to make xdpyinfo available as part of
# selinum runner and make testnode using it (testnode use the software
# of selenium runner to launch firefox and Xvfb)
"""
if subprocess.call(('xdpyinfo', '-display', display), if subprocess.call(('xdpyinfo', '-display', display),
stdout=null, stderr=subprocess.STDOUT): stdout=null, stderr=subprocess.STDOUT):
# Xvfb did not start properly so stop here # Xvfb did not start properly so stop here
raise EnvironmentError("Can not start Xvfb, stop test execution " \ raise EnvironmentError("Can not start Xvfb, stop test execution " \
+ "(display %r)" % (display,)) + "(display %r)" % (display,))
"""
def run(self): def run(self):
for display_try in self.display_list: for display_try in self.display_list:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment