Add None check to trip_parser.py

.gitignore

@@ -151,4 +151,10 @@ cars.json
 certs/
 *.log.*
 
-.vscode/
+.vscode/
+
+tests/data/otp_test2.bin
+tests/data/test_car.json
+tests/mypeugeot.apk
+tests/mypeugeot.apk.bz2
+tests/test_charge_control.json

psa_car_controller/psacc/application/trip_parser.py

@@ -25,19 +25,35 @@ def __get_energy_method(self) -> [Callable]:
 
     @staticmethod
     def get_thermal_consumption(start, end):
-        return [0, start[LEVEL_FUEL] - end[LEVEL_FUEL]]
+        try:
+            if start[LEVEL_FUEL] is None or end[LEVEL_FUEL] is None:
+                return [0, 0]
+            return [0, start[LEVEL_FUEL] - end[LEVEL_FUEL]]
+        except (KeyError, IndexError, TypeError):
+            return [0, 0]
 
     @staticmethod
     def get_elec_consumption(start, end):
-        return [start[LEVEL] - end[LEVEL], 0]
+        try:
+            start_level = start[LEVEL]
+            end_level = end[LEVEL]
+            if start_level is None or end_level is None:
+                return [0, 0]
+            return [start_level - end_level, 0]
+        except (KeyError, IndexError, TypeError):  # TypeError handles None start/end
+            return [0, 0]
 
     @staticmethod
     def get_hybrid_consumption(start, end):
         res = []
-        for energy in [LEVEL, LEVEL_FUEL]:
-            if start[energy] is not None and end[energy] is not None:
-                res.append(start[energy] - end[energy])
-            else:
+        try:
+            for energy in [LEVEL, LEVEL_FUEL]:
+                if start[energy] is not None and end[energy] is not None:
+                    res.append(start[energy] - end[energy])
+                else:
+                    res.append(0)
+        except (KeyError, IndexError, TypeError):
+            while len(res) < 2:
                 res.append(0)
         return res
 
@@ -50,8 +66,8 @@ def __is_refuel_or_recharging(self, start, end, distance):
             logger.debugv("charge detected")
             return True
         return False
-
     # pylint: disable=unused-argument
+
     def __is_refuel(self, start, end, distance):
         fuel_consumption = self.get_level_consumption(start, end)[1]
         if fuel_consumption < 0:
@@ -60,6 +76,13 @@ def __is_refuel(self, start, end, distance):
         return False
 
     def __is_recharging(self, start, end, distance):
+        try:
+            start_level = start[LEVEL]
+            end_level = end[LEVEL]
+            if start_level is None or end_level is None:
+                return False
+        except (KeyError, IndexError, TypeError):
+            return False
         decharge = self.get_level_consumption(start, end)[0]
         return TripParser.is_recharging(decharge, distance)