tests/sys/event_thread_shared: test stack size handling

maribu · maribu · commit 37eb21325fc7 · 2025-12-09T13:51:07.000+01:00
This adds a simple integration test for declaring the stack size
requirements in modules.
diff --git a/makefiles/app_dirs.blacklist b/makefiles/app_dirs.blacklist
@@ -20,4 +20,5 @@ tests/pkg/tflite-micro/external_modules/
 tests/pkg/utensor/external_modules/
 tests/pkg/wolfcrypt-ed25519-verify/
 tests/sys/suit_manifest/native_flashpage/
+tests/sys/event_thread_shared/external_modules/
 tests/unittests/
diff --git a/tests/sys/event_thread_shared/Makefile b/tests/sys/event_thread_shared/Makefile
@@ -2,4 +2,12 @@ include ../Makefile.sys_common
 
 USEMODULE += event_thread
 
+# Test that selecting event thread size works from the modules `Makefile.dep`
+# work as expected by including a bunch of empty test module
+EXTERNAL_MODULE_DIRS += external_modules
+USEMODULE += test_module_a
+USEMODULE += test_module_b
+USEMODULE += test_module_c
+USEMODULE += test_module_d
+
 include $(RIOTBASE)/Makefile.include
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_a/Makefile b/tests/sys/event_thread_shared/external_modules/test_module_a/Makefile
@@ -0,0 +1 @@
+include $(RIOTBASE)/Makefile.base
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_a/Makefile.dep b/tests/sys/event_thread_shared/external_modules/test_module_a/Makefile.dep
@@ -0,0 +1 @@
+EVENT_THREAD_HIGHEST_STACKSIZE_MIN += 3
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_b/Makefile b/tests/sys/event_thread_shared/external_modules/test_module_b/Makefile
@@ -0,0 +1 @@
+include $(RIOTBASE)/Makefile.base
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_b/Makefile.dep b/tests/sys/event_thread_shared/external_modules/test_module_b/Makefile.dep
@@ -0,0 +1 @@
+EVENT_THREAD_HIGHEST_STACKSIZE_MIN += 300
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_c/Makefile b/tests/sys/event_thread_shared/external_modules/test_module_c/Makefile
@@ -0,0 +1 @@
+include $(RIOTBASE)/Makefile.base
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_c/Makefile.dep b/tests/sys/event_thread_shared/external_modules/test_module_c/Makefile.dep
@@ -0,0 +1 @@
+EVENT_THREAD_HIGHEST_STACKSIZE_MIN += 567
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_d/Makefile b/tests/sys/event_thread_shared/external_modules/test_module_d/Makefile
@@ -0,0 +1 @@
+include $(RIOTBASE)/Makefile.base
diff --git a/tests/sys/event_thread_shared/external_modules/test_module_d/Makefile.dep b/tests/sys/event_thread_shared/external_modules/test_module_d/Makefile.dep
@@ -0,0 +1 @@
+EVENT_THREAD_HIGHEST_STACKSIZE_MIN += 256
diff --git a/tests/sys/event_thread_shared/main.c b/tests/sys/event_thread_shared/main.c
@@ -17,11 +17,17 @@
  * @}
  */
 
+#include <assert.h>
 #include <stdio.h>
 
 #include "thread.h"
 #include "event/thread.h"
 
+static_assert(EVENT_THREAD_LOWEST_STACKSIZE == 567,
+              "Selecting highest of the minimum stack size requirements "
+              "declared via `EVENT_THREAD_HIGHEST_STACKSIZE_MIN += <num>` "
+              "must work correctly");
+
 static void _handler_high(event_t *event) {
     (void)event;
     puts("high");

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+EVENT_THREAD_HIGHEST_STACKSIZE_MIN += 3`