jsontogeojson: Fix crash

libcoveofds/geojson.py

@@ -101,7 +101,7 @@ def _dereference_object(self, ref, list):
 
         if "id" in ref:
             for item in list:
-                if item.get("id") == ref["id"]:
+                if isinstance(item, dict) and item.get("id") == ref["id"]:
                     return item
 
         return ref

tests/fixtures/json_to_geojson/dont_crash_1.json

@@ -0,0 +1,24 @@
+{
+  "networks": [
+    {
+      "id": "a096d627-72e1-4f9b-b129-951b1737bff4",
+      "name": "Ghana Fibre Network",
+      "organisations": [
+        "my mate down the pub"
+      ],
+      "phases": [
+        {
+          "funders": [
+            {"id":  "1"}
+          ]
+        }
+      ],
+      "links": [
+        {
+          "rel": "describedby",
+          "href": "https://raw.githubusercontent.com/Open-Telecoms-Data/open-fibre-data-standard/0__1__0__beta/schema/network-schema.json"
+        }
+      ]
+    }
+  ]
+}

tests/test_json_to_geojson.py

@@ -65,3 +65,20 @@ def test_json_to_geojson(filename):
     with open(meta_expected_filename) as fp:
         meta_expected_data = json.load(fp)
     assert meta_expected_data == converter.get_meta_json()
+
+
+def test_dont_crash_1():
+    """Just put as much bad stuff as possible in the input and make sure it doesn't crash!
+    We don't care about testing output. Other tests can do that."""
+    json_filename = os.path.join(
+        os.path.dirname(os.path.realpath(__file__)),
+        "fixtures",
+        "json_to_geojson",
+        "dont_crash_1.json",
+    )
+
+    with open(json_filename) as fp:
+        json_data = json.load(fp)
+
+    converter = JSONToGeoJSONConverter()
+    converter.process_package(json_data)