13 pre commit (#15)

* track python and cpp changes

* shellcheck

* add trailing newline

* add ability to compile test cases

* update build script to configure testing; default off

* simple scaffold test case

Author: jeffreyqdd

.gitmodules

@@ -1,9 +1,3 @@
-[submodule "lib/concurrentqueue"]
-	path = lib/concurrentqueue
-	url = https://github.com/cameron314/concurrentqueue.git
-[submodule "lib/dlpack"]
-	path = lib/dlpack
-	url = https://github.com/dmlc/dlpack.git
-[submodule "lib/pybind11"]
-	path = lib/pybind11
-	url = https://github.com/pybind/pybind11.git
+[submodule "extern/Catch2"]
+path = extern/Catch2
+url = https://github.com/catchorg/Catch2.git

CMakeLists.txt

@@ -24,6 +24,14 @@ if(BUILD_EXAMPLES)
     add_subdirectory(examples)
 endif()
 
+# Build test cases
+option(BUILD_TESTS "Build test cases" OFF)
+if(BUILD_TESTS)
+    add_subdirectory(extern/Catch2)
+    enable_testing()
+    add_subdirectory(tests)
+endif()
+
 # Install
 include(GNUInstallDirs)
 include(CMakePackageConfigHelpers)

configure.sh

@@ -6,11 +6,12 @@ set -e
 BUILD_TYPE="Debug"
 BUILD_DIR="build"
 BUILD_EXAMPLES=OFF
+BUILD_TESTS=OFF
 
 # Parse arguments
 while [[ "$#" -gt 0 ]]; do
     case $1 in
-    -t | --type)
+    -m | --mode)
         BUILD_TYPE="$2"
         shift
         ;;
@@ -21,11 +22,15 @@ while [[ "$#" -gt 0 ]]; do
     -e | --examples)
         BUILD_EXAMPLES=ON
         ;;
+    -t | --tests)
+        BUILD_TESTS=ON
+        ;;
     -h | --help)
         echo "Usage: ./configure.sh [-t Debug|Release|RelWithDebInfo|MinSizeRel] [-d build_dir] [-e]"
-        echo "  -t | --type      : Build type (default: Debug)"
+        echo "  -m | --mode      : Build type (default: Debug)"
         echo "  -d | --dir       : Build directory (default: build)"
-        echo "  -e | --examples  : Enable building examples"
+        echo "  -e | --examples  : Enable building examples (default: OFF)"
+        echo "  -t | --tests     : Enable building tests (default: OFF)"
         exit 0
         ;;
     *)
@@ -40,11 +45,16 @@ echo "Configuring project..."
 echo "  Build type     : $BUILD_TYPE"
 echo "  Build dir      : $BUILD_DIR"
 echo "  Build examples : $BUILD_EXAMPLES"
+echo "  Build tests    : $BUILD_TESTS"
 
 # Create build directory
 mkdir -p "$BUILD_DIR"
 
 # Run CMake configuration
-cmake -B "$BUILD_DIR" -DCMAKE_BUILD_TYPE="$BUILD_TYPE" -DBUILD_EXAMPLES="$BUILD_EXAMPLES" -S .
+cmake -B "$BUILD_DIR" \
+    -DCMAKE_BUILD_TYPE="$BUILD_TYPE" \
+    -DBUILD_EXAMPLES="$BUILD_EXAMPLES" \
+    -DBUILD_TESTS="$BUILD_TESTS" \
+    -S .
 
 echo "Configuration complete. You can now run: cmake --build $BUILD_DIR"

extern/Catch2

@@ -0,0 +1,12 @@
+file(GLOB TEST_SOURCES CONFIGURE_DEPENDS *.cpp)
+
+get_target_property(_inc Catch2::Catch2WithMain INTERFACE_INCLUDE_DIRECTORIES)
+message(STATUS "Catch2::Catch2WithMain include dirs: ${_inc}")
+
+
+add_executable(pyscheduler_tests ${TEST_SOURCES})
+target_link_libraries(pyscheduler_tests PRIVATE pyscheduler Catch2::Catch2WithMain)
+
+include(CTest)
+include(Catch)
+catch_discover_tests(pyscheduler_tests)

tests/CMakeLists.txt

@@ -0,0 +1,17 @@
+#include "pyscheduler/pyscheduler.hpp"
+#include <catch2/catch_all.hpp>
+
+using namespace pyscheduler;
+
+static PyManager manager;
+
+TEST_CASE("Load Standard Module", "[basic]") {
+	PyManager::InvokeHandler reflect = manager.getPythonModule("tests.test_modules.reflect");
+}
+
+TEST_CASE("Reflect Test", "[basic]") {
+	PyManager::InvokeHandler reflect = manager.getPythonModule("tests.test_modules.reflect");
+
+	REQUIRE(reflect.invoke<std::string>("hello") == "hello");
+	REQUIRE(reflect.invoke<double>(3.1415926535) == 3.1415926535);
+}

tests/test_PyManager.cpp

@@ -0,0 +1,2 @@
+def invoke(x):
+    return x