c2cgeoportal-admin 2.6.0__py3-none-any.whl → 2.8.1.180__py3-none-any.whl

tests/test_functionalities.py

@@ -9,15 +9,18 @@ from . import AbstractViewsTests
 
 @pytest.fixture(scope="function")
 @pytest.mark.usefixtures("dbsession", "transact")
-def functionality_test_data(dbsession, transact):
+def functionality_test_data(dbsession, transact, settings):
     del transact
 
     from c2cgeoportal_commons.models.main import Functionality
 
     functionalities = []
     for i in range(0, 4):
-        functionality = Functionality(name="functionality_{}".format(i), value="value_{}".format(i))
-        functionality.description = "description_{}".format(i)
+        functionality = Functionality(
+            settings["admin_interface"]["available_functionalities"][i]["name"],
+            value=f"value_{i}",
+        )
+        functionality.description = f"description_{i}"
         dbsession.add(functionality)
         functionalities.append(functionality)
 
@@ -28,7 +31,6 @@ def functionality_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("functionality_test_data", "test_app")
 class TestFunctionality(AbstractViewsTests):
-
     _prefix = "/admin/functionalities"
 
     def test_index_rendering(self, test_app):
@@ -47,10 +49,10 @@ class TestFunctionality(AbstractViewsTests):
 
     def test_grid_search(self, test_app):
         # search on functionality name
-        self.check_search(test_app, "functionality_0", total=1)
+        self.check_search(test_app, "default_basemap", total=1)
 
     def test_submit_new(self, dbsession, test_app):
-        from c2cgeoportal_commons.models.main import Functionality
+        from c2cgeoportal_commons.models.main import Functionality, Log, LogAction
 
         resp = test_app.post(
             "/admin/functionalities/new",
@@ -63,9 +65,19 @@ class TestFunctionality(AbstractViewsTests):
         ).group(1)
         assert functionality.name == "new_name"
 
-    def test_edit(self, test_app, functionality_test_data):
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.INSERT
+        assert log.element_type == "functionality"
+        assert log.element_id == functionality.id
+        assert log.element_name == functionality.name
+        assert log.username == "test_user"
+
+    def test_edit(self, test_app, functionality_test_data, dbsession):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         functionality = functionality_test_data["functionalities"][0]
-        resp = test_app.get("/admin/functionalities/{}".format(functionality.id), status=200)
+        resp = test_app.get(f"/admin/functionalities/{functionality.id}", status=200)
         form = resp.form
         assert str(functionality.id) == self.get_first_field_named(form, "id").value
         assert "hidden" == self.get_first_field_named(form, "id").attrs["type"]
@@ -74,25 +86,51 @@ class TestFunctionality(AbstractViewsTests):
         assert form.submit().status_int == 302
         assert functionality.description == "new_description"
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.UPDATE
+        assert log.element_type == "functionality"
+        assert log.element_id == functionality.id
+        assert log.element_name == functionality.name
+        assert log.username == "test_user"
+
     def test_delete(self, test_app, functionality_test_data, dbsession):
-        from c2cgeoportal_commons.models.main import Functionality
+        from c2cgeoportal_commons.models.main import Functionality, Log, LogAction
 
         functionality = functionality_test_data["functionalities"][0]
         deleted_id = functionality.id
-        test_app.delete("/admin/functionalities/{}".format(deleted_id), status=200)
+        test_app.delete(f"/admin/functionalities/{deleted_id}", status=200)
         assert dbsession.query(Functionality).get(deleted_id) is None
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.DELETE
+        assert log.element_type == "functionality"
+        assert log.element_id == functionality.id
+        assert log.element_name == functionality.name
+        assert log.username == "test_user"
+
     def test_duplicate(self, functionality_test_data, test_app, dbsession):
         from c2cgeoportal_commons.models.main import Functionality
 
         functionality = functionality_test_data["functionalities"][3]
-        resp = test_app.get("/admin/functionalities/{}/duplicate".format(functionality.id), status=200)
+
+        resp = test_app.get(f"/admin/functionalities/{functionality.id}/duplicate", status=200)
+
         form = resp.form
