Updates to address warnings

activity_factory.py

@@ -5,7 +5,7 @@
 #   Name change
 #
 #   generate_range_check(n,a,b) - col between min / max (must be int)
-#   generate_date_range_check(n,a,b) - col between min / max (must be date)
+#   generate_date_range_check(n,a,b) - col between min / max (must be date ISO format)
 #   generate_required_check(n) - 0 len string
 #   generate_regex_check(n,regex_string) - any valid regex
 #   generate_regex_money_check(n) - money format 
@@ -74,7 +74,7 @@ def _regex_check(n):
 #   False = no match
 #
 def generate_regex_money_check(n):
-    p = re.compile('/^-?\d+(,\d{3})*(\.\d{1,2})?$/')
+    p = re.compile("/^-{0,1}[0-9]+\\.{0,1}[0-9]{0,2}$/")
     def _regex_money_check(n):
         if p.match(n) == None:
             return False

data_reader.py

@@ -34,13 +34,25 @@ def filename(self):
 
     def number_of_columns(self):
         return self._number_of_columns
-
-    def flight_test(self,test_flights,config,findings):
 
+    #
+    #   flight_test - apply the test_flights to the current data reader object (file to test)
+    #   inputs: 
+    #           test_flights object
+    #           config object
+    #           findings object
+    #
+    def flight_test(self,test_flights,config,findings):
+        #
+        if config.stats_enabled:
+            print(f"stats on") 
+        #
+        #   open the file
+        #
         with open(self.filename()) as f:
             reader = csv.reader(f)
             #
-            #   skip the first row because it iss the header
+            #   skip the first row because it is the header
             #
             next(reader)
             line_counter = 0
@@ -57,7 +69,7 @@ def flight_test(self,test_flights,config,findings):
                     continue
 
 
-                # valid row structure
+                # we have a valid row structure
                 # apply tests to the appropriate columns
                 for flight in test_flights:
 

flight.py

@@ -7,6 +7,7 @@
 def build_flight_list(column_list,config_options,findings):
     flight_list = []
     column_number = 0
+    working_flight = object
     for working_option in config_options:
         if "test" in working_option:
             #