Add more logic for seeing if a transaction succeeded for failed

[sdesmond46]

Right now if a tx is mined, we say it succeeded which isn't necessarily true