Skip to content

Feature/yaml validator v2#543

Open
David-LP99 wants to merge 11 commits into
mainfrom
feature/yaml_validator_v2
Open

Feature/yaml validator v2#543
David-LP99 wants to merge 11 commits into
mainfrom
feature/yaml_validator_v2

Conversation

@David-LP99

Copy link
Copy Markdown

Removed old python YAML validator and updating the documentation accordingly (still pending a proper update).

Updated ddsrouter_yaml so that when a router configuration is loaded, it has to be validated against the defined JSON schema for the router.

Added tests for the validator and the schema, the examples from the documentation and the resources have to pass the tests, and also a few new test files have to produce the correct result.

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 9.09091% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.03%. Comparing base (1a0f4ad) to head (3f863e0).

Files with missing lines Patch % Lines
ddsrouter_yaml/src/cpp/YamlReaderConfiguration.cpp 9.09% 7 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #543      +/-   ##
==========================================
- Coverage   31.14%   30.03%   -1.11%     
==========================================
  Files          16       16              
  Lines         594      566      -28     
  Branches      321      301      -20     
==========================================
- Hits          185      170      -15     
+ Misses        213      208       -5     
+ Partials      196      188       -8     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant