From 89a0700e1dedd0061fa844006bc29b7262b4b9af Mon Sep 17 00:00:00 2001
From: PatchOfScotland <d.marchant@ed-alumni.net>
Date: Fri, 10 Feb 2023 16:28:55 +0100
Subject: [PATCH] split up meow tests

---
 functionality/meow.py                |  4 ++-
 tests/{test_meow.py => test_base.py} | 53 ++++++++++++++++++++++++++--
 2 files changed, 54 insertions(+), 3 deletions(-)
 rename tests/{test_meow.py => test_base.py} (88%)

diff --git a/functionality/meow.py b/functionality/meow.py
index e299cff..feb6f0f 100644
--- a/functionality/meow.py
+++ b/functionality/meow.py
@@ -169,7 +169,9 @@ def create_rule(pattern:BasePattern, recipe:BaseRecipe,
     # TODO fix me
     # Imported here to avoid circular imports at top of file
     import rules
-    all_rules = {(r.pattern_type, r.recipe_type):r for r in BaseRule.__subclasses__()}
+    all_rules = {
+        (r.pattern_type, r.recipe_type):r for r in BaseRule.__subclasses__()
+    }
 
     # Add in new rules
     for rule in new_rules:
diff --git a/tests/test_meow.py b/tests/test_base.py
similarity index 88%
rename from tests/test_meow.py
rename to tests/test_base.py
index 6ecb1c7..b23cb7e 100644
--- a/tests/test_meow.py
+++ b/tests/test_base.py
@@ -14,8 +14,7 @@ from patterns import FileEventPattern
 from shared import setup, teardown, valid_pattern_one, valid_recipe_one
 
 
-# TODO split me
-class MeowTests(unittest.TestCase):
+class BaseRecipeTests(unittest.TestCase):
     def setUp(self)->None:
         super().setUp()
         setup()
@@ -43,6 +42,16 @@ class MeowTests(unittest.TestCase):
                 pass
         FullRecipe("name", "")
 
+
+class BasePatternTests(unittest.TestCase):
+    def setUp(self)->None:
+        super().setUp()
+        setup()
+
+    def tearDown(self)->None:
+        super().tearDown()
+        teardown()
+
     # Test that BaseRecipe instantiation
     def testBasePattern(self)->None:
         with self.assertRaises(TypeError):
@@ -137,6 +146,16 @@ class MeowTests(unittest.TestCase):
                 values.remove(f"{val1}/{val2}")
         self.assertEqual(len(values), 0)
 
+
+class BaseRuleTests(unittest.TestCase):
+    def setUp(self)->None:
+        super().setUp()
+        setup()
+
+    def tearDown(self)->None:
+        super().tearDown()
+        teardown()
+
     # Test that BaseRecipe instantiation
     def testBaseRule(self)->None:
         with self.assertRaises(TypeError):
@@ -156,6 +175,16 @@ class MeowTests(unittest.TestCase):
                 pass
         FullRule("name", valid_pattern_one, valid_recipe_one)
 
+
+class BaseMonitorTests(unittest.TestCase):
+    def setUp(self)->None:
+        super().setUp()
+        setup()
+
+    def tearDown(self)->None:
+        super().tearDown()
+        teardown()
+
     # Test that BaseMonitor instantiation
     def testBaseMonitor(self)->None:
         with self.assertRaises(TypeError):
@@ -197,6 +226,16 @@ class MeowTests(unittest.TestCase):
             
         FullTestMonitor({}, {})
 
+
+class BaseHandleTests(unittest.TestCase):
+    def setUp(self)->None:
+        super().setUp()
+        setup()
+
+    def tearDown(self)->None:
+        super().tearDown()
+        teardown()
+
     # Test that BaseHandler instantiation
     def testBaseHandler(self)->None:
         with self.assertRaises(TypeError):
@@ -223,6 +262,16 @@ class MeowTests(unittest.TestCase):
 
         FullTestHandler()
 
+
+class BaseConductorTests(unittest.TestCase):
+    def setUp(self)->None:
+        super().setUp()
+        setup()
+
+    def tearDown(self)->None:
+        super().tearDown()
+        teardown()
+
     # Test that BaseConductor instantiation
     def testBaseConductor(self)->None:
         with self.assertRaises(TypeError):