Skip to content

Feature/yaml validator v2#543

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

Feature/yaml validator v2#543
David-LP99 wants to merge 14 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 10.52632% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.23%. Comparing base (1a0f4ad) to head (d0d705a).

Files with missing lines Patch % Lines
ddsrouter_yaml/src/cpp/YamlReaderConfiguration.cpp 10.52% 6 Missing and 11 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #543      +/-   ##
==========================================
+ Coverage   31.14%   32.23%   +1.09%     
==========================================
  Files          16       16              
  Lines         594      577      -17     
  Branches      322      307      -15     
==========================================
+ Hits          185      186       +1     
+ Misses        213      193      -20     
- Partials      196      198       +2     

☔ 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>
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>
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