Add ctest output-on-failure and remove redundant build command for code coverage

Author: jonpas

.gitlab-ci.yml

@@ -24,7 +24,7 @@ build-linux:
     - mkdir -p build && cd build
     - cmake .. $CMAKE_ARGS -DIDICMAKE_PLATFORM_CONFIG=../platform-config.$TYPE.cmake
     - cmake --build . -j 8
-    - ctest -C Debug --output-junit ctest.xml
+    - ctest -C Debug --output-on-failure --output-junit ctest.xml
   interruptible: true
   artifacts:
     when: always
@@ -52,7 +52,7 @@ build-windows:
     - cd build
     - cmake .. $CMAKE_ARGS -DIDICMAKE_PLATFORM_CONFIG="../platform-config.$TYPE.cmake"
     - cmake --build . -j 8
-    - ctest -C Debug --output-junit ctest.xml
+    - ctest -C Debug --output-on-failure --output-junit ctest.xml
   interruptible: true
   artifacts:
     when: always

README.md

@@ -113,7 +113,6 @@ Enable coverage by passing `-DCODE_COVERAGE=ON` at configure time:
 ```sh
 $ cd build
 $ cmake .. -DCODE_COVERAGE=ON
-$ cmake --build .
 ```
 
 Then run one of the coverage build targets: