Skip to content

[20274] Feature/yaml validator v2 - #153

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

[20274] Feature/yaml validator v2#153
David-LP99 wants to merge 5 commits into
mainfrom
feature/yaml_validator_v2

Conversation

@David-LP99

Copy link
Copy Markdown

Added a JSON-schema-based YAML validator to Fast DDS Spy for to ensure the configuration files follow the expected structure.

Configuration files are now validated against a dedicated JSON schema before being loaded, empty YAML files are accepted as a valid way of using the default configuration.

Also added a comprehensive set of tests for the validator (covering both valid and invalid configuration files), and updated the existing Yaml Reader tests and documentation/resources affected by these changes.

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
@David-LP99 David-LP99 changed the title Feature/yaml validator v2 [20274] Feature/yaml validator v2 Jul 15, 2026
@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.06%. Comparing base (1c51730) to head (3ce4bed).

Files with missing lines Patch % Lines
...astddsspy_yaml/src/cpp/YamlReaderConfiguration.cpp 25.00% 0 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #153      +/-   ##
==========================================
- Coverage   62.82%   58.06%   -4.76%     
==========================================
  Files          29       28       -1     
  Lines        2480     1934     -546     
  Branches     1068      770     -298     
==========================================
- Hits         1558     1123     -435     
+ Misses        363      325      -38     
+ Partials      559      486      -73     

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