Skip to content

Fix: Increase test coverage & fix CheckLogger bug#587

Open
MaximilianSoerenPollak wants to merge 3 commits into
eclipse-score:mainfrom
MaximilianSoerenPollak:MSP_increase_test_coverage
Open

Fix: Increase test coverage & fix CheckLogger bug#587
MaximilianSoerenPollak wants to merge 3 commits into
eclipse-score:mainfrom
MaximilianSoerenPollak:MSP_increase_test_coverage

Conversation

@MaximilianSoerenPollak

@MaximilianSoerenPollak MaximilianSoerenPollak commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

📌 Description

Increases test coverage for graph-checks and fixes a bug in CheckLogger that didn't count warnings in some circumstances.

With these unit tests we now have a better test coverage for the graph_checks:
image

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@MaximilianSoerenPollak MaximilianSoerenPollak changed the title Msp increase test coverage Fix: Increase test coverage & fix CheckLogger bug Jun 9, 2026
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

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

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 37395837-3a29-450a-89be-c1e22e0fa18e
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 5.410s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant