Fixed test for bug fix that pending was not dumping now it is dumping
rerder when remove transactions so consistent
shouldn't make a difference other than readability
Channed removeTransaction so that if transaction is provided but is not at index
will do slow search to find it. Does quick search first

Found corner case where index was invalid so transaction not removed in Allower
Can't find why it was invalid
Updated version