Stroage unittest directory restructure

rajkan01 · rajkan01 · commit f407d6c95a79 · 2020-07-06T18:38:42.000+01:00
- Move UNITTESTS/features/storage to UNITTESTS/storage
- update the .cmake files to refer the new directory
diff --git a/UNITTESTS/CMakeLists.txt b/UNITTESTS/CMakeLists.txt
@@ -113,6 +113,7 @@ set(unittest-includes-base
   "${PROJECT_SOURCE_DIR}/../features"
   "${PROJECT_SOURCE_DIR}/../features/netsocket"
   "${PROJECT_SOURCE_DIR}/../platform"
+  "${PROJECT_SOURCE_DIR}/../storage"
   "${PROJECT_SOURCE_DIR}/../drivers"
   "${PROJECT_SOURCE_DIR}/../hal"
   "${PROJECT_SOURCE_DIR}/../events"
@@ -144,7 +145,7 @@ set(unittest-includes-base
   "${PROJECT_SOURCE_DIR}/../features/lorawan/system"
   "${PROJECT_SOURCE_DIR}/../features/mbedtls"
   "${PROJECT_SOURCE_DIR}/../features/mbedtls/inc"
-  "${PROJECT_SOURCE_DIR}/../features/storage/kvstore/conf"
+  "${PROJECT_SOURCE_DIR}/../storage/kvstore/conf"
 )
 
 # Create a list for test suites.
diff --git a/UNITTESTS/empty_baseline/unittest.cmake b/UNITTESTS/empty_baseline/unittest.cmake
@@ -13,33 +13,33 @@ set(unittest-includes ${unittest-includes}
 
 set(unittest-sources
     ../features/device_key/source/DeviceKey.cpp
-    ../components/storage/blockdevice/COMPONENT_SPIF/SPIFBlockDevice.cpp
-    ../components/storage/blockdevice/COMPONENT_I2CEE/I2CEEBlockDevice.cpp
-    ../components/storage/blockdevice/COMPONENT_DATAFLASH/DataFlashBlockDevice.cpp
-    ../components/storage/blockdevice/COMPONENT_FLASHIAP/FlashIAPBlockDevice.cpp
-    ../components/storage/blockdevice/COMPONENT_SD/SDBlockDevice.cpp
-    ../components/storage/blockdevice/COMPONENT_QSPIF/QSPIFBlockDevice.cpp
-    ../features/storage/filesystem/Dir.cpp
-    ../features/storage/filesystem/FileSystem.cpp
-    ../features/storage/filesystem/File.cpp
-    ../features/storage/kvstore/global_api/kvstore_global_api.cpp
-    ../features/storage/kvstore/securestore/SecureStore.cpp
-    ../features/storage/kvstore/kv_map/KVMap.cpp
-    ../features/storage/kvstore/tdbstore/TDBStore.cpp
-    ../features/storage/kvstore/direct_access_devicekey/DirectAccessDevicekey.cpp
-    ../features/storage/kvstore/conf/kv_config.cpp
-    ../features/storage/kvstore/filesystemstore/FileSystemStore.cpp
-    ../features/storage/system_storage/SystemStorage.cpp
-    ../features/storage/blockdevice/ChainingBlockDevice.cpp
-    ../features/storage/blockdevice/ReadOnlyBlockDevice.cpp
-    ../features/storage/blockdevice/SlicingBlockDevice.cpp
-    ../features/storage/blockdevice/MBRBlockDevice.cpp
-    ../features/storage/blockdevice/HeapBlockDevice.cpp
-    ../features/storage/blockdevice/FlashSimBlockDevice.cpp
-    ../features/storage/blockdevice/ObservingBlockDevice.cpp
-    ../features/storage/blockdevice/ProfilingBlockDevice.cpp
-    ../features/storage/blockdevice/BufferedBlockDevice.cpp
-    ../features/storage/blockdevice/ExhaustibleBlockDevice.cpp
+    ../storage/blockdevice/COMPONENT_SPIF/SPIFBlockDevice.cpp
+    ../storage/blockdevice/COMPONENT_I2CEE/I2CEEBlockDevice.cpp
+    ../storage/blockdevice/COMPONENT_DATAFLASH/DataFlashBlockDevice.cpp
+    ../storage/blockdevice/COMPONENT_FLASHIAP/FlashIAPBlockDevice.cpp
+    ../storage/blockdevice/COMPONENT_SD/SDBlockDevice.cpp
+    ../storage/blockdevice/COMPONENT_QSPIF/QSPIFBlockDevice.cpp
+    ../storage/filesystem/Dir.cpp
+    ../storage/filesystem/FileSystem.cpp
+    ../storage/filesystem/File.cpp
+    ../storage/kvstore/global_api/kvstore_global_api.cpp
+    ../storage/kvstore/securestore/SecureStore.cpp
+    ../storage/kvstore/kv_map/KVMap.cpp
+    ../storage/kvstore/tdbstore/TDBStore.cpp
+    ../storage/kvstore/direct_access_devicekey/DirectAccessDevicekey.cpp
+    ../storage/kvstore/conf/kv_config.cpp
+    ../storage/kvstore/filesystemstore/FileSystemStore.cpp
+    ../storage/system_storage/SystemStorage.cpp
+    ../storage/blockdevice/ChainingBlockDevice.cpp
+    ../storage/blockdevice/ReadOnlyBlockDevice.cpp
+    ../storage/blockdevice/SlicingBlockDevice.cpp
+    ../storage/blockdevice/MBRBlockDevice.cpp
+    ../storage/blockdevice/HeapBlockDevice.cpp
+    ../storage/blockdevice/FlashSimBlockDevice.cpp
+    ../storage/blockdevice/ObservingBlockDevice.cpp
+    ../storage/blockdevice/ProfilingBlockDevice.cpp
+    ../storage/blockdevice/BufferedBlockDevice.cpp
+    ../storage/blockdevice/ExhaustibleBlockDevice.cpp
 )
 
 set(unittest-test-sources
diff --git a/UNITTESTS/features/storage/blockdevice/ChainingBlockDevice/unittest.cmake b/UNITTESTS/features/storage/blockdevice/ChainingBlockDevice/unittest.cmake
diff --git a/UNITTESTS/moduletests/storage/blockdevice/SlicingBlockDevice/moduletest.cpp b/UNITTESTS/moduletests/storage/blockdevice/SlicingBlockDevice/moduletest.cpp
@@ -15,8 +15,8 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/HeapBlockDevice.h"
-#include "features/storage/blockdevice/SlicingBlockDevice.h"
+#include "storage/blockdevice/HeapBlockDevice.h"
+#include "storage/blockdevice/SlicingBlockDevice.h"
 
 #define BLOCK_SIZE (512)
 #define DEVICE_SIZE (BLOCK_SIZE*10)
diff --git a/UNITTESTS/moduletests/storage/blockdevice/SlicingBlockDevice/unittest.cmake b/UNITTESTS/moduletests/storage/blockdevice/SlicingBlockDevice/unittest.cmake
@@ -9,8 +9,8 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/SlicingBlockDevice.cpp
-  ../features/storage/blockdevice/HeapBlockDevice.cpp
+  ../storage/blockdevice/SlicingBlockDevice.cpp
+  ../storage/blockdevice/HeapBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
 )
diff --git a/UNITTESTS/moduletests/storage/kvstore/FileSystemStore/moduletest.cpp b/UNITTESTS/moduletests/storage/kvstore/FileSystemStore/moduletest.cpp
@@ -15,9 +15,9 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/HeapBlockDevice.h"
-#include "features/storage/kvstore/filesystemstore/FileSystemStore.h"
-#include "features/storage/filesystem/littlefs/LittleFileSystem.h"
+#include "storage/blockdevice/HeapBlockDevice.h"
+#include "storage/kvstore/filesystemstore/FileSystemStore.h"
+#include "storage/filesystem/littlefs/LittleFileSystem.h"
 #include "mbed_error.h"
 #include <stdlib.h>
 
diff --git a/UNITTESTS/moduletests/storage/kvstore/FileSystemStore/unittest.cmake b/UNITTESTS/moduletests/storage/kvstore/FileSystemStore/unittest.cmake
@@ -9,15 +9,15 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/HeapBlockDevice.cpp
-  ../features/storage/kvstore/filesystemstore/FileSystemStore.cpp
-  ../features/storage/filesystem/littlefs/LittleFileSystem.cpp
-  ../features/storage/filesystem/Dir.cpp
-  ../features/storage/filesystem/File.cpp
-  ../features/storage/filesystem/FileSystem.cpp
+  ../storage/blockdevice/HeapBlockDevice.cpp
+  ../storage/kvstore/filesystemstore/FileSystemStore.cpp
+  ../storage/filesystem/littlefs/LittleFileSystem.cpp
+  ../storage/filesystem/Dir.cpp
+  ../storage/filesystem/File.cpp
+  ../storage/filesystem/FileSystem.cpp
   ../features/frameworks/mbed-trace/source/mbed_trace.c
-  ../features/storage/filesystem/littlefs/littlefs/lfs_util.c
-  ../features/storage/filesystem/littlefs/littlefs/lfs.c
+  ../storage/filesystem/littlefs/littlefs/lfs_util.c
+  ../storage/filesystem/littlefs/littlefs/lfs.c
   ../platform/source/FileBase.cpp
   ../platform/source/FileSystemHandle.cpp
   ../platform/source/FileHandle.cpp
diff --git a/UNITTESTS/moduletests/storage/kvstore/TDBStore/moduletest.cpp b/UNITTESTS/moduletests/storage/kvstore/TDBStore/moduletest.cpp
@@ -15,9 +15,9 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/HeapBlockDevice.h"
-#include "features/storage/blockdevice/FlashSimBlockDevice.h"
-#include "features/storage/kvstore/tdbstore/TDBStore.h"
+#include "storage/blockdevice/HeapBlockDevice.h"
+#include "storage/blockdevice/FlashSimBlockDevice.h"
+#include "storage/kvstore/tdbstore/TDBStore.h"
 #include <stdlib.h>
 
 #define BLOCK_SIZE (512)
diff --git a/UNITTESTS/moduletests/storage/kvstore/TDBStore/unittest.cmake b/UNITTESTS/moduletests/storage/kvstore/TDBStore/unittest.cmake
@@ -10,10 +10,10 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/FlashSimBlockDevice.cpp
-  ../features/storage/blockdevice/HeapBlockDevice.cpp
-  ../features/storage/blockdevice/BufferedBlockDevice.cpp
-  ../features/storage/kvstore/tdbstore/TDBStore.cpp
+  ../storage/blockdevice/FlashSimBlockDevice.cpp
+  ../storage/blockdevice/HeapBlockDevice.cpp
+  ../storage/blockdevice/BufferedBlockDevice.cpp
+  ../storage/kvstore/tdbstore/TDBStore.cpp
   ../features/frameworks/mbed-trace/source/mbed_trace.c
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
diff --git a/UNITTESTS/storage/blockdevice/BufferedBlockDevice/test_BufferedBlockDevice.cpp b/UNITTESTS/storage/blockdevice/BufferedBlockDevice/test_BufferedBlockDevice.cpp
@@ -15,7 +15,7 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/BufferedBlockDevice.h"
+#include "storage/blockdevice/BufferedBlockDevice.h"
 #include "stubs/BlockDevice_mock.h"
 
 using ::testing::_;
diff --git a/UNITTESTS/storage/blockdevice/BufferedBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/BufferedBlockDevice/unittest.cmake
@@ -9,12 +9,12 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/BufferedBlockDevice.cpp
+  ../storage/blockdevice/BufferedBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
 )
 
 set(unittest-test-sources
-  features/storage/blockdevice/BufferedBlockDevice/test_BufferedBlockDevice.cpp
+  storage/blockdevice/BufferedBlockDevice/test_BufferedBlockDevice.cpp
   stubs/BlockDevice_mock.h
 )
diff --git a/UNITTESTS/storage/blockdevice/ChainingBlockDevice/test_ChainingBlockDevice.cpp b/UNITTESTS/storage/blockdevice/ChainingBlockDevice/test_ChainingBlockDevice.cpp
@@ -15,7 +15,7 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/ChainingBlockDevice.cpp"
+#include "storage/blockdevice/ChainingBlockDevice.cpp"
 #include "stubs/BlockDevice_mock.h"
 
 using ::testing::_;
diff --git a/UNITTESTS/storage/blockdevice/ChainingBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/ChainingBlockDevice/unittest.cmake
@@ -0,0 +1,20 @@
+
+####################
+# UNIT TESTS
+####################
+
+set(unittest-includes ${unittest-includes}
+  .
+  ..
+)
+
+set(unittest-sources
+  ../storage/blockdevice/ChainingBlockDevice.cpp
+  ../storage/blockdevice/HeapBlockDevice.cpp
+  stubs/mbed_atomic_stub.c
+  stubs/mbed_assert_stub.cpp
+)
+
+set(unittest-test-sources
+  storage/blockdevice/ChainingBlockDevice/test_ChainingBlockDevice.cpp
+)
diff --git a/UNITTESTS/storage/blockdevice/ExhaustibleBlockDevice/test_ExhaustibleBlockDevice.cpp b/UNITTESTS/storage/blockdevice/ExhaustibleBlockDevice/test_ExhaustibleBlockDevice.cpp
@@ -15,7 +15,7 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/ExhaustibleBlockDevice.h"
+#include "storage/blockdevice/ExhaustibleBlockDevice.h"
 #include "stubs/BlockDevice_mock.h"
 
 #define BLOCK_SIZE (512)
diff --git a/UNITTESTS/storage/blockdevice/ExhaustibleBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/ExhaustibleBlockDevice/unittest.cmake
@@ -9,11 +9,11 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/FlashSimBlockDevice.cpp
+  ../storage/blockdevice/ExhaustibleBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
 )
 
 set(unittest-test-sources
-  features/storage/blockdevice/FlashSimBlockDevice/test_FlashSimBlockDevice.cpp
+  storage/blockdevice/ExhaustibleBlockDevice/test_ExhaustibleBlockDevice.cpp
 )
