> The change looks good, but your test doesn't seem to be calling
> build_skeleton(). And quick coverage run tells me your code is not exercised
> through test.
Good catch, I think something got messed up while I was experimenting with the test.
It's fixed now. I also dropped the self.build_skeleton() helper in the test class, since it's a usless indirection.
Thanks!
> The change looks good, but your test doesn't seem to be calling
> build_skeleton(). And quick coverage run tells me your code is not exercised
> through test.
Good catch, I think something got messed up while I was experimenting with the test. skeleton( ) helper in the test class, since it's a usless indirection.
It's fixed now. I also dropped the self.build_
Thanks!