hmon: add C++ HMON unit tests

[arkjedrz]

C++ wrapper unit tests.

Resolves #121

[github-actions]

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output

[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.4.2) and connecting to it...
INFO: Invocation ID: 1ac5d047-61b6-4ee4-963e-34ffa676aabe
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (32 packages loaded, 10 targets configured)

Analyzing: target //:license-check (85 packages loaded, 10 targets configured)

Analyzing: target //:license-check (137 packages loaded, 2457 targets configured)

Analyzing: target //:license-check (146 packages loaded, 6015 targets configured)

Analyzing: target //:license-check (152 packages loaded, 8045 targets configured)

Analyzing: target //:license-check (157 packages loaded, 8094 targets configured)

INFO: Analyzed target //:license-check (162 packages loaded, 10232 targets configured).
[2 / 14] checking cached actions
[14 / 16] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 20.190s, Critical Path: 2.34s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

[github-actions]

The created documentation from the pull request is available at: docu-html

[Copilot]

Pull request overview

This PR extends the health monitoring library’s C++ test coverage by porting/expanding previously Rust-side unit tests into C++ and updates the Rust↔C++ FFI for HealthMonitorBuilder::build to accept optional cycle intervals.

Changes:

• Updated health_monitor_builder_build FFI to take nullable *const u64 / const uint64_t* cycle interval parameters (null = use Rust defaults).
• Added/expanded C++ gtest coverage for HealthMonitorBuilder/HealthMonitor APIs and introduced a new “integrated” usage test file.
• Updated Rust-side FFI unit tests to use the new optional-parameter calling convention.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
src/health_monitoring_lib/rust/ffi.rs	Changes the exported FFI signature for health_monitor_builder_build to nullable *const u64 and updates/extends unit tests accordingly.
src/health_monitoring_lib/rust/deadline/ffi.rs	Updates tests to call health_monitor_builder_build with null cycle interval pointers.
src/health_monitoring_lib/rust/heartbeat/ffi.rs	Updates tests to call health_monitor_builder_build with null cycle interval pointers.
src/health_monitoring_lib/rust/logic/ffi.rs	Updates tests to call health_monitor_builder_build with null cycle interval pointers.
src/health_monitoring_lib/cpp/include/score/hm/health_monitor.h	Stores cycle intervals as optionals to support passing nullable parameters over FFI.
src/health_monitoring_lib/cpp/health_monitor.cpp	Passes optional cycle intervals across FFI as nullable pointers.
src/health_monitoring_lib/cpp/tests/health_monitor_test.cpp	Replaces the previous “single integrated test” with a broader set of focused unit tests.
src/health_monitoring_lib/cpp/tests/integrated_test.cpp	Adds a higher-level integration-style test demonstrating multi-monitor setup and basic interactions.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[NicolasFussberger]

Please use the RecordProperty calls defined in Score Process to define TestType, DerivationTechnique, the Description and possible requirements links.

[MaciejKaszynski]

should use <> way and remove the ""

[MaciejKaszynski]

Can we have this one as a separate bazel target as it seems this is more of a integration test, so don't think we should mix test kinds into the same binary,