It turns out that my fix in #250 didn't cover all cases where a network
in the assigned state would be considered. If a network is in a not-busy
state but did not have a prefix-length that matched what we're looking
for, it would never get checked to see if it overlaps with candidate
subnets.
This fixes that problem by modifying children to be all children
instead of all the ones in a not-busy state, since children is what we
check candidate subnets against looking for overlaps.
It turns out that my fix in #250 didn't cover all cases where a network in the assigned state would be considered. If a network is in a not-busy state but did not have a prefix-length that matched what we're looking for, it would never get checked to see if it overlaps with candidate subnets.
This fixes that problem by modifying
childrento be all children instead of all the ones in a not-busy state, sincechildrenis what we check candidate subnets against looking for overlaps.