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

tests/test_lingua_extractor_config.py

@@ -0,0 +1,64 @@
+# Copyright (c) 2022-2023, Camptocamp SA
+# All rights reserved.
+
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are met:
+
+# 1. Redistributions of source code must retain the above copyright notice, this
+#    list of conditions and the following disclaimer.
+# 2. Redistributions in binary form must reproduce the above copyright notice,
+#    this list of conditions and the following disclaimer in the documentation
+#    and/or other materials provided with the distribution.
+
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
+# ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+# WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+# DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR
+# ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+# (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+# LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
+# ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+# SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+# The views and conclusions contained in the software and documentation are those
+# of the authors and should not be interpreted as representing official policies,
+# either expressed or implied, of the FreeBSD Project.
+
+# pylint: disable=missing-docstring
+
+from unittest.mock import Mock, mock_open, patch
+
+import pytest
+import yaml
+from c2c.template.config import config as configuration
+
+from c2cgeoportal_admin.lib.lingua_extractor import GeomapfishConfigExtractor
+
+GMF_CONFIG = """
+vars:
+  admin_interface:
+    available_metadata:
+      - name: metadata1
+        description: description1
+        translate: true
+    available_functionalities:
+      - name: functionality1
+        description: description2
+"""
+
+
+class TestGeomapfishConfigExtractor:
+    @patch(
+        "c2cgeoportal_admin.lib.lingua_extractor.open",
+        mock_open(read_data=GMF_CONFIG),
+    )
+    def test_extract_config(self):
+        extractor = GeomapfishConfigExtractor()
+
+        options = Mock()
+        options.keywords = []
+
+        messages = list(extractor("config.yaml", options))
+
+        assert {msg.msgid for msg in messages} == {"description1", "description2"}

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_main.py

@@ -5,7 +5,9 @@ from webtest import TestApp as WebTestApp
 
 @pytest.mark.usefixtures("app_env")
 def test_main(app_env):
-    """Test dev environment"""
+    """
+    Test dev environment.
+    """
     config = testing.setUp(registry=app_env["registry"])
     app = config.make_wsgi_app()
     testapp = WebTestApp(app)

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):
@@ -85,7 +84,9 @@ class TestMetadatasView(AbstractViewsTests):
             return None
         return metadata.value
 
-    def _check_metadatas(self, test_app, item, metadatas):
+    def _check_metadatas(self, test_app, item, metadatas, model):
+        from c2cgeoportal_admin.schemas.metadata import metadata_definitions
+
         settings = test_app.app.registry.settings
         self._check_sequence(
             item,
@@ -96,9 +97,7 @@ class TestMetadatasView(AbstractViewsTests):
                         "name": "name",
                         "value": [
                             {"text": s_m["name"], "value": s_m["name"], "selected": s_m["name"] == m.name}
-                            for s_m in sorted(
-                                settings["admin_interface"]["available_metadata"], key=lambda m: m["name"]
-                            )
+                            for s_m in sorted(metadata_definitions(settings, model), key=lambda m: m["name"])
                         ],
                         "label": "Name",
                     },
@@ -131,7 +130,7 @@ class TestMetadatasView(AbstractViewsTests):
         resp = self._post_metadata(test_app, url, base_mapping, name, value, 200)
         assert (
             error_msg
-            == resp.html.select_one(".item-{} .help-block".format(self.__metadata_ui_type(test_app, name)))
+            == resp.html.select_one(f".item-{self.__metadata_ui_type(test_app, name)} .help-block")
             .getText()
             .strip()
         )
@@ -156,12 +155,16 @@ class TestMetadatasView(AbstractViewsTests):
         )
 
     def test_get_true_boolean_metadata(self, metadatas_test_data, test_app):
-        metadatas_test_data["layer_wms"].get_metadatas("_boolean")[0].value = "true"
-        self._test_edit_treeitem("layers_wms", metadatas_test_data["layer_wms"], test_app)
+        from c2cgeoportal_commons.models.main import LayerWMS
+
+        metadatas_test_data["layer_wms"].get_metadata("_boolean")[0].value = "true"
+        self._test_edit_treeitem("layers_wms", metadatas_test_data["layer_wms"], test_app, LayerWMS)
 
     def test_get_false_boolean_metadata(self, metadatas_test_data, test_app):
-        metadatas_test_data["layer_wms"].get_metadatas("_boolean")[0].value = "false"
-        self._test_edit_treeitem("layers_wms", metadatas_test_data["layer_wms"], test_app)
+        from c2cgeoportal_commons.models.main import LayerWMS
+
+        metadatas_test_data["layer_wms"].get_metadata("_boolean")[0].value = "false"
+        self._test_edit_treeitem("layers_wms", metadatas_test_data["layer_wms"], test_app, LayerWMS)
 
     def test_post_true_boolean_metadata(self, test_app, metadatas_test_data, dbsession):
         from c2cgeoportal_commons.models.main import LayerWMS
@@ -175,7 +178,7 @@ class TestMetadatasView(AbstractViewsTests):
             302,
         )
         layer = dbsession.query(LayerWMS).filter(LayerWMS.name == "new_name").one()
-        assert layer.get_metadatas("_boolean")[0].value == "true"
+        assert layer.get_metadata("_boolean")[0].value == "true"
 
     def test_post_false_boolean_metadata(self, test_app, metadatas_test_data, dbsession):
         from c2cgeoportal_commons.models.main import LayerWMS
@@ -189,7 +192,7 @@ class TestMetadatasView(AbstractViewsTests):
             302,
         )
         layer = dbsession.query(LayerWMS).filter(LayerWMS.name == "new_name").one()
-        assert layer.get_metadatas("_boolean")[0].value == "false"
+        assert layer.get_metadata("_boolean")[0].value == "false"
 
     def test_valid_float_metadata(self, test_app, metadatas_test_data):
         self._post_metadata(
@@ -304,31 +307,41 @@ class TestMetadatasView(AbstractViewsTests):
             302,
         )
 
-    def _test_edit_treeitem(self, prefix, item, test_app):
-        resp = self.get(test_app, "{}/{}".format(prefix, item.id))
-        self._check_metadatas(test_app, resp.html.select_one(".item-metadatas"), item.metadatas)
+    def _test_edit_treeitem(self, prefix, item, test_app, model):
+        resp = self.get(test_app, f"{prefix}/{item.id}")
+        self._check_metadatas(test_app, resp.html.select_one(".item-metadatas"), item.metadatas, model)
         resp.form.submit("submit", status=302)
 
     def test_layer_wms_metadatas(self, metadatas_test_data, test_app):
-        self._test_edit_treeitem("layers_wms", metadatas_test_data["layer_wms"], test_app)
+        from c2cgeoportal_commons.models.main import LayerWMS
+
+        self._test_edit_treeitem("layers_wms", metadatas_test_data["layer_wms"], test_app, LayerWMS)
 
     def test_layer_wmts_metadatas(self, metadatas_test_data, test_app):
-        self._test_edit_treeitem("layers_wmts", metadatas_test_data["layer_wmts"], test_app)
+        from c2cgeoportal_commons.models.main import LayerWMTS
+
+        self._test_edit_treeitem("layers_wmts", metadatas_test_data["layer_wmts"], test_app, LayerWMTS)
 
     def test_theme_metadatas(self, metadatas_test_data, test_app):
-        self._test_edit_treeitem("themes", metadatas_test_data["theme"], test_app)
+        from c2cgeoportal_commons.models.main import Theme
+
+        self._test_edit_treeitem("themes", metadatas_test_data["theme"], test_app, Theme)
 
     def test_group_metadatas(self, metadatas_test_data, test_app):
-        self._test_edit_treeitem("layer_groups", metadatas_test_data["group"], test_app)
+        from c2cgeoportal_commons.models.main import LayerGroup
+
+        self._test_edit_treeitem("layer_groups", metadatas_test_data["group"], test_app, LayerGroup)
 
     def test_undefined_metadata(self, metadatas_test_data, test_app):
-        """Undefined metadata must be kept intact across submissions"""
+        """
+        Undefined metadata must be kept intact across submissions.
+        """
         from c2cgeoportal_commons.models.main import Metadata
 
         layer = metadatas_test_data["layer_wms"]
         layer.metadatas = [Metadata("_undefined", "This is an undefined metadata")]
 
-        resp = self.get(test_app, "layers_wms/{}".format(layer.id))
+        resp = self.get(test_app, f"layers_wms/{layer.id}")
         resp.form.submit("submit", status=302)
 
         metadata = layer.metadatas[0]

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,12 +123,20 @@ 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
 
         oauth2_client_proto = oauth2_clients_test_data["oauth2_clients"][7]
 
-        resp = test_app.get("/admin/oauth2_clients/{}/duplicate".format(oauth2_client_proto.id), status=200)
+        resp = test_app.get(f"/admin/oauth2_clients/{oauth2_client_proto.id}/duplicate", status=200)
         form = resp.form
 
         assert "" == self.get_first_field_named(form, "id").value
@@ -134,19 +153,28 @@ 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
+        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
 
-        oauth2_client_id = dbsession.query(OAuth2Client.id).first().id
-        test_app.delete("/admin/oauth2_clients/{}".format(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]
-        resp = test_app.get("/admin/oauth2_clients/{}/duplicate".format(oauth2_client_proto.id), status=200)
+        resp = test_app.get(f"/admin/oauth2_clients/{oauth2_client_proto.id}/duplicate", status=200)
 
         resp = resp.form.submit("submit")
 
-        self._check_submission_problem(resp, "{} is already used.".format(oauth2_client_proto.client_id))
+        self._check_submission_problem(resp, f"{oauth2_client_proto.client_id} is already used.")
 
     @pytest.mark.usefixtures("raise_db_error_on_query")
     def test_grid_dberror(self, dbsession):

tests/test_ogc_servers.py

@@ -1,6 +1,7 @@
 # pylint: disable=no-self-use
 
 import re
+from unittest.mock import patch
 
 import pytest
 
@@ -17,8 +18,8 @@ def ogc_server_test_data(dbsession, transact):
     auth = ["No auth", "Standard auth", "Geoserver auth", "Proxy"]
     servers = []
     for i in range(0, 8):
-        server = OGCServer(name="server_{}".format(i), description="description_{}".format(i))
-        server.url = "https://somebasicurl_{}.com".format(i)
+        server = OGCServer(name=f"server_{i}", description=f"description_{i}")
+        server.url = f"https://somebasicurl_{i}.com"
         server.image_type = "image/jpeg" if i % 2 == 0 else "image/png"
         server.auth = auth[i % 4]
         dbsession.add(server)
@@ -31,7 +32,6 @@ def ogc_server_test_data(dbsession, transact):
 
 @pytest.mark.usefixtures("ogc_server_test_data", "test_app")
 class TestOGCServer(AbstractViewsTests):
-
     _prefix = "/admin/ogc_servers"
 
     def test_index_rendering(self, test_app):
@@ -59,54 +59,82 @@ class TestOGCServer(AbstractViewsTests):
         self.check_search(test_app, "server_0", total=1)
 
     def test_submit_new(self, dbsession, test_app):
-        from c2cgeoportal_commons.models.main import OGCServer
-
-        resp = test_app.post(
-            "/admin/ogc_servers/new",
-            {
-                "name": "new_name",
-                "description": "new description",
-                "url": "www.randomurl.com",
-                "type": "mapserver",
-                "auth": "No auth",
-                "image_type": "image/png",
-            },
-            status=302,
-        )
+        from c2cgeoportal_commons.models.main import Log, LogAction, OGCServer
+
+        with patch("c2cgeoportal_admin.views.ogc_servers.OGCServerViews._update_cache"):
+            resp = test_app.post(
+                "/admin/ogc_servers/new",
+                {
+                    "name": "new_name",
+                    "description": "new description",
+                    "url": "www.randomurl.com",
+                    "type": "mapserver",
+                    "auth": "No auth",
+                    "image_type": "image/png",
+                },
+                status=302,
+            )
         ogc_server = dbsession.query(OGCServer).filter(OGCServer.name == "new_name").one()
         assert str(ogc_server.id) == re.match(
             r"http://localhost/admin/ogc_servers/(.*)\?msg_col=submit_ok", resp.location
         ).group(1)
         assert ogc_server.name == "new_name"
 
-    def test_edit(self, test_app, ogc_server_test_data):
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.INSERT
+        assert log.element_type == "ogc_server"
+        assert log.element_id == ogc_server.id
+        assert log.element_name == ogc_server.name
+        assert log.username == "test_user"
+
+    def test_edit(self, test_app, ogc_server_test_data, dbsession):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         ogc_server = ogc_server_test_data["ogc_servers"][0]
-        resp = test_app.get("/admin/ogc_servers/{}".format(ogc_server.id), status=200)
+        resp = test_app.get(f"/admin/ogc_servers/{ogc_server.id}", status=200)
         form = resp.form
         assert str(ogc_server.id) == self.get_first_field_named(form, "id").value
         assert "hidden" == self.get_first_field_named(form, "id").attrs["type"]
         assert ogc_server.name == form["name"].value
         form["description"] = "new_description"
-        assert form.submit().status_int == 302
+        with patch("c2cgeoportal_admin.views.ogc_servers.OGCServerViews._update_cache"):
+            assert form.submit().status_int == 302
         assert ogc_server.description == "new_description"
 
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.UPDATE
+        assert log.element_type == "ogc_server"
+        assert log.element_id == ogc_server.id
+        assert log.element_name == ogc_server.name
+        assert log.username == "test_user"
+
     def test_delete(self, test_app, ogc_server_test_data, dbsession):
-        from c2cgeoportal_commons.models.main import OGCServer
+        from c2cgeoportal_commons.models.main import Log, LogAction, OGCServer
 
         ogc_server = ogc_server_test_data["ogc_servers"][0]
-        deleted_id = ogc_server.id
-        test_app.delete("/admin/ogc_servers/{}".format(deleted_id), status=200)
-        assert dbsession.query(OGCServer).get(deleted_id) is None
+        test_app.delete(f"/admin/ogc_servers/{ogc_server.id}", status=200)
+        assert dbsession.query(OGCServer).get(ogc_server.id) is None
+
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.DELETE
+        assert log.element_type == "ogc_server"
+        assert log.element_id == ogc_server.id
+        assert log.element_name == ogc_server.name
+        assert log.username == "test_user"
 
     def test_duplicate(self, ogc_server_test_data, test_app, dbsession):
         from c2cgeoportal_commons.models.main import OGCServer
 
         ogc_server = ogc_server_test_data["ogc_servers"][3]
-        resp = test_app.get("/admin/ogc_servers/{}/duplicate".format(ogc_server.id), status=200)
+        resp = test_app.get(f"/admin/ogc_servers/{ogc_server.id}/duplicate", status=200)
         form = resp.form
         assert "" == self.get_first_field_named(form, "id").value
         self.set_first_field_named(form, "name", "clone")
-        resp = form.submit("submit")
+        with patch("c2cgeoportal_admin.views.ogc_servers.OGCServerViews._update_cache"):
+            resp = form.submit("submit")
         assert resp.status_int == 302
         server = dbsession.query(OGCServer).filter(OGCServer.name == "clone").one()
         assert str(server.id) == re.match(
@@ -115,18 +143,18 @@ class TestOGCServer(AbstractViewsTests):
 
     def test_unicity_validator(self, ogc_server_test_data, test_app):
         ogc_server = ogc_server_test_data["ogc_servers"][3]
-        resp = test_app.get("/admin/ogc_servers/{}/duplicate".format(ogc_server.id), status=200)
+        resp = test_app.get(f"/admin/ogc_servers/{ogc_server.id}/duplicate", status=200)
 
         resp = resp.form.submit("submit")
 
-        self._check_submission_problem(resp, "{} is already used.".format(ogc_server.name))
+        self._check_submission_problem(resp, f"{ogc_server.name} is already used.")
 
     def test_check_success(self, ogc_server_test_data, test_app):
         ogc_server = ogc_server_test_data["ogc_servers"][3]
         ogc_server.url = "config://mapserver"
-        resp = test_app.get("/admin/ogc_servers/{}/synchronize".format(ogc_server.id), status=200)
+        resp = test_app.get(f"/admin/ogc_servers/{ogc_server.id}/synchronize", status=200)
 
-        resp = resp.forms["form-check"].submit("submit")
+        resp = resp.forms["form-check"].submit("check")
 
         assert list(resp.html.find("div", class_="alert-success").stripped_strings) == [
             "OGC Server has been successfully synchronized."
@@ -135,20 +163,41 @@ class TestOGCServer(AbstractViewsTests):
     def test_dry_run_success(self, ogc_server_test_data, test_app):
         ogc_server = ogc_server_test_data["ogc_servers"][3]
         ogc_server.url = "config://mapserver"
-        resp = test_app.get("/admin/ogc_servers/{}/synchronize".format(ogc_server.id), status=200)
+        resp = test_app.get(f"/admin/ogc_servers/{ogc_server.id}/synchronize", status=200)
 
-        resp = resp.forms["form-dry-run"].submit("submit")
+        resp = resp.forms["form-dry-run"].submit("dry-run")
 
         assert list(resp.html.find("div", class_="alert-success").stripped_strings) == [
             "OGC Server has been successfully synchronized."
         ]
 
-    def test_synchronize_success(self, ogc_server_test_data, test_app):
+    def test_synchronize_success(self, ogc_server_test_data, test_app, dbsession):
+        from c2cgeoportal_commons.models.main import Log, LogAction
+
         ogc_server = ogc_server_test_data["ogc_servers"][3]
         ogc_server.url = "config://mapserver"
-        resp = test_app.get("/admin/ogc_servers/{}/synchronize".format(ogc_server.id), status=200)
+        resp = test_app.get(f"/admin/ogc_servers/{ogc_server.id}/synchronize", status=200)
+
+        resp = resp.forms["form-synchronize"].submit("synchronize")
+
+        log = dbsession.query(Log).one()
+        assert log.date != None
+        assert log.action == LogAction.SYNCHRONIZE
+        assert log.element_type == "ogc_server"
+        assert log.element_id == ogc_server.id
+        assert log.element_name == ogc_server.name
+        assert log.username == "test_user"
+
+        assert list(resp.html.find("div", class_="alert-success").stripped_strings) == [
+            "OGC Server has been successfully synchronized."
+        ]
+
+        form = resp.forms["form-synchronize"]
+        form["force-parents"].checked = True
+        form["force-ordering"].checked = True
+        form["clean"].checked = True
 
-        resp = resp.forms["form-synchronize"].submit("submit")
+        resp = form.submit("synchronize")
 
         assert list(resp.html.find("div", class_="alert-success").stripped_strings) == [
             "OGC Server has been successfully synchronized."