Test both ends of contract between AttackAction and Action
refactor AttackAction spec
Removed duplication between specs of subactions