Allow non-required fields to be None (#71)

* Allow non-required fields to be None

* Update pybotvac/account.py

Make `status` and `launched_from` more permissive

Author: Santobert

pybotvac/account.py

@@ -29,52 +29,52 @@
         Required("serial"): str,
         "prefix": Any(str, None),
         Required("name"): str,
-        "model": str,
+        "model": Any(str, None),
         Required("secret_key"): str,
         "purchased_at": Any(str, None),
         "linked_at": Any(str, None),
         Required("traits"): list,
         # Everything below this line is not documented, but still present
-        "firmware": str,
-        "timezone": str,
+        "firmware": Any(str, None),
+        "timezone": Any(str, None),
         Required("nucleo_url"): Url,
-        "mac_address": str,
-        "created_at": str,
+        "mac_address": Any(str, None),
+        "created_at": Any(str, None),
     },
     extra=ALLOW_EXTRA,
 )
 MAP_SCHEMA = Schema(
     {
-        "version": int,
+        "version": Any(int, None),
         Required("id"): str,
         Required("url"): Url,
-        "url_valid_for_seconds": int,
-        Optional("run_id"): str,  # documented, but  not present
-        "status": Any("complete", "incomplete", "canceled"),
-        "launched_from": Any("robot", "schedule", "app"),
+        "url_valid_for_seconds": Any(int, None),
+        Optional("run_id"): str,  # documented, but not present
+        "status": Any(str, None),
+        "launched_from": Any(str, None),
         "error": Any(str, None),
-        "category": int,
-        "mode": int,
-        "modifier": int,
-        "start_at": str,
-        "end_at": str,
+        "category": Any(int, None),
+        "mode": Any(int, None),
+        "modifier": Any(int, None),
+        "start_at": Any(str, None),
+        "end_at": Any(str, None),
         "end_orientation_relative_degrees": All(int, Range(min=0, max=360)),
         "run_charge_at_start": All(int, Range(min=0, max=100)),
         "run_charge_at_end": All(int, Range(min=0, max=100)),
-        "suspended_cleaning_charging_count": int,
-        "time_in_suspended_cleaning": int,
-        "time_in_error": int,
-        "time_in_pause": int,
-        "cleaned_area": float,
-        "base_count": int,
-        "is_docked": bool,
-        "delocalized": bool,
+        "suspended_cleaning_charging_count": Any(int, None),
+        "time_in_suspended_cleaning": Any(int, None),
+        "time_in_error": Any(int, None),
+        "time_in_pause": Any(int, None),
+        "cleaned_area": Any(float, None),
+        "base_count": Any(int, None),
+        "is_docked": Any(bool, None),
+        "delocalized": Any(bool, None),
         # Everything below this line is not documented, but still present
-        "generated_at": str,
+        "generated_at": Any(str, None),
         "persistent_map_id": Any(str, None),
         "cleaned_with_persistent_map_id": Any(str, None),
-        "valid_as_persistent_map": bool,
-        "navigation_mode": int,
+        "valid_as_persistent_map": Any(bool, None),
+        "navigation_mode": Any(int, None),
     },
     extra=ALLOW_EXTRA,
 )
@@ -87,8 +87,8 @@
         Required("id"): str,
         Required("name"): str,
         Required("url"): Url,
-        "raw_floor_map_url": Url,
-        "url_valid_for_seconds": int,
+        "raw_floor_map_url": Any(Url, None),
+        "url_valid_for_seconds": Any(int, None),
     },
     extra=ALLOW_EXTRA,
 )