[ovs-dev] [PATCH v2 3/7] test-util: Changes for 'assert' test on Windows.

[Gurucharan Shetty]

There is no 'kill -l' type functionality available on Windows.
So instead of looking for the string 'ABRT', check for the exit
code in both platforms. On msys (unit test environment), it is 9
and on Linux, it is 134 (SIGABRT + 128).

On Windows, stderr is fully buffered if connected to a pipe.
Make it _IONBF so that an abort does not miss log contents.

Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
---
v1-v2:
  * Check for exit status instead of 'kill -l' for all platforms.
  * For Windows, make stderr _IONBF.
---
 tests/library.at  |   11 ++++++++---
 tests/test-util.c |    3 +++
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/tests/library.at b/tests/library.at
index a94ee5c..a5dfd82 100644
--- a/tests/library.at
+++ b/tests/library.at
@@ -175,9 +175,14 @@ AT_CLEANUP
 
 AT_SETUP([ovs_assert])
 OVS_LOGDIR=`pwd`; export OVS_LOGDIR
-AT_CHECK([ovstest test-util -voff -vfile:info '-vPATTERN:file:%c|%p|%m' --log-file assert || kill -l $?],
-  [0], [ABRT
-], [stderr])
+if test "$IS_WIN32" = "yes"; then
+  exit_status=9
+else
+  # SIGABRT + 128
+  exit_status=134
+fi
+AT_CHECK([ovstest test-util -voff -vfile:info '-vPATTERN:file:%c|%p|%m' --log-file assert],
+  [$exit_status], [], [stderr])
 
 AT_CHECK([sed 's/\(opened log file\) .*/\1/
 s/|[[^|]]*: /|/' test-util.log], [0], [dnl
diff --git a/tests/test-util.c b/tests/test-util.c
index ffd4dce..093ee9c 100644
--- a/tests/test-util.c
+++ b/tests/test-util.c
@@ -1086,6 +1086,9 @@ test_util_main(int argc, char *argv[])
 {
     set_program_name(argv[0]);
     parse_options(argc, argv);
+#ifdef _WIN32
+    setvbuf(stderr, NULL, _IONBF, 0);
+#endif
     run_command(argc - optind, argv + optind, commands);
 }
 
-- 
1.7.9.5