Using typed list and index (so we can print a better failure message).

Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>
Remove stdout and catch exception to fail the test instead of returning in error.

Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>
Moves assertions on non-null bond length to the same loop. Also puts the getPoint3d() null test before the bond length check. If there is a null atom the bond length check will throw an NPE - better to fail on this.

Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>
Resolves unit test failure (previous commits). When a double bond is found and there are no 2D coordinates (e.g. unspecified configuration) then return then return '0' for the configuration value.

Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>