before there was a duplicate call to remove_node when that should have been handled by a finally clause. Also added a test to verify that the size of the graph remains unchanged before and after the call to negative_edge_cycle
Merge pull request #1079 from jstnlef/negative_edge_refactor

Duplicate remove_node call should be handled by finally