grudger: changed ForgetfulGrudger. tests now use self.versus_test

Author: eric-s-s

axelrod/strategies/grudger.py

@@ -67,12 +67,12 @@ def strategy(self, opponent: Player) -> Action:
             self.grudge_memory = 0
             self.grudged = False
 
+        if not self.grudged and D in opponent.history[-1:]:
+            self.grudged = True
+
         if self.grudged:
             self.grudge_memory += 1
             return D
-        elif D in opponent.history[-1:]:
-            self.grudged = True
-            return D
         return C
 
     def reset(self):
@@ -206,9 +206,9 @@ def strategy(self, opponent: Player) -> Action:
         """Begins by playing C, then plays Alternator for the remaining rounds
         if the opponent ever plays D."""
         if opponent.defections:
-            if self.history[-1] == Actions.C:
-                return Actions.D
-        return Actions.C
+            if self.history[-1] == C:
+                return D
+        return C
 
 
 class EasyGo(Player):
@@ -240,13 +240,14 @@ def strategy(opponent: Player) -> Action:
             return C
         return D
 
+
 class GeneralSoftGrudger(Player):
     """
-    A generalization of the SoftGrudger strategy. SoftGrudger punishes by playing:
-    D, D, D, D, C, C. after a defection by the opponent.  GeneralSoftGrudger
-    only punishes after its opponent defects a specified amount of times consecutively.
-    The punishment is in the form of a series of defections followed by a 'penance' of
-    a series of consecutive cooperations.
+    A generalization of the SoftGrudger strategy. SoftGrudger punishes by
+    playing: D, D, D, D, C, C. after a defection by the opponent.
+    GeneralSoftGrudger only punishes after its opponent defects a specified
+    amount of times consecutively. The punishment is in the form of a series of
+    defections followed by a 'penance' of a series of consecutive cooperations.
 
     Names:
 
@@ -268,11 +269,11 @@ def __init__(self, n: int=1, d: int=4, c: int=2) -> None:
         """
         Parameters
         ----------
-        n, int
+        n: int
             The number of defections by the opponent to trigger punishment
-        d, int
+        d: int
             The number of defections to punish the opponent
-        c, int
+        c: int
             The number of cooperations in the 'penance' stage
 
         Special Cases