-        assert "" == self.get_first_field_named(form, "id").value
-        self.set_first_field_named(form, "name", "clone")
+        assert form["name"].value == functionality.name
+        assert form["description"].value == functionality.description
+        assert form["value"].value == functionality.value
+        form["value"].value = "another_value"
         resp = form.submit("submit")
+
         assert resp.status_int == 302
-        functionality = dbsession.query(Functionality).filter(Functionality.name == "clone").one()
+        functionality = (
+            dbsession.query(Functionality)
+            .filter(Functionality.name == functionality.name)
+            .filter(Functionality.value == "another_value")
+            .one()
+        )
         assert str(functionality.id) == re.match(
             r"http://localhost/admin/functionalities/(.*)\?msg_col=submit_ok", resp.location
         ).group(1)

tests/test_home.py

@@ -5,7 +5,6 @@ from . import AbstractViewsTests
 
 @pytest.mark.usefixtures("test_app")
 class TestHome(AbstractViewsTests):
-
     _prefix = "/admin/"
 
     def test_index_rendering(self, test_app):

tests/test_interface.py

@@ -16,14 +16,14 @@ def interface_test_data(dbsession, transact):
 
     themes = []
     for i in range(0, 5):
-        theme = Theme(name="theme_{}".format(i), ordering=1)
+        theme = Theme(name=f"theme_{i}", ordering=1)
         themes.append(theme)
 
-    servers = [OGCServer(name="server_{}".format(i)) for i in range(0, 4)]
+    servers = [OGCServer(name=f"server_{i}") for i in range(0, 4)]
 
     layers = []
     for i in range(0, 15):
-        layer = LayerWMS(name="layer_wms_{}".format(i))
+        layer = LayerWMS(name=f"layer_wms_{i}")
         layer.public = 1 == i % 2
         layer.ogc_server = servers[i % 4]
         dbsession.add(layer)
@@ -31,7 +31,7 @@ def interface_test_data(dbsession, transact):
 
     interfaces = []
     for i in range(0, 5):
-        interface = Interface(name="interface_{}".format(i), description="description_{}".format(i))
+        interface = Interface(name=f"interface_{i}", description=f"description_{i}")
         interface.themes = [themes[i % 2], themes[(i + 5) % 5]]
         interface.layers = [layers[i % 2], layers[(i + 4) % 5]]
 
@@ -45,7 +45,6 @@ def interface_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("interface_test_data", "test_app")
 class TestInterface(AbstractViewsTests):
-
     _prefix = "/admin/interfaces"
 
     def test_index_rendering(self, test_app):
@@ -82,7 +81,7 @@ class TestInterface(AbstractViewsTests):
         self.check_search(test_app, "interface_0", total=1)
 
     def test_submit_new(self, dbsession, test_app):
-        from c2cgeoportal_commons.models.main import Interface
+        from c2cgeoportal_commons.models.main import Interface, Log, LogAction
 
         resp = test_app.post(
             "/admin/interfaces/new", {"name": "new_name", "description": "new description"}, status=302
@@ -94,14 +93,22 @@ class TestInterface(AbstractViewsTests):
         ).group(1)
         assert interface.name == "new_name"
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.INSERT
+        assert log.element_type == "interface"
+        assert log.element_id == interface.id
+        assert log.element_name == interface.name
+        assert log.username == "test_user"
+
     def test_edit(self, test_app, interface_test_data, dbsession):
-        from c2cgeoportal_commons.models.main import Interface
+        from c2cgeoportal_commons.models.main import Interface, Log, LogAction
 
         interface = interface_test_data["interfaces"][0]
         descriptions = "{}, {}".format(
             interface_test_data["interfaces"][0].description, interface_test_data["interfaces"][1].description
         )
-        resp = test_app.get("/admin/interfaces/{}".format(interface.id), status=200)
+        resp = test_app.get(f"/admin/interfaces/{interface.id}", status=200)
         form = resp.form
         form["description"] = descriptions
         assert str(interface.id) == self.get_first_field_named(form, "id").value
