From 1d3b00f0fd8870772abf653c2e9fff4dd6b16e65 Mon Sep 17 00:00:00 2001
From: Yaroslav Stefinko <ext-yaroslav.stefinko@here.com>
Date: Thu, 23 Apr 2026 16:29:46 +0300
Subject: [PATCH] Improve CodeCov scan
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- add more new flags/features
- range: 80..90, red below 80
- Fail if total project coverage drops by more than 0.5%
- after_n_builds: 1
codecov-action@v6 no longer auto-detects how many uploads
to expect. Without this, Codecov waits for more uploads that
never arrive, causing the GitHub status check to stay pending
until it times out. Since only one job uploads coverage
(psv-linux-22-04-gcc9-build-test-codecov), setting 1 tells
Codecov to process and post results immediately after that
- wait_for_ci: true
Ensures Codecov still waits for the CI job itself to finish
before evaluating — avoids a race where Codecov processes a
partial upload.

Relates-To: MINOR

Signed-off-by: Yaroslav Stefinko <ext-yaroslav.stefinko@here.com>
---
 codecov.yml | 109 ++++++++++++++++++++++++++++++++++++++--------------
 1 file changed, 80 insertions(+), 29 deletions(-)

diff --git a/codecov.yml b/codecov.yml
index 891c325fa..81a3d476e 100644
--- a/codecov.yml
+++ b/codecov.yml
@@ -1,48 +1,99 @@
+# Codecov configuration for here-data-sdk-cpp
+# Docs: https://docs.codecov.com/docs/codecovyml-reference
+# CI upload: codecov/codecov-action@v6 (single upload from psv-linux-22-04-gcc9-build-test-codecov)
+
+# Tell Codecov to finalise after exactly 1 upload and not wait for more.
+# This prevents PR status checks from hanging when only one coverage job exists.
+codecov:
+  notify:
+    after_n_builds: 1
+    wait_for_ci: true
+
 coverage:
-  range: 60..85
+  # Visualization band: green above 90%, red below 80%
+  range: 80..90
   round: nearest
   precision: 2
+
   status:
+    # ── Project (overall) checks ────────────────────────────────────────────
     project:
+      default:
+        # Fail if total project coverage drops by more than 0.5%
+        threshold: 0.5%
+        informational: false
+
       core:
         paths:
           - olp-cpp-sdk-core
-        threshold: 1%
+        target: 80%
+        threshold: 0.5%
+
       authentication:
         paths:
           - olp-cpp-sdk-authentication
-        threshold: 1%
+        target: 80%
+        threshold: 0.5%
+
       dataservice-read:
         paths:
           - olp-cpp-sdk-dataservice-read
-        threshold: 1%
+        target: 80%
+        threshold: 0.5%
+
       dataservice-write:
         paths:
           - olp-cpp-sdk-dataservice-write
-        threshold: 1%
+        target: 61%
+        threshold: 0.5%
+
+    # ── Patch checks (new/changed lines in a PR must be covered) ───────────
+    patch:
+      default:
+        target: 80%
+        threshold: 5%
+        informational: false
+
+# ── PR comment behaviour ──────────────────────────────────────────────────────
+comment:
+  layout: "reach,diff,flags,files"
+  behavior: default       # post once, update on new commits
+  require_changes: true   # don't comment if nothing changed
+  require_base: true
+  require_head: true
+  hide_project_coverage: false
+
+# ── GitHub Checks (inline PR annotations) ────────────────────────────────────
+github_checks:
+  annotations: true
+
+# ── Paths excluded from coverage reporting ───────────────────────────────────
 ignore:
-  - olp-cpp-sdk-authentication/tests
-  - olp-cpp-sdk-core/src/http/android
-  - olp-cpp-sdk-core/src/http/curl
-  - olp-cpp-sdk-core/src/http/ios
-  - olp-cpp-sdk-core/src/http/winhttp
-  - olp-cpp-sdk-core/src/http/Network.cpp
-  - olp-cpp-sdk-core/tests
-  - olp-cpp-sdk-dataservice-read/tests
-  - olp-cpp-sdk-dataservice-write/tests
-  - olp-cpp-sdk-dataservice-write/src/AutoFlushController.cpp
-  - olp-cpp-sdk-dataservice-write/src/AutoFlushController.h
-  - olp-cpp-sdk-dataservice-write/src/AutoFlushSettings.h
-  - olp-cpp-sdk-dataservice-write/src/BackgroundTaskCollection.cpp
-  - olp-cpp-sdk-dataservice-write/src/BackgroundTaskCollection.h
-  - olp-cpp-sdk-dataservice-write/src/DefaultFlushEventListener.cpp
-  - olp-cpp-sdk-dataservice-write/src/DefaultFlushEventListener.h
-  - olp-cpp-sdk-dataservice-write/src/FlushEventListener.h
-  - olp-cpp-sdk-dataservice-write/src/FlushMetrics.h
-  - tests
-  - scripts
-  - examples
-  - docs
-  - external
-  - build
+  # Test directories for every component
+  - "olp-cpp-sdk-authentication/tests"
+  - "olp-cpp-sdk-core/tests"
+  - "olp-cpp-sdk-dataservice-read/tests"
+  - "olp-cpp-sdk-dataservice-write/tests"
+  - "tests"
+
+  # Platform-specific HTTP backends (not exercised in CI coverage runs)
+  - "olp-cpp-sdk-core/src/http/android"
+  - "olp-cpp-sdk-core/src/http/curl"
+  - "olp-cpp-sdk-core/src/http/ios"
+  - "olp-cpp-sdk-core/src/http/winhttp"
+  - "olp-cpp-sdk-core/src/http/Network.cpp"
+
+  # Deprecated / flush-controller files in dataservice-write
+  - "olp-cpp-sdk-dataservice-write/src/AutoFlushController.*"
+  - "olp-cpp-sdk-dataservice-write/src/AutoFlushSettings.h"
+  - "olp-cpp-sdk-dataservice-write/src/BackgroundTaskCollection.*"
+  - "olp-cpp-sdk-dataservice-write/src/DefaultFlushEventListener.*"
+  - "olp-cpp-sdk-dataservice-write/src/FlushEventListener.h"
+  - "olp-cpp-sdk-dataservice-write/src/FlushMetrics.h"
 
+  # Non-source trees
+  - "scripts"
+  - "examples"
+  - "docs"
+  - "external"
+  - "build"