fix: fix tests and ignore failed transfers if already executed

[tcar121293]

Description

• if failedExecution comes after the transfer is successfully executed, just ignore that event
• added edge case fix - always upsert deposit, if updating transfer, don't update the status

Related Issue Or Context

Closes: #

How Has This Been Tested? Testing details.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation

Checklist:

• I have commented my code, particularly in hard-to-understand areas.
• I have ensured that all acceptance criteria (or expected behavior) from issue are met
• I have updated the documentation locally and in chainbridge-docs.
• I have added tests to cover my changes.
• I have ensured that all the checks are passing and green, I've signed the CLA bot