@@ -110,16 +117,32 @@ class TestInterface(AbstractViewsTests):
         assert form.submit().status_int == 302
         assert len(dbsession.query(Interface).filter(Interface.description == descriptions).all()) == 1
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.UPDATE
+        assert log.element_type == "interface"
+        assert log.element_id == interface.id
+        assert log.element_name == interface.name
+        assert log.username == "test_user"
+
     def test_delete(self, test_app, interface_test_data, dbsession):
-        from c2cgeoportal_commons.models.main import Interface
+        from c2cgeoportal_commons.models.main import Interface, Log, LogAction
 
         interface = interface_test_data["interfaces"][0]
-        test_app.delete("/admin/interfaces/{}".format(interface.id), status=200)
+        test_app.delete(f"/admin/interfaces/{interface.id}", status=200)
         assert len(dbsession.query(Interface).filter(Interface.id == interface.id).all()) == 0
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.DELETE
+        assert log.element_type == "interface"
+        assert log.element_id == interface.id
+        assert log.element_name == interface.name
+        assert log.username == "test_user"
+
     def test_duplicate(self, interface_test_data, test_app):
         interface = interface_test_data["interfaces"][3]
-        resp = test_app.get("/admin/interfaces/{}/duplicate".format(interface.id), status=200)
+        resp = test_app.get(f"/admin/interfaces/{interface.id}/duplicate", status=200)
         form = resp.form
         assert "" == self.get_first_field_named(form, "id").value
         assert str(interface.description or "") == "description_3"

tests/test_layer_groups.py

@@ -22,7 +22,7 @@ def layer_groups_test_data(dbsession, transact):
 
     groups = []
     for i in range(0, 12):
-        group = LayerGroup(name="groups_{num:02d}".format(num=i), is_expanded=False)
+        group = LayerGroup(name=f"groups_{i:02d}")
         group.metadatas = [
             Metadata(name=metadatas_protos[id][0], value=metadatas_protos[id][1])
             for id in [i % 3, (i + 2) % 3]
@@ -60,7 +60,6 @@ def layer_groups_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("layer_groups_test_data", "test_app")
 class TestLayersGroups(TestTreeGroup):
-
     _prefix = "/admin/layer_groups"
 
     def test_index_rendering(self, test_app):
@@ -73,7 +72,6 @@ class TestLayersGroups(TestTreeGroup):
             ("id", "id", "true"),
             ("name", "Name"),
             ("description", "Description"),
-            ("is_expanded", "Expanded"),
             ("parents_relation", "Parents", "false"),
             ("metadatas", "Metadatas", "false"),
         ]
@@ -95,6 +93,8 @@ class TestLayersGroups(TestTreeGroup):
         self.check_search(test_app, "copyable", total=8)
 
     def test_edit(self, test_app, layer_groups_test_data, dbsession):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         group = layer_groups_test_data["groups"][1]
 
         form = self.get_item(test_app, group.id).form
@@ -103,8 +103,6 @@ class TestLayersGroups(TestTreeGroup):
         assert "hidden" == self.get_first_field_named(form, "id").attrs["type"]
         assert group.name == self.get_first_field_named(form, "name").value
         assert str(group.description or "") == self.get_first_field_named(form, "description").value
