more

.github/workflows/CI.yml

@@ -22,10 +22,10 @@ jobs:
     strategy:
       matrix:
         config:
-        - {cc: gcc, cxx: g++, cmake_args: "-DCMAKE_BUILD_TYPE=Debug", coverage: 'yes', benchmark: 'no'}
-        - {cc: gcc, cxx: g++, cmake_args: "-DCMAKE_BUILD_TYPE=Release -DUSE_BENCHMARK=ON", coverage: 'no', benchmark: 'yes'}
-        - {cc: clang, cxx: clang++, cmake_args: "-DCMAKE_BUILD_TYPE=Debug", coverage: 'no', benchmark: 'no'}
-        - {cc: clang, cxx: clang++, cmake_args: "-DCMAKE_BUILD_TYPE=Release", coverage: 'no', benchmark: 'no'}
+        - {cc: gcc, cxx: g++, cmake_args: "-DCMAKE_BUILD_TYPE=Debug", coverage: true, benchmark: false}
+        - {cc: gcc, cxx: g++, cmake_args: "-DCMAKE_BUILD_TYPE=Release -DUSE_BENCHMARK=ON", coverage: false, benchmark: true}
+        - {cc: clang, cxx: clang++, cmake_args: "-DCMAKE_BUILD_TYPE=Debug", coverage: false, benchmark: false}
+        - {cc: clang, cxx: clang++, cmake_args: "-DCMAKE_BUILD_TYPE=Release", coverage: false, benchmark: false}
 
     steps:
     - uses: actions/checkout@v1
@@ -50,9 +50,9 @@ jobs:
         CXX: ${{ matrix.config.cxx }}
     - name: Kcov
       run: curl -s https://raw.githubusercontent.com/Geode-solutions/actions/master/kcov.sh | bash -s -- ${GITHUB_WORKSPACE}/build/opengeode/bin
-      if: matrix.config.coverage == 'yes' && (github.event_name == 'pull_request' || github.ref == 'refs/heads/master')
+      if: matrix.config.coverage && (github.event_name == 'pull_request' || github.ref == 'refs/heads/master')
     - uses: codecov/[email protected]
-      if: matrix.config.coverage == 'yes' && (github.event_name == 'pull_request' || github.ref == 'refs/heads/master')
+      if: matrix.config.coverage && (github.event_name == 'pull_request' || github.ref == 'refs/heads/master')
       with: 
         token: ${{ secrets.CODECOV_TOKEN }}
         file: kcov/kcov-merged/cobertura.xml

.vscode/settings.json

@@ -12,5 +12,6 @@
     ],
     "files.watcherExclude": {
         "${workspaceFolder}/build/**": true
-    }
+    },
+    "C_Cpp.dimInactiveRegions": false
 }

tests/common.h

@@ -25,6 +25,12 @@
 
 #include <geode/basic/common.h>
 
+/*!
+ * The OPENGEODE_TEST macro takes a name as input and also use a function
+ * named "test" that takes no argument. This function should be accessible
+ * from the context in which OPENGEODE_TEST is called.
+ */
+
 #ifdef OPENGEODE_BENCHMARK
 #define CATCH_CONFIG_MAIN
 #define CATCH_CONFIG_PREFIX_ALL