Merge pull request #12 from nuzcraft/part_9

Part 9

Author: nuzcraft

README.md

@@ -12,6 +12,18 @@ I want to use this as an opportunity to:
 4. make a roguelike
 5. practice with git
 
+## Part 9 Dev Notes
+
+### Ranged Scrolls and Targeting
+
+https://rogueliketutorials.com/tutorials/tcod/v2/part-9/
+
+Woot! No additional refactoring yet! This section should be fun because it will give us the opportunity to use the code we just built to add more items that do unique things. Hopefully (outside of targeting) the additional things should be relatively minor and relatively atomic.
+
+This part was really insightful to develop as I believe it will pave the way for a significant amount of future development. We developed some unique spells that function in very different ways; one that automatically targets the nearest actor and 2 the require additional user input mid-cast. This additional input is implemented in an interesting way and by watching it get out in place I think I understand the code flow. 
+
+From a unit testing standpoint, this part was intense. Adding new handlers for user input meant a significant amount of unit testing. I'm also noticing that there is a lot id dependencies around the player, engine, and game map objects that makes it cumbersome to set up tests on other objects. All in, unit testing is becoming more comfortable and I'm looking forward to a time when I feel like I can push what I know to make it more organized and more performant.
+
 ## Part 8 Dev Notes
 
 ### Items and Inventory

Tests/test_ai.py

@@ -4,7 +4,7 @@
 
 from numpy import power
 
-from components.ai import BaseAI, HostileEnemy
+from components.ai import BaseAI, HostileEnemy, ConfusedEnemy
 from components.fighter import Fighter
 from components.inventory import Inventory
 from entity import Entity, Actor
@@ -109,6 +109,53 @@ def test_get_path_to_avoid_entities(self):
         self.assertEqual(path, path_should_be)
 
 
+class TestConfusedEnemy(unittest.TestCase):
+    def test_init(self):
+        '''
+        test that the confused enemy can be initized without issues
+        '''
+        actor = Actor(ai_cls=HostileEnemy, fighter=Fighter(
+            hp=10, defense=10, power=10), inventory=Inventory(capacity=5))
+        ai = ConfusedEnemy(
+            entity=actor, previous_ai=HostileEnemy, turns_remaining=5)
+        self.assertEqual(ai.previous_ai, HostileEnemy)
+        self.assertEqual(ai.turns_remaining, 5)
+
+    def test_perform_switch_ai(self):
+        '''
+        test that with 0 or less turns remaining the ai switches back 
+        '''
+        actor = Actor(ai_cls=HostileEnemy, fighter=Fighter(
+            hp=10, defense=10, power=10), inventory=Inventory(capacity=5))
+        ai = ConfusedEnemy(
+            entity=actor, previous_ai=HostileEnemy(entity=actor), turns_remaining=0)
+        actor.ai = ai
+        eng = Engine(player=actor)
+        gm = GameMap(engine=eng, width=10, height=10)
+        actor.parent = gm
+        self.assertIsInstance(actor.ai, ConfusedEnemy)
+        actor.ai.perform()
+        self.assertIsInstance(actor.ai, HostileEnemy)
+
+    def test_perform_bump_action(self):
+        '''
+        test that with turns remaining the ai will perform a bump action 
+        in a random direction and reduce the number of turns remaining
+        '''
+        actor = Actor(ai_cls=HostileEnemy, fighter=Fighter(
+            hp=10, defense=10, power=10), inventory=Inventory(capacity=5))
+        ai = ConfusedEnemy(
+            entity=actor, previous_ai=HostileEnemy(entity=actor), turns_remaining=5)
+        actor.ai = ai
+        eng = Engine(player=actor)
+        gm = GameMap(engine=eng, width=10, height=10)
+        actor.parent = gm
+        with patch('actions.BumpAction.perform') as patch_perform:
+            ai.perform()
+        self.assertEqual(ai.turns_remaining, 4)
+        patch_perform.assert_called_once()
+
+
 class TestHostileEnemy(unittest.TestCase):
     def test_init(self):
         '''