gre-ilya/libuv
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: skip fork fsevent cases when lacking support

Browse Source 
On platforms not supporting fsevents do not run the fork
fsevent tests.

PR-URL: https://github.com/libuv/libuv/pull/1312
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
This commit is contained in:
Brad King 2017-02-20 22:03:11 -05:00 committed by Ben Noordhuis
parent f4f1f57e7c
commit 773d678c6a
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
test/test-fork.c
View File

@ -506,17 +506,18 @@ static int _do_fork_fs_events_child(int file_or_dir) {
TEST_IMPL(fork_fs_events_child) { TEST_IMPL(fork_fs_events_child) {
#if defined(__MVS__) #if defined(NO_FS_EVENTS)
RETURN_SKIP("Filesystem watching not supported on this platform."); RETURN_SKIP(NO_FS_EVENTS);
#endif #endif
return _do_fork_fs_events_child(FS_TEST_FILE); return _do_fork_fs_events_child(FS_TEST_FILE);
} }
TEST_IMPL(fork_fs_events_child_dir) { TEST_IMPL(fork_fs_events_child_dir) {
#if defined(__MVS__) #if defined(NO_FS_EVENTS)
RETURN_SKIP("Filesystem watching not supported on this platform."); RETURN_SKIP(NO_FS_EVENTS);
#elif defined(__APPLE__) || defined (__linux__) #endif
#if defined(__APPLE__) || defined (__linux__)
return _do_fork_fs_events_child(FS_TEST_DIR); return _do_fork_fs_events_child(FS_TEST_DIR);
#else #else
/* You can't spin up a cfrunloop thread on an apple platform /* You can't spin up a cfrunloop thread on an apple platform
@ -529,9 +530,10 @@ TEST_IMPL(fork_fs_events_child_dir) {
TEST_IMPL(fork_fs_events_file_parent_child) { TEST_IMPL(fork_fs_events_file_parent_child) {
#if defined(__MVS__) #if defined(NO_FS_EVENTS)
RETURN_SKIP("Filesystem watching not supported on this platform."); RETURN_SKIP(NO_FS_EVENTS);
#elif defined(__sun) || defined(_AIX) #endif
#if defined(__sun) || defined(_AIX)
/* It's not possible to implement this without additional /* It's not possible to implement this without additional
* bookkeeping on SunOS. For AIX it is possible, but has to be * bookkeeping on SunOS. For AIX it is possible, but has to be
* written. See https://github.com/libuv/libuv/pull/846#issuecomment-287170420 * written. See https://github.com/libuv/libuv/pull/846#issuecomment-287170420