From 55245b99cc4e7992f34244b773eeb510f3af8904 Mon Sep 17 00:00:00 2001
From: David Robillard <d@drobilla.net>
Date: Mon, 10 Jun 2024 09:25:23 -0400
Subject: Fix build and tests with optional X11 features disabled

---
 test/meson.build       | 34 ++++++++++++++++++----------------
 test/test_cursor.c     |  4 +++-
 test/test_gl_hints.c   |  3 ++-
 test/test_stub_hints.c |  3 ++-
 4 files changed, 25 insertions(+), 19 deletions(-)

(limited to 'test')

diff --git a/test/meson.build b/test/meson.build
index f8d2fd1..791cf04 100644
--- a/test/meson.build
+++ b/test/meson.build
@@ -39,10 +39,7 @@ if not meson.is_subproject() and get_option('lint')
   endif
 endif
 
-basic_exclusive_tests = [
-  'local_copy_paste',
-  'remote_copy_paste',
-]
+basic_exclusive_tests = []
 
 basic_tests = [
   'cursor',
@@ -53,7 +50,6 @@ basic_tests = [
   'strerror',
   'stub',
   'stub_hints',
-  'timer',
   'update',
   'view',
   'world',
@@ -69,6 +65,11 @@ gl_tests = [
 
 vulkan_tests = ['vulkan']
 
+if with_timers
+  basic_exclusive_tests += ['local_copy_paste', 'remote_copy_paste']
+  basic_tests += ['timer']
+endif
+
 includes = [
   '.',
   '../include',
@@ -152,7 +153,7 @@ if vulkan_dep.found()
   endforeach
 endif
 
-unified_args = []
+unified_args = core_args
 unified_deps = [core_deps]
 if cairo_dep.found()
   unified_args += ['-DWITH_CAIRO']
@@ -174,9 +175,9 @@ if host_machine.system() == 'darwin'
 
   objcpp = meson.get_compiler('objcpp')
 
-  objcpp_args = []
+  objcpp_unified_args = unified_args
   if objcpp.get_id() == 'clang'
-    objcpp_args += [
+    objcpp_unified_args += [
       '-Wno-c++98-compat',
       '-Wno-c++98-compat-pedantic',
       '-Wno-deprecated-declarations',
@@ -184,7 +185,7 @@ if host_machine.system() == 'darwin'
     ]
   endif
 
-  unified_args += objcpp.get_supported_arguments(test_cpp_args + objcpp_args)
+  objcpp_unified_args = objcpp.get_supported_arguments(objcpp_unified_args)
 
   test(
     'inline_objcpp',
@@ -193,23 +194,22 @@ if host_machine.system() == 'darwin'
       'test_inline_objcpp.mm',
       dependencies: unified_deps,
       include_directories: include_directories(includes),
-      objcpp_args: unified_args,
+      objcpp_args: objcpp_unified_args,
     ),
     suite: 'unit',
   )
 
 elif is_variable('cpp')
-  unified_args = []
-
+  cpp_unified_args = unified_args
   if cpp.get_id() == 'clang'
-    unified_args += [
+    cpp_unified_args += [
       '-Wno-old-style-cast',
       '-Wno-switch-default',
       '-Wno-switch-enum',
       '-Wno-unused-macros', # Mac
     ]
   elif cpp.get_id() == 'gcc'
-    unified_args += [
+    cpp_unified_args += [
       '-Wno-conditionally-supported',
       '-Wno-old-style-cast',
       '-Wno-switch-default',
@@ -217,17 +217,19 @@ elif is_variable('cpp')
       '-Wno-useless-cast',
     ]
   elif cpp.get_id() == 'msvc'
-    unified_args += [
+    cpp_unified_args += [
       '/wd4464', # relative include path contains '..'
     ]
   endif
 
+  cpp_unified_args = cpp.get_supported_arguments(cpp_unified_args)
+
   test(
     'inline_cpp',
     executable(
       'test_inline_cpp',
       'test_inline_cpp.cpp',
-      cpp_args: test_cpp_args + unified_args,
+      cpp_args: cpp_unified_args,
       dependencies: unified_deps,
       include_directories: include_directories(includes),
     ),
diff --git a/test/test_cursor.c b/test/test_cursor.c
index 11a52d7..9b0fa66 100644
--- a/test/test_cursor.c
+++ b/test/test_cursor.c
@@ -63,7 +63,9 @@ main(int argc, char** argv)
   // Change the cursor, updating each time
   assert(puglSetCursor(test.view, (PuglCursor)-1));
   for (unsigned i = 0; i < (unsigned)PUGL_CURSOR_ALL_SCROLL; ++i) {
-    assert(!puglSetCursor(test.view, (PuglCursor)i));
+    const PuglStatus st = puglSetCursor(test.view, (PuglCursor)i);
+
+    assert(!st || st == PUGL_UNSUPPORTED);
     assert(!puglUpdate(test.world, 0.1));
   }
 
diff --git a/test/test_gl_hints.c b/test/test_gl_hints.c
index 16b51c5..250136a 100644
--- a/test/test_gl_hints.c
+++ b/test/test_gl_hints.c
@@ -85,7 +85,8 @@ main(void)
   assert(puglGetViewHint(view, PUGL_SWAP_INTERVAL) != PUGL_DONT_CARE);
   assert(puglGetViewHint(view, PUGL_RESIZABLE) != PUGL_DONT_CARE);
   assert(puglGetViewHint(view, PUGL_IGNORE_KEY_REPEAT) != PUGL_DONT_CARE);
-  assert(puglGetViewHint(view, PUGL_REFRESH_RATE) != PUGL_DONT_CARE);
+  assert(puglGetViewHint(view, PUGL_REFRESH_RATE) == PUGL_DONT_CARE ||
+         puglGetViewHint(view, PUGL_REFRESH_RATE) > 0);
 
   // Tear down
   puglFreeView(view);
diff --git a/test/test_stub_hints.c b/test/test_stub_hints.c
index 4c9d0c4..b7d54e3 100644
--- a/test/test_stub_hints.c
+++ b/test/test_stub_hints.c
@@ -77,7 +77,8 @@ main(void)
   assert(puglGetViewHint(view, PUGL_ALPHA_BITS) != PUGL_DONT_CARE);
   assert(puglGetViewHint(view, PUGL_RESIZABLE) != PUGL_DONT_CARE);
   assert(puglGetViewHint(view, PUGL_IGNORE_KEY_REPEAT) != PUGL_DONT_CARE);
-  assert(puglGetViewHint(view, PUGL_REFRESH_RATE) != PUGL_DONT_CARE);
+  assert(puglGetViewHint(view, PUGL_REFRESH_RATE) == PUGL_DONT_CARE ||
+         puglGetViewHint(view, PUGL_REFRESH_RATE) > 0);
 
   // Tear down
   puglFreeView(view);
-- 
cgit v1.2.1