Fix execution tests

Don't rely on database ids in execution test assertions.

Re #2431

execution_tests/simple_importer_on_server/execution_test.py

@@ -49,7 +49,6 @@ def test_execution(self):
         # Check that DS1.sqlite is empty
         with DatabaseMapping(self._db_url) as db_map:
             entities = db_map.get_items("entity")
-            print(entities)
             self.assertEqual(0, len(entities))
         completed = subprocess.run(
             (
@@ -66,16 +65,7 @@ def test_execution(self):
         # Check that entities are now in DB
         with DatabaseMapping(self._db_url) as db_map:
             entities = db_map.get_items("entity")
-            self.assertEqual(3, len(entities))
-            for entity in entities:
-                if entity["id"] == 1:
-                    self.assertEqual("Factory1", entity["name"])
-                elif entity["id"] == 2:
-                    self.assertEqual("Factory2", entity["name"])
-                elif entity["id"] == 3:
-                    self.assertEqual("Factory3", entity["name"])
-                else:
-                    self.fail()
+            self.assertCountEqual([entity["name"] for entity in entities], ["Factory1", "Factory2", "Factory3"])
 
 
 if __name__ == '__main__':