Fix cancel payment action to reverse financial items related to cancelled payment

Fix cancel payment action to reverse financial items related to cancelled payment

The unit test to test & ensure this was failing to pick up that the records were not being created because it iterated
through the created rows & checked they were right but there was no check to ensure the rows were created :-(

This still leaves a gap I think for refunds not tied to a specific payment but that can be a later step