Add find_package for Catch2 to check if library is installed on system path first. Make various formatting updates.

Author: Kartik Kumar

apps/cppbase.cpp

@@ -9,7 +9,7 @@
 
 #include "cppbase/cppbaseAll.hpp"
 
-int main( const int numberOfInputs, const char* inputArguments[ ] )
+int main(const int numberOfInputs, const char* inputArguments[])
 {
 	const int factorial = cppbase::computeFactorial(10);
 	std::cout << "10! = " << factorial << std::endl;

apps/helloWorld.cpp

@@ -7,7 +7,7 @@
 #include <cstdlib>
 #include <iostream>
 
-int main( const int numberOfInputs, const char* inputArguments[ ] )
+int main(const int numberOfInputs, const char* inputArguments[ ])
 {
     std::cout << "Hello World!" << std::endl;
 	return EXIT_SUCCESS;

cmake

@@ -9,13 +9,18 @@
 # @TODO: add find_package call to use Catch2 that is already present and only fetch otherwise
 # @TODO: consider only using the Catch2 single header to avoid building the whole library (slow)
 
-# Fetch Catch2 testing library
-FetchContent_Declare(
-  Catch2
-  GIT_REPOSITORY https://github.com/catchorg/Catch2.git
-  GIT_TAG        v3.0.1 # or a later release
-)
-FetchContent_MakeAvailable(Catch2)
+find_package(Catch2 3 REQUIRED)
+if(TARGET Catch2::Catch2)
+  message(STATUS "Found Catch2")
+else()
+  # Fetch Catch2 testing library
+  FetchContent_Declare(
+    Catch2
+    GIT_REPOSITORY https://github.com/catchorg/Catch2.git
+    GIT_TAG        v3.0.1 # or a later release
+  )
+  FetchContent_MakeAvailable(Catch2)
+endif(TARGET Catch2::Catch2)
 
 # Add test executables and linked libraries
 add_executable(cppbase_tests testFactorial.cpp)

tests/CMakeLists.txt

@@ -13,12 +13,12 @@ namespace cppbase
 namespace tests
 {
 
-TEST_CASE( "Compute factorials", "[factorial]" )
+TEST_CASE("Compute factorials", "[factorial]")
 {
-    REQUIRE( computeFactorial(1) == 1 );
-    REQUIRE( computeFactorial(2) == 2 );
-    REQUIRE( computeFactorial(3) == 6 );
-    REQUIRE( computeFactorial(10) == 3628800 );
+    REQUIRE(computeFactorial(1) == 1);
+    REQUIRE(computeFactorial(2) == 2);
+    REQUIRE(computeFactorial(3) == 6);
+    REQUIRE(computeFactorial(10) == 3628800);
 }
 
 } // namespace tests