Infinite loop from move_spam_check() in Red Louse

[miketwo]

Not sure if it's the only enemy, but it seems to occassionally get into a situation where it has played both Grow and Bite 3 times in a row each, and then gets stuck forever.

Check if logic works when past moves are like ['move1', 'move1', 'move1', 'move2', 'move2', 'move2']

[vesper-arch]

I think the spam check function needs to go through the list in reverse. Might fix it but I don't see why it would get caught in a loop.

[miketwo]

Suggest using a unit test. I think this is what you'd want:

def test_move_spam_check():
    enemy = Enemy([100,100],8,"enemy")
    enemy.past_moves = ['move1', 'move1', 'move1', 'move2', 'move2', 'move2']
    assert enemy.move_spam_check('move1', 3) is True     
    assert enemy.move_spam_check('move2', 3) is False

[miketwo]

The problem is if all moves get flagged as spam, set_intent() loops forever trying to find a move.

[vesper-arch]

Try making the function return False early if the first move is not what is being searched for