Optional sanity check for actions.
authorSimon Cross <hodgestar@gmail.com>
Sat, 17 May 2014 20:13:27 +0000 (22:13 +0200)
committerSimon Cross <hodgestar@gmail.com>
Sat, 17 May 2014 20:14:06 +0000 (22:14 +0200)
naja/actions.py
naja/gameboard.py

index ff5bdba7dbfd6037afdd5764d4cf30c0ec138c60..395a1f502d63aa4e73f24a7761fe500d72c1ebf8 100644 (file)
@@ -16,6 +16,9 @@ class LocationAction(object):
         self.required_bits = required_bits
         self.data = data
 
+    def sanity_check(self, location):
+        pass
+
     def get_glyphs(self):
         return self.GLYPHS
 
@@ -153,6 +156,16 @@ class GenericBits(LocationAction):
         self.acquire_win = self.data.get('acquire_win', False)
         self.lose_health = self.data.get('lose_health', False)
 
+    def sanity_check(self, location):
+        missing_bits = set()
+        missing_bits.update(self.set_bits - set(location.bitwise_operand))
+        missing_bits.update(self.clear_bits - set(location.bitwise_operand))
+        missing_bits.update(self.toggle_bits - set(location.bitwise_operand))
+        if missing_bits:
+            raise ValueError(
+                "Location %s missing bits %r"
+                % (location.card_name, sorted(list(missing_bits))))
+
     def perform_action(self, board, location):
         bits = board.player.bits
         bits.set_bits(self.set_bits)
index 52d1323bd176a5cd3ca7205086b7fe44455f450f..689c3fd15fcf324d9110763331a6a0eeb93023d2 100644 (file)
@@ -305,6 +305,9 @@ class LocationCard(object):
         self.actions = location_actions
         self.max_number = max_number
         self.replacement_time = replacement_time
+        if options.debug:
+            for action in self.actions:
+                action.sanity_check(self)
 
     @classmethod
     def import_location(cls, state):