Set picotest_error_code in PICOTEST_CHECK_AND_ABORT (#2722)

peterharperuk · web-flow · commit 5986b8386ca7 · 2026-01-05T11:30:19.000-06:00
Fixes #2721
diff --git a/test/pico_test/include/pico/test.h b/test/pico_test/include/pico/test.h
@@ -39,11 +39,13 @@ but not sure that is implemented yet.
 
 #define PICOTEST_CHECK_AND_ABORT(COND, MESSAGE) if (!(COND)) {                                     \
                                         printf("Module %s: %s\n", picotest_module, MESSAGE);       \
+                                        picotest_error_code = -1;                                   \
                                         return -1;                                                  \
                                     }
 
 #define PICOTEST_CHECK_CHANNEL_AND_ABORT(CHANNEL, COND, MESSAGE) if (!(COND)) {                    \
                                         printf("Module %s, channel %d: %s\n", picotest_module, CHANNEL, MESSAGE);  \
+                                        picotest_error_code = -1;                                   \
                                         return -1;                                                  \
                                     }
 

Original file line number	Diff line number	Diff line change
`@@ -39,11 +39,13 @@ but not sure that is implemented yet.`
`39`	`39`
`40`	`40`	`#define PICOTEST_CHECK_AND_ABORT(COND, MESSAGE) if (!(COND)) { \`
`41`	`41`	`printf("Module %s: %s\n", picotest_module, MESSAGE); \`
	`42`	`+ picotest_error_code = -1; \`
`42`	`43`	`return -1; \`
`43`	`44`	`}`
`44`	`45`
`45`	`46`	`#define PICOTEST_CHECK_CHANNEL_AND_ABORT(CHANNEL, COND, MESSAGE) if (!(COND)) { \`
`46`	`47`	`printf("Module %s, channel %d: %s\n", picotest_module, CHANNEL, MESSAGE); \`
	`48`	`+ picotest_error_code = -1; \`
`47`	`49`	`return -1; \`
`48`	`50`	`}`
`49`	`51`