bug fixes

Author: ElJocho

mapswipe_workers/mapswipe_workers/project_types/arbitrary_geometry/grouping_functions.py

@@ -40,7 +40,7 @@ def group_input_geometries(input_geometries_file, group_size):
         try:
             groups[group_id_string]
         except KeyError:
-            groups[group_id_string] = {"feature_ids": [], "feature": []}
+            groups[group_id_string] = {"feature_ids": [], "features": []}
 
         # we use a new id here based on the count
         # since we are not sure that GetFID returns unique values

mapswipe_workers/mapswipe_workers/project_types/arbitrary_geometry/project.py

@@ -139,7 +139,8 @@ def validate_geometries(self):
         for feature in layer:
             feat_geom = feature.GetGeometryRef()
             geom_name = feat_geom.GetGeometryName()
-            fid = feature.GetFID
+            fid = feature.GetFID()
+
             if not feat_geom.IsValid():
                 layer.DeleteFeature(fid)
                 logger.warning(
@@ -194,7 +195,7 @@ def create_groups(self):
 
         for group_id, item in raw_groups.items():
             group = Group(self, group_id)
-            group.create_tasks(item["feature_ids"], item["feature"])
+            group.create_tasks(item["feature_ids"], item["features"])
 
             # only append valid groups
             if group.is_valid():

mapswipe_workers/tests/integration/test_create_footprint_project.py

@@ -33,12 +33,12 @@ def test_create_footprint_project(self):
             self.assertEqual(result, element)
 
             # check if usernames made it to postgres
-            if element != "footprint_link":
+            if element != "test_footprint_link":
                 query = """
                     SELECT count(*)
                     FROM tasks
                     WHERE project_id = %s
-                        and project_type_specifics->username is not null
+                        and project_type_specifics->'properties'->'username' is not null
                 """
                 result = pg_db.retr_query(query, [element])[0][0]
                 self.assertGreater(result, 0)