Skip to content

Feature/yaml validator v2#187

Merged
rsanchez15 merged 21 commits into
mainfrom
feature/yaml_validator_v2
Jul 2, 2026
Merged

Feature/yaml validator v2#187
rsanchez15 merged 21 commits into
mainfrom
feature/yaml_validator_v2

Conversation

@David-LP99

Copy link
Copy Markdown
Contributor

Implemented a new validator for YAML files, this one checks the file against a JSON scheme that can be provided to the validator.

The tests contain the updated JSON schema for the DDS-Router, and a set of files for testing.

@codecov-commenter

codecov-commenter commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 33.95062% with 107 lines in your changes missing coverage. Please review.
✅ Project coverage is 36.67%. Comparing base (8eaf047) to head (84befaa).

Files with missing lines Patch % Lines
ddspipe_yaml/src/cpp/YamlValidator.cpp 48.24% 15 Missing and 44 partials ⚠️
ddspipe_yaml/src/cpp/YamlReader_features.cpp 0.00% 0 Missing and 19 partials ⚠️
ddspipe_yaml/src/cpp/YamlReader_generic.cpp 0.00% 17 Missing ⚠️
...pipe_yaml/include/ddspipe_yaml/impl/YamlReader.ipp 0.00% 4 Missing and 5 partials ⚠️
ddspipe_yaml/src/cpp/YamlReader_participants.cpp 0.00% 2 Missing ⚠️
...rticipants/src/cpp/xml/XmlHandlerConfiguration.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #187      +/-   ##
==========================================
+ Coverage   36.22%   36.67%   +0.44%     
==========================================
  Files         174      158      -16     
  Lines       12388     7643    -4745     
  Branches     5657     3058    -2599     
==========================================
- Hits         4488     2803    -1685     
+ Misses       5026     3339    -1687     
+ Partials     2874     1501    -1373     

☔ 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>
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>
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>
@Danipiza Danipiza added this to the 1.6.0 milestone Jun 25, 2026
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Comment thread ddspipe_yaml/test/unittest/yaml_validator/YamlValidatorTest.cpp
Comment thread ddspipe_yaml/test/unittest/yaml_validator/YamlValidatorTest.cpp
Comment thread ddspipe_yaml/src/cpp/YamlValidator.cpp
Comment thread ddspipe_yaml/src/cpp/YamlValidator.cpp
Comment thread ddspipe_yaml/src/cpp/YamlValidator.cpp Outdated
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
@Danipiza
Danipiza self-requested a review July 2, 2026 09:27
@rsanchez15
rsanchez15 merged commit d07191f into main Jul 2, 2026
26 of 29 checks passed
@rsanchez15
rsanchez15 deleted the feature/yaml_validator_v2 branch July 2, 2026 09:43
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.

4 participants