cmake: unit tests

Author: igagis

build/cmake/CMakeLists.txt

@@ -24,3 +24,8 @@ myci_declare_library(${name}
     DEPENDENCIES
         utki
 )
+
+# ==============
+# = unit tests =
+
+include(unit_tests.cmake)

build/cmake/unit_tests.cmake

@@ -0,0 +1,30 @@
+# if the library is compiled by vcpkg during the port build (i.e. during the package installation),
+# then we don't need to build unit tests
+if(IS_VCPKG_PORT_BUILD)
+    return()
+endif()
+
+# no unit tests for ios
+if(IOS)
+    return()
+endif()
+
+set(tests_srcs)
+myci_add_source_files(tests_srcs
+    DIRECTORY
+        ${CMAKE_CURRENT_LIST_DIR}/../../tests/unit/src
+    RECURSIVE
+)
+
+myci_declare_application(${PROJECT_NAME}-tests
+    SOURCES
+        ${tests_srcs}
+    DEPENDENCIES
+        tst
+        r4
+)
+
+myci_declare_test(
+    RUN_TARGET
+        run-${PROJECT_NAME}-tests
+)

build/vcpkg/portfile.cmake.in

@@ -10,6 +10,8 @@ vcpkg_from_github(
 
 vcpkg_cmake_configure(
     SOURCE_PATH "${SOURCE_PATH}/build/cmake"
+    OPTIONS
+        -D IS_VCPKG_PORT_BUILD=True
 )
 
 vcpkg_cmake_install()

build/vcpkg/test/CMakeLists.txt

@@ -4,8 +4,8 @@ set(CMAKE_TOOLCHAIN_FILE $ENV{VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake)
 
 project(test)
 
+find_package(myci CONFIG REQUIRED)
+find_package(tst CONFIG REQUIRED)
 find_package(r4 CONFIG REQUIRED)
 
-add_executable(test main.cpp)
-
-target_link_libraries(test PRIVATE r4::r4)
+include(../../cmake/unit_tests.cmake)

build/vcpkg/test/main.cpp

@@ -429,7 +429,7 @@ class vector :
 	 * @return Reference to this vector.
 	 */
 	template <typename unary_operation_type>
-	vector& comp_operation(unary_operation_type op)
+	constexpr vector& comp_operation(unary_operation_type op)
 	{
 		std::transform(
 			this->begin(), //
@@ -449,9 +449,18 @@ class vector :
 	 * @return Reference to this vector.
 	 */
 	template <typename binary_operation_type>
-	vector& comp_operation(const vector& vec, binary_operation_type op)
+	constexpr vector& comp_operation(
+		const vector& vec, //
+		binary_operation_type op
+	)
 	{
-		std::transform(this->begin(), this->end(), vec.begin(), this->begin(), op);
+		std::transform(
+			this->begin(), //
+			this->end(),
+			vec.begin(),
+			this->begin(),
+			op
+		);
 		return *this;
 	}
 
@@ -689,7 +698,7 @@ class vector :
 	 * @param num - scalar to divide by.
 	 * @return Vector resulting from division of this vector by scalar.
 	 */
-	vector operator/(component_type num) const noexcept
+	constexpr vector operator/(component_type num) const noexcept
 	{
 		return vector(*this) /= num;
 	}
@@ -711,11 +720,12 @@ class vector :
 	 * @param num - scalar to divide by.
 	 * @return Reference to this vector object.
 	 */
-	vector& operator/=(component_type num) noexcept
+	constexpr vector& operator/=(component_type num) noexcept
 	{
-		ASSERT(num != 0, [&](auto& o) {
-			o << "vector::operator/=(): division by 0";
-		})
+		// TODO: uncomment when there is a solution to have the assertion in constexpr context
+		// utki::assert(num != 0, [&](auto& o) {
+		// 	o << "vector::operator/=(): division by 0";
+		// });
 		return this->comp_operation([&num](auto& a) {
 			return a / num;
 		});