Fix strategy to run for five defections

Author: souravsingh

axelrod/strategies/titfortat.py

@@ -696,8 +696,9 @@ def __init__(self):
     def strategy(self, opponent: Player) -> Action:
         if not self.history:
             return C
-        if self.is_defector or opponent.history[-5:] == [D] * 5:
+        if not (self.is_defector) and opponent.defections >= 5:
             self.is_defector = True
+        if self.is_defector:
             return D
         return opponent.history[-1]
 

axelrod/tests/strategies/test_titfortat.py

@@ -642,8 +642,7 @@ def test_strategy(self):
                          match_attributes={"length": -1})
 
         # Becomes defector after 5 defections
-        opponent = axelrod.MockPlayer(actions=[C, D, D, D, D, D, C, C])
-        actions = [(C, C), (C, D), (D, D), (D, D),
+        opponent = axelrod.MockPlayer(actions=[D, C, D, D, D, D, C, C])
+        actions = [(C, D), (D, C), (C, D), (D, D),
                    (D, D), (D, D), (D, C), (D, C)]
-        self.versus_test(opponent, expected_actions=actions,
-                         attrs={"is_defector": True})
+        self.versus_test(opponent, expected_actions=actions)