This branch resets the weapon cooldown at the start of target kill tests. Since app state isn't reset between tests the weapon cooldown was (in some cases) preventing a fire event from occurring as the weapon is still in its 0.1s cooldown.
Overall we should probably improve test infrastructure by resetting most of this state in the app prior to running each test instead of just relying on the fixture to do this. It is likely a certain sequence of tests could potentially still result in (illegitimate) failures.
This branch resets the weapon cooldown at the start of target kill tests. Since app state isn't reset between tests the weapon cooldown was (in some cases) preventing a fire event from occurring as the weapon is still in its 0.1s cooldown.
Overall we should probably improve test infrastructure by resetting most of this state in the app prior to running each test instead of just relying on the fixture to do this. It is likely a certain sequence of tests could potentially still result in (illegitimate) failures.
Merging this PR closes #275 and closes #243.