openconfig / ygot / 947

91%

master: 89%

push

travis-ci

Fix PruneEmptyBranches.

When ygot.BuildEmptyTree is run on a struct, then the struct fields that are
struct pointers are recursively initialised. ygot.PruneEmptyBranches checked
whether a struct was non-nil, and removed it if not. However, this only worked
where the struct had only non-struct ptr children, since BuildEmptyTree had
already initialised struct ptr fields. The test cases that were used for
PruneEmptyBranches did not reflect this completely.

Additionally, PruneEmptyBranches did not iterate through maps and recursively
remove branches lower down in the tree. This made it less effective at removing
initialised structs that are empty than it should have been.

This CL fixes PruneEmptyBranches, adds better testing, and tests against the
exampleoc schema.

47 of 47 new or added lines in 1 file covered. (100.0%)

7496 of 8257 relevant lines covered (90.78%)

175.59 hits per line