process_tracker_python-124 Add Extract Tracker finder

Added check and test for if process_tracking_id is not provided, then
need to ensure that process_name and process_type are not None.  Had to
provide default values as a result of adding in the process_tracking_id
instantiation.

Author: Alex Meadows

process_tracker/process_tracker.py

@@ -153,6 +153,10 @@ def __init__(
                 self.logger.error(error_msg)
                 raise Exception(error_msg)
         else:
+            if process_name is None or process_type is None:
+                error_msg = "process_name and process_type must be set."
+                self.logger.error(error_msg)
+                raise Exception(error_msg)
 
             self.actor = self.data_store.get_or_create_item(
                 model=Actor, actor_name=actor_name

tests/test_process_tracker.py

@@ -1674,9 +1674,6 @@ def test_raise_run_error_with_fail(self):
             fail_date,
         ]
 
-        print(self.provided_end_date)
-        print(process[0].last_failed_run_date_time)
-
         expected_result = [
             self.process_tracker.process_status_failed,
             self.provided_end_date,
@@ -2169,3 +2166,19 @@ def test_determine_process_targets(self):
         expected_targets = self.process_tracker.targets
 
         return expected_targets == given_targets
+
+    def test_ensure_nulls_caught_on_instantiation(self):
+        """
+        With the adding of the ability of have a process_tracking_id we have to allow for nulled values for process_name
+        and process_type.  If ProcessTracker is instantiated with either (or both) being null, an exception should be
+        raised.
+        :return:
+        """
+
+        with self.assertRaises(Exception) as context:
+
+            ProcessTracker()
+
+        return self.assertTrue(
+            "process_name and process_type must be set." in str(context.exception)
+        )