diff --git a/UNITTESTS/storage/blockdevice/FlashSimBlockDevice/test_FlashSimBlockDevice.cpp b/UNITTESTS/storage/blockdevice/FlashSimBlockDevice/test_FlashSimBlockDevice.cpp
@@ -15,7 +15,7 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/FlashSimBlockDevice.h"
+#include "storage/blockdevice/FlashSimBlockDevice.h"
 #include "stubs/BlockDevice_mock.h"
 
 #define BLOCK_SIZE (512)
diff --git a/UNITTESTS/storage/blockdevice/FlashSimBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/FlashSimBlockDevice/unittest.cmake
@@ -9,11 +9,11 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/MBRBlockDevice.cpp
+  ../storage/blockdevice/FlashSimBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
 )
 
 set(unittest-test-sources
-  features/storage/blockdevice/MBRBlockDevice/test_MBRBlockDevice.cpp
+  storage/blockdevice/FlashSimBlockDevice/test_FlashSimBlockDevice.cpp
 )
diff --git a/UNITTESTS/storage/blockdevice/HeapBlockDevice/test.cpp b/UNITTESTS/storage/blockdevice/HeapBlockDevice/test.cpp
@@ -15,7 +15,7 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/HeapBlockDevice.h"
+#include "storage/blockdevice/HeapBlockDevice.h"
 #include <string.h>
 #include "mbed_assert.h"
 