-        assert group.is_expanded is False
-        assert group.is_expanded == form["is_expanded"].checked
 
         self.check_children(
             form,
@@ -121,7 +119,6 @@ class TestLayersGroups(TestTreeGroup):
         new_values = {
             "name": "new_name",
             "description": "new description",
-            "is_expanded": True,
         }
         for key, value in new_values.items():
             self.set_first_field_named(form, key, value)
@@ -138,13 +135,21 @@ class TestLayersGroups(TestTreeGroup):
             else:
                 assert str(value or "") == str(getattr(group, key) or "")
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.UPDATE
+        assert log.element_type == "layergroup"
+        assert log.element_id == group.id
+        assert log.element_name == group.name
+        assert log.username == "test_user"
+
     def test_post_new_with_children_invalid(self, test_app, layer_groups_test_data):
         """
         Check there is no rendering error when validation fails.
         """
         groups = layer_groups_test_data["groups"]
         resp = test_app.post(
-            "{}/new".format(self._prefix),
+            f"{self._prefix}/new",
             (
                 ("_charset_", "UTF-8"),
                 ("__formid__", "deform"),
@@ -163,9 +168,11 @@ class TestLayersGroups(TestTreeGroup):
         assert "Required" == resp.html.select_one(".item-name .help-block").getText().strip()
 
     def test_post_new_with_children_success(self, test_app, dbsession, layer_groups_test_data):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         groups = layer_groups_test_data["groups"]
         resp = test_app.post(
-            "{}/new".format(self._prefix),
+            f"{self._prefix}/new",
             (
                 ("_charset_", "UTF-8"),
                 ("__formid__", "deform"),
@@ -206,11 +213,21 @@ class TestLayersGroups(TestTreeGroup):
             rel.treeitem_id for rel in group.children_relation
         ]
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.INSERT
+        assert log.element_type == "layergroup"
+        assert log.element_id == group.id
+        assert log.element_name == group.name
+        assert log.username == "test_user"
+
     def test_post_with_ancestor(self, layer_groups_test_data, test_app):
-        """Check that ancestors are refused to avoid cycles"""
+        """
+        Check that ancestors are refused to avoid cycles.
+        """
         groups = layer_groups_test_data["groups"]
         resp = test_app.post(
-            "{}/{}".format(self._prefix, groups[3].id),
+            f"{self._prefix}/{groups[3].id}",
             (
                 ("_charset_", "UTF-8"),
                 ("__formid__", "deform"),
@@ -228,7 +245,7 @@ class TestLayersGroups(TestTreeGroup):
             status=200,
         )
         assert (
-            "Value {} does not exist in table treeitem or is not allowed to avoid cycles".format(groups[1].id)
+            f"Value {groups[1].id} does not exist in table treeitem or is not allowed to avoid cycles"
             == resp.html.select_one(".item-children_relation + .help-block").getText().strip()
         )
 
@@ -237,7 +254,7 @@ class TestLayersGroups(TestTreeGroup):
 
         group = layer_groups_test_data["groups"][1]
 
-        resp = test_app.get("{}/{}/duplicate".format(self._prefix, group.id), status=200)
+        resp = test_app.get(f"{self._prefix}/{group.id}/duplicate", status=200)
         form = resp.form
 
         group = dbsession.query(LayerGroup).filter(LayerGroup.id == group.id).one()
@@ -245,8 +262,6 @@ class TestLayersGroups(TestTreeGroup):
         assert "" == self.get_first_field_named(form, "id").value
         assert group.name == self.get_first_field_named(form, "name").value
         assert str(group.description or "") == self.get_first_field_named(form, "description").value
-        assert group.is_expanded is False
-        assert group.is_expanded == form["is_expanded"].checked
 
         self.check_children(
             form,
@@ -263,7 +278,7 @@ class TestLayersGroups(TestTreeGroup):
         duplicated = dbsession.query(LayerGroup).filter(LayerGroup.name == "duplicated").one()
 
         assert str(duplicated.id) == re.match(
-            r"http://localhost{}/(.*)\?msg_col=submit_ok".format(self._prefix), resp.location
+            rf"http://localhost{self._prefix}/(.*)\?msg_col=submit_ok", resp.location
         ).group(1)
         assert duplicated.id != group.id
         assert duplicated.children_relation[0].id != group.children_relation[0].id
@@ -271,41 +286,56 @@ class TestLayersGroups(TestTreeGroup):
 
     def test_unicity_validator(self, layer_groups_test_data, test_app):
         group = layer_groups_test_data["groups"][1]
-        resp = test_app.get("{}/{}/duplicate".format(self._prefix, group.id), status=200)
+        resp = test_app.get(f"{self._prefix}/{group.id}/duplicate", status=200)
 
         resp = resp.form.submit("submit")
 
-        self._check_submission_problem(resp, "{} is already used.".format(group.name))
+        self._check_submission_problem(resp, f"{group.name} is already used.")
 
     def test_delete(self, test_app, dbsession, layer_groups_test_data):
-        from c2cgeoportal_commons.models.main import LayerGroup, LayergroupTreeitem, TreeGroup, TreeItem
+        from c2cgeoportal_commons.models.main import (
+            LayerGroup,
+            LayergroupTreeitem,
+            Log,
+            LogAction,
+            TreeGroup,
+            TreeItem,
+        )
 
-        group_id = layer_groups_test_data["groups"][9].id
+        group = layer_groups_test_data["groups"][9]
 
         assert (
             3
-            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treegroup_id == group_id).count()
+            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treegroup_id == group.id).count()
         )
 
         assert (
             1
-            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treeitem_id == group_id).count()
+            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treeitem_id == group.id).count()
         )
 
-        test_app.delete("/admin/layer_groups/{}".format(group_id), status=200)
+        test_app.delete(f"/admin/layer_groups/{group.id}", status=200)
 
         dbsession.expire_all()
 
-        assert dbsession.query(LayerGroup).get(group_id) is None
-        assert dbsession.query(TreeGroup).get(group_id) is None
-        assert dbsession.query(TreeItem).get(group_id) is None
+        assert dbsession.query(LayerGroup).get(group.id) is None
+        assert dbsession.query(TreeGroup).get(group.id) is None
+        assert dbsession.query(TreeItem).get(group.id) is None
 
         assert (
             0
-            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treegroup_id == group_id).count()
+            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treegroup_id == group.id).count()
         )
 
         assert (
             0
-            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treeitem_id == group_id).count()
+            == dbsession.query(LayergroupTreeitem).filter(LayergroupTreeitem.treeitem_id == group.id).count()
         )
