Skip to content

Conversation

@margalva
Copy link
Collaborator

When a tree has multiple levels, it is possible that a top node has "value" empty. That should be accepted.
Check for the validity of 'value' only if it's not empty, or if it's at the last level of the tree hierarchy.

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.58%. Comparing base (ead301d) to head (3cf4c7a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #430   +/-   ##
=======================================
  Coverage   81.57%   81.58%           
=======================================
  Files          28       28           
  Lines        7980     7981    +1     
  Branches     1474     1475    +1     
=======================================
+ Hits         6510     6511    +1     
  Misses       1001     1001           
  Partials      469      469           

☔ View full report in Codecov by Sentry.
📢 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.

@margalva margalva merged commit 9e69f27 into main Oct 22, 2025
17 checks passed
@margalva margalva deleted the fix/tree_validator_last_level branch October 22, 2025 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants