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

tests/test_layer_groups.py

@@ -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):
@@ -94,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
@@ -134,6 +135,14 @@ 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.
@@ -159,6 +168,8 @@ 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(
             f"{self._prefix}/new",
@@ -202,6 +213,14 @@ 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.
@@ -274,34 +293,49 @@ class TestLayersGroups(TestTreeGroup):
         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(f"/admin/layer_groups/{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

@@ -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
@@ -158,8 +159,16 @@ class TestLayerVectortiles(AbstractViewsTests):
         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",
@@ -178,6 +187,14 @@ 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
 
@@ -212,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(f"/admin/layers_vectortiles/{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"

tests/test_layers_wms.py

@@ -39,7 +39,6 @@ def layer_wms_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("layer_wms_test_data", "test_app")
 class TestLayerWMSViews(AbstractViewsTests):
-
     _prefix = "/admin/layers_wms"
 
     def test_index_rendering(self, test_app):
@@ -154,6 +153,8 @@ class TestLayerWMSViews(AbstractViewsTests):
         assert form["public"].checked
 
     def test_edit(self, test_app, layer_wms_test_data, dbsession):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         layer = layer_wms_test_data["layers"][0]
 
         form = self.get_item(test_app, layer.id).form
@@ -212,8 +213,16 @@ class TestLayerWMSViews(AbstractViewsTests):
         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_wms"
+        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_wms_test_data):
-        from c2cgeoportal_commons.models.main import LayerWMS
+        from c2cgeoportal_commons.models.main import LayerWMS, Log, LogAction
 
         resp = test_app.post(
             "/admin/layers_wms/new",
@@ -237,6 +246,14 @@ class TestLayerWMSViews(AbstractViewsTests):
             r"http://localhost/admin/layers_wms/(.*)\?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_wms"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"
+
     def test_duplicate(self, layer_wms_test_data, test_app, dbsession):
         from c2cgeoportal_commons.models.main import LayerWMS
 
@@ -339,12 +356,20 @@ class TestLayerWMSViews(AbstractViewsTests):
         assert "image/jpeg" == resp.form["image_type"].value
 
     def test_convert_without_wmts_defaults(self, test_app, layer_wms_test_data, dbsession):
-        from c2cgeoportal_commons.models.main import LayerWMTS
+        from c2cgeoportal_commons.models.main import LayerWMTS, Log, LogAction
 
         dbsession.delete(LayerWMTS.get_default(dbsession))
         layer = layer_wms_test_data["layers"][3]
         test_app.post(f"/admin/layers_wms/{layer.id}/convert_to_wmts", status=200)
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.CONVERT_TO_WMTS
+        assert log.element_type == "layer_wms"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"
+
     def test_unicity_validator(self, layer_wms_test_data, test_app):
         layer = layer_wms_test_data["layers"][2]
         resp = test_app.get(f"/admin/layers_wms/{layer.id}/duplicate", status=200)
@@ -371,17 +396,25 @@ class TestLayerWMSViews(AbstractViewsTests):
         # assert str(layer.id) == re.match('http://localhost/admin/layers_wms/(.*)', resp.location).group(1)
 
     def test_delete(self, test_app, dbsession):
-        from c2cgeoportal_commons.models.main import Layer, LayerWMS, TreeItem
+        from c2cgeoportal_commons.models.main import Layer, LayerWMS, Log, LogAction, TreeItem
+
+        layer = dbsession.query(LayerWMS).first()
 
-        layer_id = dbsession.query(LayerWMS.id).first().id
+        test_app.delete(f"/admin/layers_wms/{layer.id}", status=200)
 
-        test_app.delete(f"/admin/layers_wms/{layer_id}", status=200)
+        assert dbsession.query(LayerWMS).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(LayerWMS).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_wms"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"
 
-    def test_submit_new_no_layer_name(self, test_app, layer_wms_test_data):
+    def test_submit_new_no_layer_name(self, test_app, layer_wms_test_data, dbsession):
         resp = test_app.post(
             "/admin/layers_wms/new",
             {

tests/test_layers_wmts.py

@@ -39,7 +39,6 @@ def layer_wmts_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("layer_wmts_test_data", "test_app")
 class TestLayerWMTS(AbstractViewsTests):
-
     _prefix = "/admin/layers_wmts"
 
     def test_index_rendering(self, test_app):
@@ -102,6 +101,8 @@ class TestLayerWMTS(AbstractViewsTests):
         assert default_wmts.matrix_set == self.get_first_field_named(form, "matrix_set").value
 
     def test_edit(self, test_app, layer_wmts_test_data, dbsession):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         layer = layer_wmts_test_data["layers"][0]
 
         form = self.get_item(test_app, layer.id).form
@@ -160,6 +161,14 @@ class TestLayerWMTS(AbstractViewsTests):
         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_wmts"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"
+
     def test_duplicate(self, layer_wmts_test_data, test_app, dbsession):
         from c2cgeoportal_commons.models.main import LayerWMTS
 
@@ -190,15 +199,23 @@ class TestLayerWMTS(AbstractViewsTests):
         ).group(1)
 
     def test_delete(self, test_app, dbsession):
-        from c2cgeoportal_commons.models.main import Layer, LayerWMTS, TreeItem
+        from c2cgeoportal_commons.models.main import Layer, LayerWMTS, Log, LogAction, TreeItem
 
-        layer_id = dbsession.query(LayerWMTS.id).first().id
+        layer = dbsession.query(LayerWMTS).first()
 
-        test_app.delete(f"/admin/layers_wmts/{layer_id}", status=200)
+        test_app.delete(f"/admin/layers_wmts/{layer.id}", status=200)
 
-        assert dbsession.query(LayerWMTS).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(LayerWMTS).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_wmts"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"
 
     def test_unicity_validator(self, layer_wmts_test_data, test_app):
         layer = layer_wmts_test_data["layers"][2]
@@ -248,8 +265,16 @@ class TestLayerWMTS(AbstractViewsTests):
         )
 
     def test_convert_without_wms_defaults(self, test_app, layer_wmts_test_data, dbsession):
-        from c2cgeoportal_commons.models.main import LayerWMS
+        from c2cgeoportal_commons.models.main import LayerWMS, Log, LogAction
 
         dbsession.delete(LayerWMS.get_default(dbsession))
         layer = layer_wmts_test_data["layers"][3]
         test_app.post(f"/admin/layers_wmts/{layer.id}/convert_to_wms", status=200)
+
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.CONVERT_TO_WMS
+        assert log.element_type == "layer_wmts"
+        assert log.element_id == layer.id
+        assert log.element_name == layer.name
+        assert log.username == "test_user"

tests/test_layertree.py

@@ -103,7 +103,6 @@ def layertree_test_data(dbsession, transact):
 )
 @pytest.mark.usefixtures("layertree_test_data", "test_app")
 class TestLayerTreeView(AbstractViewsTests):
-
     _prefix = "/admin/layertree"
 
     def test_index(self, test_app):

tests/test_left_menu.py

@@ -24,7 +24,6 @@ def left_menu_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("test_app")
 class TestLeftMenu(AbstractViewsTests):
-
     _prefix = "/admin/roles"
 
     def test_index(self, test_app):

tests/test_lingua_extractor_config.py

@@ -1,6 +1,4 @@
-# -*- coding: utf-8 -*-
-
-# Copyright (c) 2022, Camptocamp SA
+# Copyright (c) 2022-2023, Camptocamp SA
 # All rights reserved.
 
 # Redistribution and use in source and binary forms, with or without

tests/test_logs.py

@@ -0,0 +1,103 @@
+# pylint: disable=no-self-use,unsubscriptable-object
+
+import datetime
+
+import pytest
+import pytz
+from pyramid.testing import DummyRequest
+
+from . import AbstractViewsTests
+
+
+@pytest.fixture(scope="function")
+@pytest.mark.usefixtures("dbsession", "transact")
+def logs_test_data(dbsession, transact):
+    del transact
+
+    from c2cgeoportal_commons.models.main import AbstractLog
+    from c2cgeoportal_commons.models.main import Log as MainLog
+    from c2cgeoportal_commons.models.main import LogAction
+    from c2cgeoportal_commons.models.static import Log as StaticLog
+
+    logs = []
+    for i in range(0, 5):
+        log = MainLog(
+            date=datetime.datetime.now(pytz.utc),
+            action=[LogAction.INSERT, LogAction.UPDATE, LogAction.DELETE][i % 3],
+            element_type="role",
+            element_id=i,
+            element_name=f"role_{i}",
+            element_url_table="roles",
+            username="testuser",
+        )
+        dbsession.add(log)
+        logs.append(log)
+
+        log = StaticLog(
+            date=datetime.datetime.now(pytz.utc),
+            action=[LogAction.INSERT, LogAction.UPDATE, LogAction.DELETE][i % 3],
+            element_type="user",
+            element_id=i,
+            element_name=f"user_{i}",
+            element_url_table="users",
+            username="testuser",
+        )
+        dbsession.add(log)
+        logs.append(log)
+
+    dbsession.flush()
+
+    yield {
+        "logs": logs,
+    }
+
+
+@pytest.mark.usefixtures("logs_test_data", "test_app")
+class TestLog(AbstractViewsTests):
+    _prefix = "/admin/logs"
+
+    def test_index_rendering(self, test_app):
+        resp = self.get(test_app)
+
+        self.check_left_menu(resp, "Logs")
+
+        expected = [
+            ("actions", "", "false"),
+            ("id", "id", "true"),
+            ("date", "Date"),
+            ("username", "Username"),
+            ("action", "Action"),
+            ("element_type", "Element type"),
+            ("element_id", "Element identifier"),
+            ("element_name", "Element name"),
+        ]
+        self.check_grid_headers(resp, expected, new=False)
+
+    def test_grid_default_sort_on_date_desc(self, test_app, logs_test_data):
+        json = self.check_search(test_app)
+        expected_ids = [
+            log.id
+            for log in sorted(
+                logs_test_data["logs"],
+                key=lambda log: log.date,
+                reverse=True,
+            )
+        ]
+        result_ids = [int(row["_id_"]) for row in json["rows"]]
+        assert result_ids == expected_ids
+
+    def test_grid_sort_on_element_type(self, test_app, logs_test_data):
+        json = self.check_search(test_app, sort="element_type")
+        expected_ids = [
+            log.id
+            for log in sorted(
+                logs_test_data["logs"],
+                key=lambda log: (log.element_type, -log.date.timestamp()),
+            )
+        ]
+        result_ids = [int(row["_id_"]) for row in json["rows"]]
+        assert result_ids == expected_ids
+
+    def test_grid_search(self, test_app):
+        self.check_search(test_app, "role", total=5)
+        self.check_search(test_app, "user_2", total=1)

tests/test_metadatas.py

@@ -59,7 +59,6 @@ def metadatas_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("metadatas_test_data", "test_app")
 class TestMetadatasView(AbstractViewsTests):
-
     _prefix = "/admin/"
 
     def __metadata_ui_types(self):

tests/test_oauth2_clients.py

@@ -35,7 +35,6 @@ def oauth2_clients_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("oauth2_clients_test_data", "test_app")
 class TestOAuth2Client(TestTreeGroup):
-
     _prefix = "/admin/oauth2_clients"
 
     def test_index_rendering(self, test_app):
@@ -59,7 +58,8 @@ class TestOAuth2Client(TestTreeGroup):
         self.check_search(test_app, client.client_id, total=1)
 
     def test_submit_new(self, dbsession, test_app, oauth2_clients_test_data):
-        from c2cgeoportal_commons.models.static import OAuth2Client
+        from c2cgeoportal_commons.models.main import LogAction
+        from c2cgeoportal_commons.models.static import Log, OAuth2Client
 
         resp = test_app.post(
             "/admin/oauth2_clients/new",
@@ -84,7 +84,18 @@ class TestOAuth2Client(TestTreeGroup):
         assert oauth2_client.secret == "12345"
         assert oauth2_client.redirect_uri == "http://127.0.0.1:7070/bis"
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.INSERT
+        assert log.element_type == "oauth2_client"
+        assert log.element_id == oauth2_client.id
+        assert log.element_name == oauth2_client.client_id
+        assert log.username == "test_user"
+
     def test_edit_then_save(self, dbsession, test_app, oauth2_clients_test_data):
+        from c2cgeoportal_commons.models.main import LogAction
+        from c2cgeoportal_commons.models.static import Log
+
         oauth2_client = oauth2_clients_test_data["oauth2_clients"][10]
 
         dbsession.expire(oauth2_client)
@@ -112,6 +123,14 @@ class TestOAuth2Client(TestTreeGroup):
         assert "New secret" == oauth2_client.secret
         assert "New redirect URI" == oauth2_client.redirect_uri
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.UPDATE
+        assert log.element_type == "oauth2_client"
+        assert log.element_id == oauth2_client.id
+        assert log.element_name == oauth2_client.client_id
+        assert log.username == "test_user"
+
     def test_duplicate(self, oauth2_clients_test_data, test_app, dbsession):
         from c2cgeoportal_commons.models.static import OAuth2Client
 
@@ -134,11 +153,20 @@ class TestOAuth2Client(TestTreeGroup):
         assert oauth2_client_proto.id != oauth2_client.id
 
     def test_delete(self, test_app, dbsession):
-        from c2cgeoportal_commons.models.static import OAuth2Client
-
-        oauth2_client_id = dbsession.query(OAuth2Client.id).first().id
-        test_app.delete(f"/admin/oauth2_clients/{oauth2_client_id}", status=200)
-        assert dbsession.query(OAuth2Client).get(oauth2_client_id) is None
+        from c2cgeoportal_commons.models.main import LogAction
+        from c2cgeoportal_commons.models.static import Log, OAuth2Client
+
+        oauth2_client = dbsession.query(OAuth2Client).first()
+        test_app.delete(f"/admin/oauth2_clients/{oauth2_client.id}", status=200)
+        assert dbsession.query(OAuth2Client).get(oauth2_client.id) is None
+
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.DELETE
+        assert log.element_type == "oauth2_client"
+        assert log.element_id == oauth2_client.id
+        assert log.element_name == oauth2_client.client_id
+        assert log.username == "test_user"
 
     def test_unicity_validator(self, oauth2_clients_test_data, test_app):
         oauth2_client_proto = oauth2_clients_test_data["oauth2_clients"][7]