fix(mempool): Avoid panicking when a transaction is unexpectedly missing in the mempool

[arya2]

Motivation

Closes #10038.

Solution

• Logs a warning instead of panicking when a transaction that should be removed from the mempool is missing in the mempool's verified set, and
• Avoids calling remove() with tx ids that are not in the mempool.

Related:

• Fixes a bug where mined transactions were not being removed as dependencies of transactions in the mempool.

PR Checklist

• The PR name is suitable for the release notes.
• The PR follows the contribution guidelines.
• The library crate changelogs are up to date.
• The solution is tested.
• The documentation is up to date.

[conradoplg]

Looks good. I think it just needs making clippy happy

[conradoplg]

@arya2 just a reminder about this PR that should merge before the release, the clippy fix is related to the PR itself

[mergify]

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks:

• ❌ test success

You can check the last failing draft PR here: #10078.

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.