add coverage test and ABI check

Author: scivision

.flake8

@@ -1,5 +1,6 @@
 [flake8]
 max-line-length = 132
-exclude = .git,__pycache__,.eggs/,doc/,docs/,build/,dist/,.archive/
+ignore = E501, W503, W504
+exclude = .git,__pycache__,.eggs/,doc/,docs/,build/,dist/,archive/,.archive/
 per-file-ignores =
-    __init__.py:F401
+    __init__.py:F401, F403

.gitattributes

@@ -0,0 +1 @@
+cmake/Modules/* linguist-vendored

.github/workflows/ci.yml

@@ -94,6 +94,36 @@ jobs:
       run: ctest --test-dir Examples/build -V
 
 
+  linux-coverage:
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+
+    steps:
+    - uses: actions/checkout@v2
+    - uses: actions/setup-python@v2
+      with:
+        python-version: '3.x'
+
+    - name: install hdf5
+      run: |
+        sudo apt update -yq
+        sudo apt install -yq --no-install-recommends libhdf5-openmpi-dev hdf5-tools libopenmpi-dev openmpi-bin
+
+    - name: install Gcovr
+      run: pip install gcovr
+
+    - run: cmake --preset coverage
+    - run: cmake --build --preset coverage
+
+    - name: Code coverage
+      run: cmake --build --preset run-coverage
+
+    - uses: actions/upload-artifact@v2
+      with:
+        name: coverage-report-html
+        path: build-coverage/coverage/
+
+
   linux-build-static:
     needs: linux-static
     runs-on: ubuntu-latest

.mypy.ini

@@ -1,7 +1,7 @@
 [mypy]
 files = src/
 
-ignore_missing_imports = true
+ignore_missing_imports = True
 strict_optional = False
 allow_redefinition = True
 show_error_context = False

CMakeLists.txt

@@ -20,13 +20,26 @@ HOMEPAGE_URL https://github.com/scivision/h5fortran-mpi
 )
 
 include(CTest)
+if(NOT DEFINED ${PROJECT_NAME}_BUILD_TESTING)
+  set(${PROJECT_NAME}_BUILD_TESTING ${BUILD_TESTING})
+endif()
 
 if(CMAKE_GENERATOR STREQUAL "Ninja Multi-Config")
   set(EXTPROJ_GENERATOR "Ninja")
 else()
   set(EXTPROJ_GENERATOR ${CMAKE_GENERATOR})
 endif()
 
+if(BUILD_SHARED_LIBS)
+  if(WIN32 AND CMAKE_VERSION VERSION_LESS 3.21 AND BUILD_TESTING)
+    message(STATUS "Windows with shared libs needs CMake >= 3.21 to run tests")
+  endif()
+  if(MSVC)
+    message(WARNING "Intel oneAPI has trouble with shared libs in general on Windows, try
+      cmake -DBUILD_SHARED_LIBS=off")
+  endif()
+endif()
+
 list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/cmake/Modules)
 if(ourFindMPI)
   list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/cmake/Modules/MPI)
@@ -71,7 +84,7 @@ endif()
 
 add_subdirectory(src)
 
-if(BUILD_TESTING)
+if(${PROJECT_NAME}_BUILD_TESTING)
   add_subdirectory(src/tests)
 endif()
 

CMakePresets.json

@@ -8,6 +8,17 @@
   "displayName": "Ninja Multi-Config",
   "generator": "Ninja Multi-Config"
 },
+{
+  "name": "coverage",
+  "binaryDir": "${sourceDir}/build-coverage",
+  "displayName": "Code Coverage",
+  "description": "Build with code coverage enabled.",
+  "cacheVariables": {
+    "CMAKE_BUILD_TYPE": "Debug",
+    "ENABLE_COVERAGE": true,
+    "autobuild": false
+  }
+},
 {
   "name": "phony"
 }
@@ -28,6 +39,15 @@
     "name": "debug",
     "configurePreset": "multi",
     "configuration": "Debug"
+  },
+  {
+    "name": "coverage",
+    "configurePreset": "coverage"
+  },
+  {
+    "name": "run-coverage",
+    "configurePreset": "coverage",
+    "targets": "coverage"
   }
 ],
 "testPresets": [