+
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.DELETE
+        assert log.element_type == "layergroup"
+        assert log.element_id == group.id
+        assert log.element_name == group.name
+        assert log.username == "test_user"

tests/test_layers_vectortiles.py

@@ -14,13 +14,13 @@ def layer_vectortiles_test_data(dbsession, transact):
 
     from c2cgeoportal_commons.models.main import LayerVectorTiles, OGCServer
 
-    servers = [OGCServer(name="server_{}".format(i)) for i in range(0, 4)]
+    servers = [OGCServer(name=f"server_{i}") for i in range(0, 4)]
     for i, server in enumerate(servers):
-        server.url = "http://wms.geo.admin.ch_{}".format(i)
+        server.url = f"http://wms.geo.admin.ch_{i}"
         server.image_type = "image/jpeg" if i % 2 == 0 else "image/png"
 
     def layer_builder(i):
-        name = "layer_vectortiles_{}".format(i)
+        name = f"layer_vectortiles_{i}"
         layer = LayerVectorTiles(name=name)
         layer.layer = name
         layer.public = 1 == i % 2
@@ -38,7 +38,6 @@ def layer_vectortiles_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("layer_vectortiles_test_data", "test_app")
 class TestLayerVectortiles(AbstractViewsTests):
-
     _prefix = "/admin/layers_vectortiles"
 
     def test_index_rendering(self, test_app):
@@ -106,6 +105,8 @@ class TestLayerVectortiles(AbstractViewsTests):
         assert form["public"].checked
 
     def test_edit(self, test_app, layer_vectortiles_test_data, dbsession):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         layer = layer_vectortiles_test_data["layers"][0]
 
         form = self.get_item(test_app, layer.id).form
@@ -122,11 +123,11 @@ class TestLayerVectortiles(AbstractViewsTests):
         assert str(layer.xyz or "") == form["xyz"].value
 
         interfaces = layer_vectortiles_test_data["interfaces"]
-        assert set((interfaces[0].id, interfaces[2].id)) == set(i.id for i in layer.interfaces)
+        assert {interfaces[0].id, interfaces[2].id} == {i.id for i in layer.interfaces}
         self._check_interfaces(form, interfaces, layer)
 
         ras = layer_vectortiles_test_data["restrictionareas"]
