Merge pull request #83 from olifre/install-targets-also-if-testing

Revert "only set install targets when not doing unit tests"

Author: djw8605

.github/workflows/ccpp.yml

@@ -16,14 +16,20 @@ jobs:
   build:
 
     runs-on: ubuntu-latest
-    
+
+    strategy:
+      matrix:
+        external-gtest: [ YES, NO ]
+
+    name: Build with external_gtest=${{ matrix.external-gtest }}
+
     steps:
     - uses: actions/checkout@v1
       with:
         submodules: recursive
     - name: install deps
       run: |
-        sudo apt update && sudo apt-get install libssl-dev sqlite3 libsqlite3-dev cmake libcurl4 libcurl4-openssl-dev uuid-dev
+        sudo apt update && sudo apt-get install libssl-dev sqlite3 libsqlite3-dev cmake libcurl4 libcurl4-openssl-dev uuid-dev libgtest-dev
         
     - name: Create Build Environment
       # Some projects don't allow in-source building, so create a separate build directory
@@ -38,7 +44,7 @@ jobs:
       # Note the current convention is to use the -S and -B options here to specify source 
       # and build directories, but this is only available with CMake 3.13 and higher.  
       # The CMake binaries on the Github Actions machines are (as of this writing) 3.12
-      run: cmake $GITHUB_WORKSPACE -DCMAKE_BUILD_TYPE=$BUILD_TYPE -DBUILD_UNITTESTS=yes
+      run: cmake $GITHUB_WORKSPACE -DCMAKE_BUILD_TYPE=$BUILD_TYPE -DBUILD_UNITTESTS=yes -DEXTERNAL_GTEST=${{ matrix.external-gtest }}
 
     - name: Build
       working-directory: ${{runner.workspace}}/build
@@ -52,5 +58,3 @@ jobs:
       # Execute tests defined by the CMake configuration.  
       # See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail
       run: ctest -C $BUILD_TYPE --verbose
-
-