Make minimal setup for tests

Author: msluszniak

packages/react-native-executorch/common/rnexecutorch/data_processing/ImageProcessing.cpp

@@ -76,16 +76,18 @@ static cv::Mat convertBufferToColorMat(std::span<const float> buffer,
 }
 
 std::string saveToTempFile(const cv::Mat &image) {
-  std::string filename = "rn_executorch_" + file_utils::getTimeID() + ".png";
+  const auto filename = std::string("rn_executorch_")
+                            .append(file_utils::getTimeID())
+                            .append(".png");
 
-  fs::path tempDir = fs::temp_directory_path();
-  fs::path filePath = tempDir / filename;
+  const fs::path tempDir = fs::temp_directory_path();
+  const fs::path filePath = tempDir / filename;
 
   if (!cv::imwrite(filePath.string(), image)) {
     throw std::runtime_error("Failed to save the image: " + filePath.string());
   }
 
-  return "file://" + filePath.string();
+  return std::string("file://").append(filePath.string());
 }
 
 static cv::Mat handleBase64Data(const std::string &imageURI) {

packages/react-native-executorch/common/rnexecutorch/tests/CMakeLists.txt

@@ -5,26 +5,37 @@ project(RNExecutorchTests)
 set(CMAKE_CXX_STANDARD 20)
 set(CMAKE_CXX_STANDARD_REQUIRED TRUE)
 
-# googletest subdirectory
-# Using an absolute path from the top-level source directory
-add_subdirectory(${CMAKE_SOURCE_DIR}/../../../../../third-party/googletest ${PROJECT_BINARY_DIR}/googletest)
+# Include directories for your project
+include_directories("${PROJECT_SOURCE_DIR}/../")
+include_directories("${PROJECT_SOURCE_DIR}/../data_processing")
+
+# Enable testing with GoogleTest
+enable_testing()
+
+# Include Google Test
+add_subdirectory("${PROJECT_SOURCE_DIR}/../../../../../third-party/googletest" "${CMAKE_BINARY_DIR}/googletest")
 
-# Directories to include
-include_directories(${CMAKE_SOURCE_DIR}/../data_processing)
-include_directories(${CMAKE_SOURCE_DIR}/..)
 
-# Source files
-set(SOURCE_FILES ${CMAKE_SOURCE_DIR}/../data_processing/Numerical.cpp)
+# Include Google Test's headers
+include_directories("${gtest_SOURCE_DIR}/include" "${gmock_SOURCE_DIR}/include")
 
-# Executables for the tests
-add_executable(NumericalTests NumericalTest.cpp ${SOURCE_FILES})
-add_executable(LogTests LogTest.cpp)
+# Gather all test source files
+set(TEST_SOURCES
+    # "${PROJECT_SOURCE_DIR}/ImageProcessingTest.cpp"
+    "${PROJECT_SOURCE_DIR}/LogTest.cpp"
+    "${PROJECT_SOURCE_DIR}/NumericalTest.cpp"
+)
+
+set(FILE_SOURCES
+    # "${PROJECT_SOURCE_DIR}/../data_processing/ImageProcessing.cpp"
+    "${PROJECT_SOURCE_DIR}/../data_processing/Numerical.cpp"
+)
+
+# Add test executable
+add_executable(RNExecutorchTests "${PROJECT_SOURCE_DIR}/TestRunner.cpp" ${TEST_SOURCES} ${FILE_SOURCES})
 
 # Libraries linking
-target_link_libraries(NumericalTests gtest gtest_main)
-target_link_libraries(LogTests gtest gtest_main)
+target_link_libraries(RNExecutorchTests gtest gtest_main)
 
 # Testing functionalities
-enable_testing()
-add_test(NAME NumericalTests COMMAND NumericalTests)
-add_test(NAME LogTests COMMAND LogTests)
+add_test(NAME RNExecutorchTests COMMAND RNExecutorchTests)

packages/react-native-executorch/common/rnexecutorch/tests/ImageProcessingTest.cpp

@@ -0,0 +1,30 @@
+#include "data_processing/ImageProcessing.h"
+#include <filesystem>
+#include <gtest/gtest.h>
+#include <opencv2/opencv.hpp>
+
+namespace rnexecutorch::image_processing {
+
+namespace fs = std::filesystem;
+
+class ImageProcessingTests : public ::testing::Test {
+protected:
+  cv::Mat testImage;
+  void SetUp() override {
+    testImage = cv::Mat::ones(100, 100, CV_8UC3) * 255; // white test image
+  }
+};
+
+// Test for saveToTempFile
+TEST_F(ImageProcessingTests, SaveToTempFile) {
+  std::string filePath = saveToTempFile(testImage);
+  ASSERT_TRUE(filePath.starts_with("file://"));
+  ASSERT_TRUE(filePath.ends_with(".png"));
+  ASSERT_TRUE(fs::exists(filePath.substr(7)));
+  ASSERT_NO_THROW(auto mat = cv::imread(filePath, cv::IMREAD_COLOR));
+  ASSERT(!mat.empty());
+
+  fs::remove(filePath.substr(7));
+}
+
+} // namespace rnexecutorch::image_processing

packages/react-native-executorch/common/rnexecutorch/tests/LogTest.cpp

@@ -522,9 +522,4 @@ TEST(MovingSequencable, MovingSequencableTest) {
   ASSERT_EQ(q.size(), 0);
 }
 
-} // namespace rnexecutorch
-
-int main(int argc, char **argv) {
-  ::testing::InitGoogleTest(&argc, argv);
-  return RUN_ALL_TESTS();
-}
+} // namespace rnexecutorch

packages/react-native-executorch/common/rnexecutorch/tests/TestRunner.cpp

@@ -0,0 +1,6 @@
+#include <gtest/gtest.h>
+
+int main(int argc, char **argv) {
+  ::testing::InitGoogleTest(&argc, argv);
+  return RUN_ALL_TESTS();
+}