ruuvi / ruuvi.firmware.c

Ruuvi Firmware version 3. Built on top of Nordic SDK 15, uses both Ruuvi and external repositories as submodules. In Beta, no breaking changes are intended but may be done if absolutely necessary

BSD 3-Clause "New" or "Revised" License

52 stars 39 forks source link

Add common file containing testing macros #297

Closed DG12 closed 2 years ago

DG12 commented 2 years ago

At present only contains TESTABLE_static for use with Ceedling unit testing

Replacing 3 lines:

ifdef CEEDING

static

endif

++++ As files are updated, replace those 3 lines with a prefix of TESTABLE_static (i.e. no need to change files just for this )

ojousima-jenkins commented 2 years ago

Can one of the admins verify this patch?

ojousima commented 2 years ago

ok to test

ojousima commented 2 years ago

Thanks. Please rename "TESTABLE_static" to "TESTABLE_STATIC", all uppercase in macros

ojousima commented 2 years ago

Thank you for the contribution

DG12 commented 2 years ago

Sorry there were so many things wrong, including needing to rename the file to app_testing.h Thanks for correcting them.