Since rP22754, it is possible for units to be sent "Attacked" messages even though they cannot be attacked
(e.g. a non-capturable being the target of a Capture attack effect).
This seems bad for a few reasons:
- performance. Messages get sent for no reason
- likely source of bug.
- general cleanliness of the code.
This fixes it and adds a test.