Removing contexts now also reassign to parent or remove references to the context.

src/ifcopenshell-python/test/api/context/test_edit_context.py

@@ -0,0 +1,32 @@
+import test.bootstrap
+import ifcopenshell.api
+
+
+class TestEditContext(test.bootstrap.IFC4):
+    def test_editing_a_context(self):
+        context = self.file.createIfcGeometricRepresentationContext()
+        ifcopenshell.api.run("context.edit_context", self.file, context=context, attributes={
+            "ContextIdentifier": "ContextIdentifier",
+            "ContextType": "ContextType",
+            "CoordinateSpaceDimension": 1,
+            "Precision": 1,
+        })
+        assert context.ContextIdentifier == "ContextIdentifier"
+        assert context.ContextType == "ContextType"
+        assert context.CoordinateSpaceDimension == 1
+        assert context.Precision == 1
+
+    def test_editing_a_subcontext(self):
+        subcontext = self.file.createIfcGeometricRepresentationSubcontext()
+        ifcopenshell.api.run("context.edit_context", self.file, context=subcontext, attributes={
+            "ContextIdentifier": "ContextIdentifier",
+            "ContextType": "ContextType",
+            "TargetScale": 0.5,
+            "TargetView": "MODEL_VIEW",
+            "UserDefinedTargetView": "UserDefinedTargetView",
+        })
+        assert subcontext.ContextIdentifier == "ContextIdentifier"
+        assert subcontext.ContextType == "ContextType"
+        assert subcontext.TargetScale == 0.5
+        assert subcontext.TargetView == "MODEL_VIEW"
+        assert subcontext.UserDefinedTargetView == "UserDefinedTargetView"

src/ifcopenshell-python/test/api/context/test_remove_context.py

@@ -0,0 +1,32 @@
+import test.bootstrap
+import ifcopenshell.api
+
+
+class TestRemoveContext(test.bootstrap.IFC4):
+    def test_removing_a_context(self):
+        context = self.file.createIfcGeometricRepresentationContext()
+        ifcopenshell.api.run("context.remove_context", self.file, context=context)
+        assert len(self.file.by_type("IfcGeometricRepresentationContext")) == 0
+
+    def test_removing_a_context_with_subcontexts(self):
+        context = self.file.createIfcGeometricRepresentationContext()
+        subcontext = self.file.createIfcGeometricRepresentationSubcontext()
+        subcontext.ParentContext = context
+        ifcopenshell.api.run("context.remove_context", self.file, context=context)
+        assert len(self.file.by_type("IfcGeometricRepresentationContext")) == 0
+
+    def test_removing_a_subcontext_and_reassigning_references_to_its_parent(self):
+        context = self.file.createIfcGeometricRepresentationContext()
+        subcontext = self.file.createIfcGeometricRepresentationSubcontext()
+        subcontext.ParentContext = context
+        representation = self.file.createIfcRepresentation(ContextOfItems=subcontext)
+        ifcopenshell.api.run("context.remove_context", self.file, context=subcontext)
+        assert len(self.file.by_type("IfcGeometricRepresentationSubcontext")) == 0
+        assert representation in self.file.get_inverse(context)
+
+    def test_removing_a_context_with_references(self):
+        subcontext = self.file.createIfcGeometricRepresentationSubContext()
+        representation = self.file.createIfcRepresentation(ContextOfItems=subcontext)
+        map_conversion = self.file.createIfcMapConversion(SourceCRS=subcontext)
+        ifcopenshell.api.run("context.remove_context", self.file, context=subcontext)
+        assert len([e for e in self.file]) == 0