Instead of having two ways of using abilities directly, have one
way which is consistent with the API. Now unit has one method called
`ability` which takes a AbilityID and optionally either a Position or
another Unit which to use the ability at.

returns true if unit has target

This is the result of some bug-hunting. But I don't know if it makes any
difference. It isn't worse, at least.