diff options
author | Guy Harris <guy@alum.mit.edu> | 2013-09-22 21:51:05 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2013-09-22 21:51:05 +0000 |
commit | 800b45f0a988dcd694b771396fcbd617075d3493 (patch) | |
tree | eb843d42c75e31de17eb40ff907a7180870d0927 /test/suite-capture.sh | |
parent | d158983d7e246ef47035143878f7614d181cb794 (diff) |
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
Diffstat (limited to 'test/suite-capture.sh')
-rwxr-xr-x | test/suite-capture.sh | 17 |
1 files changed, 17 insertions, 0 deletions
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! |