diff --git a/UNITTESTS/storage/blockdevice/HeapBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/HeapBlockDevice/unittest.cmake
@@ -9,13 +9,13 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/HeapBlockDevice.cpp
+  ../storage/blockdevice/HeapBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
 )
 
 set(unittest-test-sources
-  features/storage/blockdevice/HeapBlockDevice/test.cpp
+  storage/blockdevice/HeapBlockDevice/test.cpp
 )
 
 set(unittest-test-flags
diff --git a/UNITTESTS/storage/blockdevice/MBRBlockDevice/test_MBRBlockDevice.cpp b/UNITTESTS/storage/blockdevice/MBRBlockDevice/test_MBRBlockDevice.cpp
@@ -15,7 +15,7 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/MBRBlockDevice.h"
+#include "storage/blockdevice/MBRBlockDevice.h"
 #include "stubs/BlockDevice_mock.h"
 
 #define BLOCK_SIZE (512)
diff --git a/UNITTESTS/storage/blockdevice/MBRBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/MBRBlockDevice/unittest.cmake
@@ -9,11 +9,11 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/ExhaustibleBlockDevice.cpp
+  ../storage/blockdevice/MBRBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
 )
 
 set(unittest-test-sources
-  features/storage/blockdevice/ExhaustibleBlockDevice/test_ExhaustibleBlockDevice.cpp
+  storage/blockdevice/MBRBlockDevice/test_MBRBlockDevice.cpp
 )