-        assert set((ras[0].id, ras[2].id)) == set(i.id for i in layer.restrictionareas)
+        assert {ras[0].id, ras[2].id} == {i.id for i in layer.restrictionareas}
         self._check_restrictionsareas(form, ras, layer)
 
         new_values = {
@@ -146,7 +147,7 @@ class TestLayerVectortiles(AbstractViewsTests):
 
         resp = form.submit("submit")
         assert str(layer.id) == re.match(
-            r"http://localhost{}/(.*)\?msg_col=submit_ok".format(self._prefix), resp.location
+            rf"http://localhost{self._prefix}/(.*)\?msg_col=submit_ok", resp.location
         ).group(1)
 
         dbsession.expire(layer)
@@ -155,13 +156,19 @@ class TestLayerVectortiles(AbstractViewsTests):
                 assert value == getattr(layer, key)
             else:
                 assert str(value or "") == str(getattr(layer, key) or "")
-        assert set([interfaces[1].id, interfaces[3].id]) == set(
-            [interface.id for interface in layer.interfaces]
-        )
-        assert set([ras[1].id, ras[3].id]) == set([ra.id for ra in layer.restrictionareas])
+        assert {interfaces[1].id, interfaces[3].id} == {interface.id for interface in layer.interfaces}
+        assert {ras[1].id, ras[3].id} == {ra.id for ra in layer.restrictionareas}
+
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.UPDATE
+        assert log.element_type == "layer_vectortiles"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"
 
     def test_submit_new(self, dbsession, test_app, layer_vectortiles_test_data):
-        from c2cgeoportal_commons.models.main import LayerVectorTiles
+        from c2cgeoportal_commons.models.main import LayerVectorTiles, Log, LogAction
 
         resp = test_app.post(
             "/admin/layers_vectortiles/new",
@@ -180,12 +187,20 @@ class TestLayerVectortiles(AbstractViewsTests):
             r"http://localhost/admin/layers_vectortiles/(.*)\?msg_col=submit_ok", resp.location
         ).group(1)
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.INSERT
+        assert log.element_type == "layer_vectortiles"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"
+
     def test_duplicate(self, layer_vectortiles_test_data, test_app, dbsession):
         from c2cgeoportal_commons.models.main import LayerVectorTiles
 
         layer = layer_vectortiles_test_data["layers"][3]
 
-        resp = test_app.get("/admin/layers_vectortiles/{}/duplicate".format(layer.id), status=200)
+        resp = test_app.get(f"/admin/layers_vectortiles/{layer.id}/duplicate", status=200)
         form = resp.form
 
         assert "" == self.get_first_field_named(form, "id").value
@@ -198,7 +213,7 @@ class TestLayerVectortiles(AbstractViewsTests):
         assert str(layer.style or "") == form["style"].value
         assert str(layer.xyz or "") == form["xyz"].value
         interfaces = layer_vectortiles_test_data["interfaces"]
-        assert set((interfaces[3].id, interfaces[1].id)) == set(i.id for i in layer.interfaces)
+        assert {interfaces[3].id, interfaces[1].id} == {i.id for i in layer.interfaces}
         self._check_interfaces(form, interfaces, layer)
 
         self.set_first_field_named(form, "name", "clone")
@@ -214,12 +229,20 @@ class TestLayerVectortiles(AbstractViewsTests):
         assert layer_vectortiles_test_data["layers"][3].metadatas[1].name == layer.metadatas[1].name
 
     def test_delete(self, test_app, dbsession):
-        from c2cgeoportal_commons.models.main import Layer, LayerVectorTiles, TreeItem
+        from c2cgeoportal_commons.models.main import Layer, LayerVectorTiles, Log, LogAction, TreeItem
+
+        layer = dbsession.query(LayerVectorTiles).first()
 
-        layer_id = dbsession.query(LayerVectorTiles.id).first().id
+        test_app.delete(f"/admin/layers_vectortiles/{layer.id}", status=200)
 
-        test_app.delete("/admin/layers_vectortiles/{}".format(layer_id), status=200)
+        assert dbsession.query(LayerVectorTiles).get(layer.id) is None
+        assert dbsession.query(Layer).get(layer.id) is None
+        assert dbsession.query(TreeItem).get(layer.id) is None
 
-        assert dbsession.query(LayerVectorTiles).get(layer_id) is None
-        assert dbsession.query(Layer).get(layer_id) is None
-        assert dbsession.query(TreeItem).get(layer_id) is None
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.DELETE
+        assert log.element_type == "layer_vectortiles"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"