Fix archery_damage_test CI occasional failure

[ShnitzelX2]

Summary

Bugfixes "fix archery_damage_test CI occasional failure"

Purpose of change

Fed up with CI failures, this is one of them

Describe the solution

Turns out that attack.missed_by was being slightly incremented in deal_projectile_attack() and not being reset:

Cataclysm-DDA/tests/archery_damage_test.cpp

Lines 60 to 62 in fc8e177

	for( int i = 0; i < 10; ++i ) {
	monster target{ mtype_id( target_type ), tripoint_zero };
	target.deal_projectile_attack( nullptr, attack, false );

So if the missed_by rolls (10 d 1000, in this case) were too low, the sum of the 10 iterations' additions to attack.missed_by would set it above the crit threshold and the bolts shot for the test wouldn't do enough damage. It took about 30 runs of archery_damage_test to isolate this.

I made attack.missed_by reset in the loop, so it should be fine now.

Describe alternatives you've considered

Testing

I made sure attack.missed_by wasn't incrementing anymore, and I ran the test successfully a couple times, but keep an eye out for future failures.

Additional context

Thankfully, this was just an issue with the test as far as I can tell.