Merge pull request #14882 from LDong-Arm/storage_tests_cleanup

Unit tests: storage: Only use headers, sources and definitions needed by each test
2021-07-07 16:12:10 +01:00 · 2021-07-07 16:12:10 +01:00 · b490c67241
parent dc6dd17bb0 fcb71fb617
commit b490c67241
13 changed files with 30 additions and 60 deletions
--- a/storage/blockdevice/tests/UNITTESTS/SFDP/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/SFDP/CMakeLists.txt
@ -18,8 +18,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-platform
        gmock_main
 )
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/BufferedBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/BufferedBlockDevice/CMakeLists.txt
@ -13,8 +13,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/ChainingBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/ChainingBlockDevice/CMakeLists.txt
@ -8,14 +8,14 @@ add_executable(${TEST_NAME})
 target_sources(${TEST_NAME}
    PRIVATE
        ${mbed-os_SOURCE_DIR}/storage/blockdevice/source/ChainingBlockDevice.cpp
-        ${mbed-os_SOURCE_DIR}/storage/blockdevice/source/HeapBlockDevice.cpp
        test_ChainingBlockDevice.cpp
 )

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/ExhaustibleBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/ExhaustibleBlockDevice/CMakeLists.txt
@ -13,8 +13,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/FlashSimBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/FlashSimBlockDevice/CMakeLists.txt
@ -13,8 +13,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/HeapBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/HeapBlockDevice/CMakeLists.txt
@ -5,50 +5,6 @@ set(TEST_NAME heap-blockdevice-unittest)

 add_executable(${TEST_NAME})