diff --git a/UNITTESTS/storage/blockdevice/ObservingBlockDevice/test_ObservingBlockDevice.cpp b/UNITTESTS/storage/blockdevice/ObservingBlockDevice/test_ObservingBlockDevice.cpp
@@ -15,8 +15,8 @@
  */
 
 #include "gtest/gtest.h"
-#include "features/storage/blockdevice/ObservingBlockDevice.h"
-#include "features/storage/blockdevice/ReadOnlyBlockDevice.h"
+#include "storage/blockdevice/ObservingBlockDevice.h"
+#include "storage/blockdevice/ReadOnlyBlockDevice.h"
 #include "stubs/BlockDevice_mock.h"
 
 using ::testing::_;
diff --git a/UNITTESTS/storage/blockdevice/ObservingBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/ObservingBlockDevice/unittest.cmake
@@ -9,14 +9,14 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/ObservingBlockDevice.cpp
-  ../features/storage/blockdevice/ReadOnlyBlockDevice.cpp
+  ../storage/blockdevice/ObservingBlockDevice.cpp
+  ../storage/blockdevice/ReadOnlyBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
   stubs/mbed_error.c
 )
 
 set(unittest-test-sources
-  features/storage/blockdevice/ObservingBlockDevice/test_ObservingBlockDevice.cpp
+  storage/blockdevice/ObservingBlockDevice/test_ObservingBlockDevice.cpp
   stubs/BlockDevice_mock.h
 )
