[V132-specific] Only spend UTXO unused gas for Migrate outbounds

[V132-specific]

Intended to close #1878 (closed)
'Commit e8fe6be0 appears to avoidably send unused gas to UTXO outbound recipients'.

(This only affects the consolidated UTXO client code as the chain-specific client code has conveniently already been cleared in !3378 (merged).)

(Please note that the TxMigrate check doesn't need a simultaneous TxConsolidate check as that already exists just below;
it assumes MaxGas to be empty for TxConsolidate, unlike for TxMigrate.)

added chain-client label

mentioned in issue #1878 (closed)

mentioned in merge request !3418 (merged)

added 1 commit

• 508713af - Smoke tests update

Compare with previous version

added 1 commit

• c38cbda3 - Smoke test update reflecting change in ETH gas cost (but occurring why?)

Compare with previous version

I support this, but think it should be done after we have changes in place for a chain specific outbound fee multiplier - charging all outbounds the max gas will be quite expensive for the user.

marked this merge request as draft

added 1 commit

• 57f02aac - Smoke tests handle_withdraw outbound amount update

Compare with previous version

changed the description

changed the description

mentioned in merge request !3420 (merged)

mentioned in merge request !3423 (merged)

changed the description

added 58 commits

• 57f02aac...d1ec174d - 53 commits from branch develop
• 69787814 - Only spend UTXO unused gas for Migrate outbounds (since wanting to sweep from...
• 82f90879 - Unit test updates (for no empty memo, which would cause ParseMemo error and return)
• c1627fe5 - Smoke tests update
• abed94eb - Smoke test update reflecting change in ETH gas cost (but occurring why?)
• 9fbbed74 - Smoke tests handle_withdraw outbound amount update

Compare with previous version

marked this merge request as ready

changed the description

mentioned in issue #1909 (closed)

Symbolic approve - will refrain from real approval until !3423 (merged) is merged to ensure this is not erroneously merged early.

added 33 commits

• 9fbbed74...6e9105fd - 28 commits from branch develop
• 64319d51 - Only spend UTXO unused gas for Migrate outbounds (since wanting to sweep from...
• c21eac1c - Unit test updates (for no empty memo, which would cause ParseMemo error and return)
• 6b8e8594 - Smoke tests update
• fa0adc72 - Smoke test update reflecting change in ETH gas cost (but occurring why?)
• cd2c3334 - Smoke tests handle_withdraw outbound amount update

Compare with previous version

added 1 commit

• 5e46a375 - Smoke tests handle_withdraw outbound amount update

Compare with previous version