-target_compile_definitions(${TEST_NAME} 
-    PRIVATE
-        MBED_CONF_FAT_CHAN_FFS_DBG=0
-        MBED_CONF_FAT_CHAN_FF_FS_READONLY=0
-        MBED_CONF_FAT_CHAN_FF_FS_MINIMIZE=0
-        MBED_CONF_FAT_CHAN_FF_USE_STRFUNC=0
-        MBED_CONF_FAT_CHAN_FF_USE_FIND=0
-        MBED_CONF_FAT_CHAN_FF_USE_MKFS=1
-        MBED_CONF_FAT_CHAN_FF_USE_FASTSEEK=0
-        MBED_CONF_FAT_CHAN_FF_USE_EXPAND=0
-        MBED_CONF_FAT_CHAN_FF_USE_CHMOD=0
-        MBED_CONF_FAT_CHAN_FF_USE_LABEL=0
-        MBED_CONF_FAT_CHAN_FF_USE_FORWARD=0
-        MBED_CONF_FAT_CHAN_FF_CODE_PAGE=437
-        MBED_CONF_FAT_CHAN_FF_USE_LFN=3
-        MBED_CONF_FAT_CHAN_FF_MAX_LFN=255
-        MBED_CONF_FAT_CHAN_FF_LFN_UNICODE=0
-        MBED_CONF_FAT_CHAN_FF_LFN_BUF=255
-        MBED_CONF_FAT_CHAN_FF_SFN_BUF=12
-        MBED_CONF_FAT_CHAN_FF_STRF_ENCODE=3
-        MBED_CONF_FAT_CHAN_FF_FS_RPATH=1
-        MBED_CONF_FAT_CHAN_FF_VOLUMES=4
-        MBED_CONF_FAT_CHAN_FF_STR_VOLUME_ID=0
-        MBED_CONF_FAT_CHAN_FF_VOLUME_STRS=\"RAM\",\"NAND\",\"CF\",\"SD\",\"SD2\",\"USB\",\"USB2\",\"USB3\"
-        MBED_CONF_FAT_CHAN_FF_MULTI_PARTITION=0
-        MBED_CONF_FAT_CHAN_FF_MIN_SS=512
-        MBED_CONF_FAT_CHAN_FF_MAX_SS=4096
-        MBED_CONF_FAT_CHAN_FF_USE_TRIM=1
-        MBED_CONF_FAT_CHAN_FF_FS_NOFSINFO=0
-        MBED_CONF_FAT_CHAN_FF_FS_TINY=1
-        MBED_CONF_FAT_CHAN_FF_FS_EXFAT=0
-        MBED_CONF_FAT_CHAN_FF_FS_HEAPBUF=1
-        MBED_CONF_FAT_CHAN_FF_FS_NORTC=0
-        MBED_CONF_FAT_CHAN_FF_NORTC_MON=1
-        MBED_CONF_FAT_CHAN_FF_NORTC_MDAY=1
-        MBED_CONF_FAT_CHAN_FF_NORTC_YEAR=2017
-        MBED_CONF_FAT_CHAN_FF_FS_LOCK=0
-        MBED_CONF_FAT_CHAN_FF_FS_REENTRANT=0
-        MBED_CONF_FAT_CHAN_FF_FS_TIMEOUT=1000
-        MBED_CONF_FAT_CHAN_FF_SYNC_t=HANDLE
-        MBED_CONF_FAT_CHAN_FLUSH_ON_NEW_CLUSTER=0
-        MBED_CONF_FAT_CHAN_FLUSH_ON_NEW_SECTOR=1
-)
-
 target_sources(${TEST_NAME}
    PRIVATE
        ${mbed-os_SOURCE_DIR}/storage/blockdevice/source/HeapBlockDevice.cpp
@ -57,8 +13,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-platform
        mbed-stubs-blockdevice
        gmock_main
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/MBRBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/MBRBlockDevice/CMakeLists.txt
@ -13,8 +13,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/ObservingBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/ObservingBlockDevice/CMakeLists.txt
@ -14,8 +14,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/ProfilingBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/ProfilingBlockDevice/CMakeLists.txt
@ -8,14 +8,14 @@ add_executable(${TEST_NAME})
 target_sources(${TEST_NAME}
    PRIVATE
        ${mbed-os_SOURCE_DIR}/storage/blockdevice/source/ProfilingBlockDevice.cpp
-        ${mbed-os_SOURCE_DIR}/storage/blockdevice/source/HeapBlockDevice.cpp
        test_ProfilingBlockDevice.cpp
 )

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/ReadOnlyBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/ReadOnlyBlockDevice/CMakeLists.txt
@ -8,14 +8,14 @@ add_executable(${TEST_NAME})
 target_sources(${TEST_NAME}
    PRIVATE
        ${mbed-os_SOURCE_DIR}/storage/blockdevice/source/ReadOnlyBlockDevice.cpp
-        ${mbed-os_SOURCE_DIR}/storage/blockdevice/source/HeapBlockDevice.cpp
        test_ReadOnlyBlockDevice.cpp
 )

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-headers
        mbed-stubs-platform
        mbed-stubs-blockdevice
--- a/storage/blockdevice/tests/UNITTESTS/blockdevice/SlicingBlockDevice/CMakeLists.txt
+++ b/storage/blockdevice/tests/UNITTESTS/blockdevice/SlicingBlockDevice/CMakeLists.txt
@ -14,8 +14,9 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-platform
        mbed-stubs-platform
        mbed-stubs-blockdevice
        gmock_main
--- a/storage/kvstore/filesystemstore/tests/UNITTESTS/FileSystemStore/CMakeLists.txt
+++ b/storage/kvstore/filesystemstore/tests/UNITTESTS/FileSystemStore/CMakeLists.txt
@ -31,10 +31,13 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-drivers
        mbed-headers-filesystem
+        mbed-headers-hal
        mbed-headers-kvstore
+        mbed-headers-platform
        mbed-stubs-platform
        mbed-stubs-filesystemstore
        gmock_main
--- a/storage/kvstore/tdbstore/tests/UNITTESTS/TDBStore/CMakeLists.txt
+++ b/storage/kvstore/tdbstore/tests/UNITTESTS/TDBStore/CMakeLists.txt
@ -15,9 +15,12 @@ target_sources(${TEST_NAME}

 target_link_libraries(${TEST_NAME}
    PRIVATE
-        mbed-headers
+        mbed-headers-base
        mbed-headers-blockdevice
+        mbed-headers-drivers
+        mbed-headers-hal
        mbed-headers-kvstore
+        mbed-headers-platform
        mbed-stubs-platform
        gmock_main
 )