Summary

Pulls address base prefix checks for each action into a separate trait.

Background

It's easy to miss checks which we should be performing when writing a new action's check_and_execute trait method. This change pulls the address base prefix checks for each action into a separate trait so that they can be automatically performed for every action upon execution as well as tested in a more streamlined way.

Changes

• Removed base prefix checks and tests from action ActionHandler impls.
• Added new module address_check to address, with traits Addresses and AddressCheck: Addresses. Added impls of these for all existing actions.
• Added call to AddressCheck::check_addresses() in action_handler::transaction::check_execute_and_pay_fees().

Testing

• Added unit tests for the base prefix checks for all actions which contain addresses
• Added app unit test to ensure that check_addresses() is called under execute_transaction().

Changelogs

No updates required.

Related Issues

closes #2014