Several contract unit tests setup the use-case by storing orders directly in storage, rather than via message with funds. This has a side-effect of no actual coins in the contract escrow if not handled via passing in a contract_balance to mock_dependencies().
Standardizing on either a more "integration" level test setup style of passing messages with coins, or setting via contract_balance would be beneficial in separating unit from integration tests.
Several contract unit tests setup the use-case by storing orders directly in storage, rather than via message with funds. This has a side-effect of no actual coins in the contract escrow if not handled via passing in a
contract_balanceto mock_dependencies().Standardizing on either a more "integration" level test setup style of passing messages with coins, or setting via
contract_balancewould be beneficial in separating unit from integration tests.