Hello Reshad,
Thanks for the support! Also, thanks for starting to imagine how the coverage tool should work (therefore also how coverage should be defined), here are my initial thoughts on your points: - Presence container: this should be a must, also noting that presence containers can also be empty, therefore, we cannot count only on the leaves within to account for coverage. - coverage in groupings: Benoit and I were thinking about this one too. We might need to think about how to define this, but it is indeed a very practical application, and we really want this to be a practical tool. - Types in a union: this can be a bit tricky to visualise, but it can certainly be useful, let us think about it. Thanks! Camilo C From: Reshad Rahman <[email protected]> Reply to: Reshad Rahman <[email protected]> Date: Friday, 7 November 2025 at 12:42 To: NetMod WG <[email protected]>, "[email protected]" <[email protected]> Subject: Comments on draft-cardona-claise-onion-yang-coverage-00 Resent from: <[email protected]> Resent to: <[email protected]>, <[email protected]> Resent date: Fri, 7 Nov 2025 09:42:51 -0800 (PST) Hi Camilo, Benoit, As (I think) I mentioned in the meeting chat, I support this work, very useful. Often when reviewing documents containing YANG modules, I've asked myself how complete the examples are. Some authors do include minimal examples just to be able to tick the checkbox... So thanks from all YANG reviewers and YANG Doctors. I would like coverage to include more than simple calculation on leaf nodes, for example: - If a leaf is of type union, all union variants should be counted. e.g. if a module has 1 leaf node which has 2 union variants (string and int) and the example covers only 1 union variant for that leaf, IMO the coverage should be 50%. - Presence containers. I think they should be included in the calculation since they're often equivalent to "enable" flag. - Enums. Should all enum values be covered or just 1 is enough? - If a module has groupings which aren't used in the document, the coverage should include leaf nodes in the groupings (can be done by an example module which uses the groupings). Regards, Reshad.
_______________________________________________ netmod mailing list -- [email protected] To unsubscribe send an email to [email protected]
