tests: exit native with error value on failure

core/lib/init.c

@@ -74,6 +74,17 @@ static void *main_trampoline(void *arg)
     }
 #endif
 
+#ifdef CPU_NATIVE
+    extern unsigned _native_retval;
+    if (!_native_retval) {
+        _native_retval = res;
+    }
+#endif
+
+    if (IS_ACTIVE(CONFIG_CORE_EXIT_WITH_MAIN)) {
+        pm_off();
+    }
+
     return NULL;
 }
 

cpu/native/native_cpu.c

@@ -166,7 +166,7 @@
         sched_run();
     }
 
     DEBUG("isr_cpu_switch_context_exit: calling setcontext(%" PRIkernel_pid ")\n\n", thread_getpid());
     /* Use intermediate cast to uintptr_t to silence -Wcast-align.
      * stacks are manually word aligned in thread_static_init() */
     ctx = (ucontext_t *)(uintptr_t)(thread_get_active()->sp);
@@ -184,8 +184,9 @@
 {
 #ifdef NATIVE_AUTO_EXIT
     if (sched_num_threads <= 1) {
+        extern unsigned _native_retval;
         DEBUG("cpu_switch_context_exit: last task has ended. exiting.\n");
-        real_exit(EXIT_SUCCESS);
+        real_exit(_native_retval);
     }
 #endif
 

sys/embunit/TestRunner.c

@@ -43,20 +43,20 @@
 static Test* root_;
 int TestRunnerHadErrors;
 
 static void TestRunner_startTest(TestListner* self,Test* test)
 {
     (void)self;
     (void)test;
     stdimpl_print(".");
 }
 
 static void TestRunner_endTest(TestListner* self,Test* test)
 {
     (void)self;
     (void)test;
 }
 
 static void TestRunner_addFailure(TestListner* self,Test* test,char* msg,int line,char* file)
 {
     (void)self;
     stdimpl_print("\n");
@@ -99,7 +99,7 @@
     Test_run(test, &result_);
 }
 
-void TestRunner_end(void)
+int TestRunner_end(void)
 {
     char buf[16];
     if (result_.failureCount) {
@@ -118,4 +118,6 @@
         stdimpl_print(buf);
         stdimpl_print(" tests)\n");
     }
+
+    return TestRunnerHadErrors;
 }

sys/include/embUnit/TestRunner.h

@@ -41,7 +41,7 @@ extern "C" {
 
 void TestRunner_start(void);
 void TestRunner_runTest(Test* test);
-void TestRunner_end(void);
+int TestRunner_end(void);
 
 extern int TestRunnerHadErrors;
 

tests/Makefile.tests_common

@@ -12,6 +12,9 @@ ifneq (,$(wildcard $(CURDIR)/tests*/.))
   endif
 endif
 
+# terminate native when the test is complete
+CFLAGS += -DNATIVE_AUTO_EXIT=1
+
 BOARD ?= native
 RIOTBASE ?= $(CURDIR)/../..
 QUIET ?= 1

tests/sys/pthread_condition_variable/Makefile

@@ -4,6 +4,4 @@ USEMODULE += posix_headers
 USEMODULE += pthread
 USEMODULE += xtimer
 
-CFLAGS += -DNATIVE_AUTO_EXIT
-
 include $(RIOTBASE)/Makefile.include

tests/unittests/Makefile

@@ -35,6 +35,7 @@ INCLUDES += -I$(RIOTBASE)/tests/unittests/common
 
 # some tests need more stack
 CFLAGS += -DTHREAD_STACKSIZE_MAIN=THREAD_STACKSIZE_LARGE
+CFLAGS += -DCONFIG_CORE_EXIT_WITH_MAIN=1
 
 # for these boards, enable asan (Address Sanitizer)
 ASAN_BOARDS ?= native native64

tests/unittests/main.c

@@ -49,7 +49,5 @@ int main(void)
 #ifndef NO_TEST_SUITES
     UNCURRY(RUN_TEST_SUITES, TEST_SUITES)
 #endif
-    TESTS_END();
-
-    return 0;
+    return TESTS_END();
 }