diff --git a/UNITTESTS/storage/blockdevice/ProfilingBlockDevice/test_ProfilingBlockDevice.cpp b/UNITTESTS/storage/blockdevice/ProfilingBlockDevice/test_ProfilingBlockDevice.cpp
@@ -16,7 +16,7 @@
 
 #include "gtest/gtest.h"
 #include "stubs/BlockDevice_mock.h"
-#include "features/storage/blockdevice/ProfilingBlockDevice.h"
+#include "storage/blockdevice/ProfilingBlockDevice.h"
 
 #define BLOCK_SIZE (512)
 #define DEVICE_SIZE (BLOCK_SIZE*10)
diff --git a/UNITTESTS/storage/blockdevice/ProfilingBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/ProfilingBlockDevice/unittest.cmake
@@ -9,13 +9,13 @@ set(unittest-includes ${unittest-includes}
 )
 
 set(unittest-sources
-  ../features/storage/blockdevice/ReadOnlyBlockDevice.cpp
-  ../features/storage/blockdevice/HeapBlockDevice.cpp
+  ../storage/blockdevice/ProfilingBlockDevice.cpp
+  ../storage/blockdevice/HeapBlockDevice.cpp
   stubs/mbed_atomic_stub.c
   stubs/mbed_assert_stub.cpp
 )
 
 set(unittest-test-sources
-  features/storage/blockdevice/ReadOnlyBlockDevice/test_ReadOnlyBlockDevice.cpp
+  storage/blockdevice/ProfilingBlockDevice/test_ProfilingBlockDevice.cpp
   stubs/mbed_error.c
 )
diff --git a/UNITTESTS/storage/blockdevice/ReadOnlyBlockDevice/test_ReadOnlyBlockDevice.cpp b/UNITTESTS/storage/blockdevice/ReadOnlyBlockDevice/test_ReadOnlyBlockDevice.cpp
@@ -16,7 +16,7 @@
 
 #include "gtest/gtest.h"
 #include "stubs/BlockDevice_mock.h"
-#include "features/storage/blockdevice/ReadOnlyBlockDevice.h"
+#include "storage/blockdevice/ReadOnlyBlockDevice.h"
 #include "platform/mbed_error.h"
 
 #define BLOCK_SIZE (512)
diff --git a/UNITTESTS/storage/blockdevice/ReadOnlyBlockDevice/unittest.cmake b/UNITTESTS/storage/blockdevice/ReadOnlyBlockDevice/unittest.cmake
diff --git a/UNITTESTS/stubs/BlockDevice_mock.h b/UNITTESTS/stubs/BlockDevice_mock.h
diff --git a/UNITTESTS/stubs/EmulatedSD.h b/UNITTESTS/stubs/EmulatedSD.h
diff --git a/UNITTESTS/stubs/kv_config_stub.cpp b/UNITTESTS/stubs/kv_config_stub.cpp

Original file line number	Diff line number	Diff line change
`@@ -9,8 +9,8 @@ set(unittest-includes ${unittest-includes}`
`9`	`9`	`)`
`10`	`10`
`11`	`11`	`set(unittest-sources`
`12`		`- ../features/storage/blockdevice/SlicingBlockDevice.cpp`
`13`		`- ../features/storage/blockdevice/HeapBlockDevice.cpp`
	`12`	`+ ../storage/blockdevice/SlicingBlockDevice.cpp`
	`13`	`+ ../storage/blockdevice/HeapBlockDevice.cpp`
`14`	`14`	`stubs/mbed_atomic_stub.c`
`15`	`15`	`stubs/mbed_assert_stub.cpp`
`16`	`16`	`)`