mirror of https://github.com/ARMmbed/mbed-os.git
				
				
				
			rtos: Remove UNITTEST macro from rtos
Production code should not contain any test-specific checks. Rather than checking `UNITTEST`, unit tests can directly set `MBED_CONF_RTOS_PRESENT=1` to make RTOS available for testing. Note: The cellular ATHandler test also has `MBED_CONF_RTOS_PRESENT=1` added because `ATHandler.cpp` contains a check of this variable.pull/14884/head
							parent
							
								
									8d3fc33965
								
							
						
					
					
						commit
						3a24305916
					
				| 
						 | 
				
			
			@ -13,6 +13,7 @@ target_compile_definitions(${TEST_NAME}
 | 
			
		|||
        DEVICE_INTERRUPTIN=1
 | 
			
		||||
        MBED_CONF_PLATFORM_DEFAULT_SERIAL_BAUD_RATE=115200
 | 
			
		||||
        MBED_CONF_CELLULAR_AT_HANDLER_BUFFER_SIZE=32
 | 
			
		||||
        MBED_CONF_RTOS_PRESENT=1
 | 
			
		||||
)
 | 
			
		||||
 | 
			
		||||
target_sources(${TEST_NAME}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -33,7 +33,7 @@
 | 
			
		|||
#include "rtos/Semaphore.h"
 | 
			
		||||
#include "rtos/Mutex.h"
 | 
			
		||||
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT || defined(DOXYGEN_ONLY) || defined(UNITTEST)
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT || defined(DOXYGEN_ONLY)
 | 
			
		||||
 | 
			
		||||
namespace rtos {
 | 
			
		||||
/** \addtogroup rtos-public-api */
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -17,7 +17,7 @@
 | 
			
		|||
#ifndef MBED_RTOS_RTX1_TYPES_H
 | 
			
		||||
#define MBED_RTOS_RTX1_TYPES_H
 | 
			
		||||
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT || defined(UNITTEST)
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT
 | 
			
		||||
 | 
			
		||||
#include "cmsis_os.h"
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -17,7 +17,7 @@
 | 
			
		|||
#ifndef MBED_RTOS_STORAGE_H
 | 
			
		||||
#define MBED_RTOS_STORAGE_H
 | 
			
		||||
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT || defined(UNITTEST)
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT
 | 
			
		||||
 | 
			
		||||
#include "mbed_rtx_storage.h"
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -17,7 +17,7 @@
 | 
			
		|||
#ifndef RTOS_TYPES_H_
 | 
			
		||||
#define RTOS_TYPES_H_
 | 
			
		||||
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT || defined(DOXYGEN_ONLY) || defined(UNITTEST)
 | 
			
		||||
#if MBED_CONF_RTOS_PRESENT || defined(DOXYGEN_ONLY)
 | 
			
		||||
#include "cmsis_os2.h"
 | 
			
		||||
#else
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -18,7 +18,10 @@ target_include_directories(mbed-stubs-rtos-headers
 | 
			
		|||
 | 
			
		||||
add_library(mbed-stubs-rtos)
 | 
			
		||||
 | 
			
		||||
add_definitions(-DUNITTEST)
 | 
			
		||||
target_compile_definitions(mbed-stubs-rtos
 | 
			
		||||
    PRIVATE
 | 
			
		||||
        MBED_CONF_RTOS_PRESENT=1
 | 
			
		||||
)
 | 
			
		||||
 | 
			
		||||
target_sources(mbed-stubs-rtos
 | 
			
		||||
    PRIVATE
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue