From 800b45f0a988dcd694b771396fcbd617075d3493 Mon Sep 17 00:00:00 2001 From: Guy Harris Date: Sun, 22 Sep 2013 21:51:05 +0000 Subject: From Peter Wu: on systems other than Windows, skip the GUI tests if the DISPLAY environment variable isn't set. From me: note that this won't be the right thing to do on OS X if we switch to a toolkit that doesn't use X11, and that this may or may not be the right thing to do with toolkits using Wayland or Mir directly. svn path=/trunk/; revision=52182 --- test/suite-capture.sh | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) (limited to 'test/suite-capture.sh') diff --git a/test/suite-capture.sh b/test/suite-capture.sh index 1a7b4f2bdf..773b73b039 100755 --- a/test/suite-capture.sh +++ b/test/suite-capture.sh @@ -413,6 +413,23 @@ capture_step_snapshot() { wireshark_capture_suite() { # k: start capture immediately # WIRESHARK_QUIT_AFTER_CAPTURE needs to be set. + + # + # NOTE: if, on OS X, we start using a native-Quartz toolkit, + # this would need to change to check for WS_SYSTEM being + # "Darwin" and, if it is, check whether the standard output + # of "launchctl managername" is "Aqua". + # + # This may not do the right thing if we use toolkits that + # use Wayland or Mir directly, unless they also depend on + # the DISPLAY environment variable. + # + if [[ $WS_SYSTEM != Windows ]] && [ -z "$DISPLAY" ]; then + echo -n ' (X server not available)' + test_step_skipped + return + fi + DUT="$WIRESHARK_CMD" test_step_add "Capture 10 packets" capture_step_10packets # piping to stdout doesn't work with Wireshark and capturing! -- cgit v1.2.3