umap-project 3.3.6__py3-none-any.whl → 3.4.0__py3-none-any.whl

umap/tests/integration/test_anonymous_owned_map.py

@@ -18,25 +18,24 @@ def owner_session(anonymap, context, live_server):
     key, value = anonymap.signed_cookie_elements
     signed = get_cookie_signer(salt=key).sign(value)
     context.add_cookies([{"name": key, "value": signed, "url": live_server.url}])
-    return context.new_page()
 
 
-def test_map_load_with_owner(anonymap, live_server, owner_session):
-    owner_session.goto(f"{live_server.url}{anonymap.get_absolute_url()}")
-    map_el = owner_session.locator("#map")
+def test_map_load_with_owner(anonymap, live_server, owner_session, page):
+    page.goto(f"{live_server.url}{anonymap.get_absolute_url()}")
+    map_el = page.locator("#map")
     expect(map_el).to_be_visible()
-    enable = owner_session.get_by_role("button", name="Edit")
+    enable = page.get_by_role("button", name="Edit")
     expect(enable).to_be_visible()
     enable.click()
-    disable = owner_session.get_by_role("button", name="View")
+    disable = page.get_by_role("button", name="View")
     expect(disable).to_be_visible()
-    save = owner_session.get_by_role("button", name="Save")
+    save = page.get_by_role("button", name="Save")
     expect(save).to_be_visible()
-    add_marker = owner_session.get_by_title("Draw a marker")
+    add_marker = page.get_by_title("Draw a marker")
     expect(add_marker).to_be_visible()
-    edit_settings = owner_session.get_by_title("Map advanced properties")
+    edit_settings = page.get_by_title("Map advanced properties")
     expect(edit_settings).to_be_visible()
-    edit_permissions = owner_session.get_by_title("Update permissions and editors")
+    edit_permissions = page.get_by_title("Update permissions and editors")
     expect(edit_permissions).to_be_visible()
 
 
@@ -71,35 +70,31 @@ def test_map_load_with_anonymous_but_editable_layer(
     expect(edit_permissions).to_be_hidden()
 
 
-def test_owner_permissions_form(map, datalayer, live_server, owner_session):
-    owner_session.goto(f"{live_server.url}{map.get_absolute_url()}?edit")
-    edit_permissions = owner_session.get_by_title("Update permissions and editors")
+def test_owner_permissions_form(map, datalayer, live_server, owner_session, page):
+    page.goto(f"{live_server.url}{map.get_absolute_url()}?edit")
+    edit_permissions = page.get_by_title("Update permissions and editors")
     expect(edit_permissions).to_be_visible()
     edit_permissions.click()
-    owner_field = owner_session.locator(".umap-field-owner")
+    owner_field = page.locator(".umap-field-owner")
     expect(owner_field).to_be_hidden()
-    editors_field = owner_session.locator(".umap-field-editors input")
+    editors_field = page.locator(".umap-field-editors input")
     expect(editors_field).to_be_hidden()
-    datalayer_label = owner_session.get_by_text('Who can edit "test datalayer"')
+    datalayer_label = page.get_by_text('Who can edit "test datalayer"')
     expect(datalayer_label).to_be_visible()
-    options = owner_session.locator(
-        ".datalayer-permissions select[name='edit_status'] option"
-    )
+    options = page.locator(".datalayer-permissions select[name='edit_status'] option")
     expect(options).to_have_count(3)
-    option = owner_session.locator(
+    option = page.locator(
         ".datalayer-permissions select[name='edit_status'] option:checked"
     )
     expect(option).to_have_text("Inherit")
-    expect(owner_session.locator(".umap-field-share_status select")).to_be_visible()
+    expect(page.locator(".umap-field-share_status select")).to_be_visible()
     options = [
         int(option.get_attribute("value"))
-        for option in owner_session.locator(
-            ".umap-field-share_status select option"
-        ).all()
+        for option in page.locator(".umap-field-share_status select option").all()
     ]
     assert options == [Map.DRAFT, Map.PUBLIC]
     # This field should not be present in anonymous maps
-    expect(owner_session.locator(".umap-field-owner")).to_be_hidden()
+    expect(page.locator(".umap-field-owner")).to_be_hidden()
 
 
 def test_anonymous_can_add_marker_on_editable_layer(
@@ -180,7 +175,7 @@ def test_alert_message_after_create(
     new_map = Map.objects.last()
     expect(alert).to_be_visible()
     expect(
-        alert.get_by_text("Your map has been created with an anonymous account!")
+        alert.get_by_text("Hey, you created a map without an account!")
     ).to_be_visible()
     expect(alert.get_by_role("button", name="Copy")).to_be_visible()
     expect(alert.get_by_role("button", name="Send me the link")).to_be_visible()
@@ -198,7 +193,7 @@ def test_alert_message_after_create(
         )
 
 
-def test_email_sending_error_are_catched(tilelayer, page, live_server):
+def test_email_sending_error_are_caught(tilelayer, page, live_server):
     page.goto(f"{live_server.url}/en/map/new")
     alert_creation = page.locator('umap-alert-creation div[role="dialog"]')
     with page.expect_response(re.compile(r".*/map/create/")):
@@ -224,22 +219,22 @@ def test_alert_message_after_create_show_link_even_without_mail(
     alert = page.locator('umap-alert-creation div[role="dialog"]')
     expect(alert).to_be_visible()
     expect(
-        alert.get_by_text("Your map has been created with an anonymous account!")
+        alert.get_by_text("Hey, you created a map without an account!")
     ).to_be_visible()
     expect(alert.get_by_role("button", name="Copy")).to_be_visible()
     expect(alert.get_by_role("button", name="Send me the link")).to_be_hidden()
 
 
-def test_anonymous_owner_can_delete_the_map(anonymap, live_server, owner_session):
+def test_anonymous_owner_can_delete_the_map(anonymap, live_server, owner_session, page):
     assert Map.objects.count() == 1
-    owner_session.goto(f"{live_server.url}{anonymap.get_absolute_url()}")
-    owner_session.get_by_role("button", name="Edit").click()
-    owner_session.get_by_role("button", name="Map advanced properties").click()
-    owner_session.get_by_text("Advanced actions").click()
-    expect(owner_session.get_by_role("button", name="Delete")).to_be_visible()
-    owner_session.get_by_role("button", name="Delete").click()
-    with owner_session.expect_response(re.compile(r".*/update/delete/.*")):
-        owner_session.get_by_role("button", name="OK").click()
+    page.goto(f"{live_server.url}{anonymap.get_absolute_url()}")
+    page.get_by_role("button", name="Edit").click()
+    page.get_by_role("button", name="Map advanced properties").click()
+    page.get_by_text("Advanced actions").click()
+    expect(page.get_by_role("button", name="Delete")).to_be_visible()
+    page.get_by_role("button", name="Delete").click()
+    with page.expect_response(re.compile(r".*/update/delete/.*")):
+        page.get_by_role("button", name="OK").click()
     assert Map.objects.get(pk=anonymap.pk).share_status == Map.DELETED
 
 
@@ -251,3 +246,61 @@ def test_non_owner_cannot_see_delete_button(anonymap, live_server, page):
     page.get_by_role("button", name="Map advanced properties").click()
     page.get_by_text("Advanced actions").click()
     expect(page.get_by_role("button", name="Delete")).to_be_hidden()
+
+
+def test_logged_in_user_should_have_a_message_to_attach_map(
+    anonymap, live_server, login, user, page, owner_session
+):
+    page.goto(f"{live_server.url}{anonymap.get_absolute_url()}")
+    page.get_by_role("button", name="Edit").click()
+    expect(
+        page.get_by_text(
+            "This map is anonymous, do you want to attach it to your account?"
+        )
+    ).to_be_hidden()
+
+    page = login(user)
+    page.goto(f"{live_server.url}{anonymap.get_absolute_url()}")
+    page.get_by_role("button", name="Edit").click()
+    expect(
+        page.get_by_text(
+            "This map is anonymous, do you want to attach it to your account?"
+        )
+    ).to_be_visible()
+    with page.expect_response(re.compile(r".*/update/owner/.*")):
+        page.get_by_role("button", name="OK").click()
+    saved = Map.objects.get(pk=anonymap.pk)
+    assert saved.owner
+
+
+def test_can_attach_map_after_save(
+    live_server, user, page, tilelayer, context, settings
+):
+    settings.ENABLE_ACCOUNT_LOGIN = True
+    page.goto(f"{live_server.url}/en/map/new")
+    expect(page.get_by_role("button", name="Anonymous")).to_be_visible()
+    expect(page.get_by_role("button", name="Visibility: Draft")).to_have_class(
+        re.compile(r".*anonymous.*")
+    )
+    assert not Map.objects.count()
+    with page.expect_response(re.compile(r".*/map/create/")):
+        page.get_by_role("button", name="Save draft", exact=True).click()
+    assert Map.objects.count() == 1
+    saved = Map.objects.last()
+    assert not saved.owner
+    with context.expect_page() as login_page_info:
+        page.get_by_role("link", name="log in").click()
+    login_page = login_page_info.value
+    expect(login_page).to_have_title("Login - Online map creator")
+    login_page.get_by_placeholder("Username").fill(user.username)
+    login_page.get_by_placeholder("Password").fill("123123")
+    with page.expect_response(re.compile(r".*/update/owner/")):
+        login_page.locator('#login_form input[type="submit"]').click()
+    expect(page.get_by_text("Map has been attached to your account")).to_be_visible()
+    saved = Map.objects.last()
+    assert saved.owner == user
+    expect(page.get_by_role("button", name="Anonymous")).to_be_hidden()
+    expect(page.get_by_role("button", name=user.username)).to_be_visible()
+    expect(page.get_by_role("button", name="Visibility: Draft")).not_to_have_class(
+        re.compile(r".*anonymous.*")
+    )

umap/tests/integration/test_basics.py

@@ -79,7 +79,9 @@ def test_cannot_put_script_tag_in_datalayer_name_or_description(
     expect(page.get_by_text("before after")).to_be_visible()
 
 
-def test_login_from_map_page(live_server, page, tilelayer, settings, user, context):
+def test_login_from_map_page_after_save(
+    live_server, page, tilelayer, settings, user, context
+):
     settings.ENABLE_ACCOUNT_LOGIN = True
     assert Map.objects.count() == 0
     page.goto(f"{live_server.url}/en/map/new/")
@@ -102,3 +104,25 @@ def test_login_from_map_page(live_server, page, tilelayer, settings, user, conte
     assert Map.objects.count() == 1
     # Use name should now appear on the header toolbar
     expect(page.get_by_role("button", name="Joe")).to_be_visible()
+
+
+def test_login_from_unsaved_map_page(
+    live_server, page, tilelayer, settings, user, context
+):
+    settings.ENABLE_ACCOUNT_LOGIN = True
+    assert Map.objects.count() == 0
+    page.goto(f"{live_server.url}/en/map/new/")
+    page.locator('[data-ref="user"]').click()
+    with context.expect_page() as login_page_info:
+        page.get_by_role("button", name="Login").click()
+    login_page = login_page_info.value
+    expect(login_page).to_have_title("Login - Online map creator")
+    login_page.get_by_placeholder("Username").fill(user.username)
+    login_page.get_by_placeholder("Password").fill("123123")
+    login_page.locator('#login_form input[type="submit"]').click()
+    # Login page should be closed
+    page.wait_for_timeout(500)  # Seems needed from time to time…
+    assert len(context.pages) == 1
+    # Save should have proceed
+    # Use name should now appear on the header toolbar
+    expect(page.get_by_role("button", name="Joe")).to_be_visible()

umap/tests/integration/test_browser.py

@@ -353,6 +353,43 @@ def test_should_redraw_list_on_feature_delete(live_server, openmap, page, bootst
     expect(buttons).to_have_count(3)
 
 
+def test_should_change_feature_title_and_color_on_edit(
+    live_server, openmap, page, bootstrap
+):
+    page.goto(f"{live_server.url}{openmap.get_absolute_url()}?edit#2/19/-2")
+    expect(page.locator(".umap-browser .feature.marker")).to_contain_text(
+        "one point in france"
+    )
+    page.locator(".leaflet-marker-icon").click(modifiers=["Shift"])
+    page.locator('input[name="name"]').fill("changed name")
+    expect(page.locator(".umap-browser .feature.marker")).to_contain_text(
+        "changed name"
+    )
+
+    # Change color
+    expect(
+        page.get_by_role("listitem").filter(has_text="changed name").locator("i")
+    ).to_have_css("background-color", "rgb(0, 0, 139)")
+    page.get_by_role("heading", name="Shape properties").click()
+    page.locator(".umap-field-color").get_by_role("button", name="define").first.click()
+    page.get_by_title("Crimson").click()
+    expect(
+        page.get_by_role("listitem").filter(has_text="changed name").locator("i")
+    ).to_have_css("background-color", "rgb(220, 20, 60)")
+
+    # Undo color
+    page.locator(".edit-undo").click()
+    expect(
+        page.get_by_role("listitem").filter(has_text="changed name").locator("i")
+    ).to_have_css("background-color", "rgb(0, 0, 139)")
+
+    # Undo title change
+    page.locator(".edit-undo").click()
+    expect(page.locator(".umap-browser .feature.marker")).to_contain_text(
+        "one point in france"
+    )
+
+
 def test_should_show_header_for_display_on_load_false(
     live_server, page, bootstrap, map, datalayer
 ):

umap/tests/integration/test_conditional_rules.py

@@ -8,13 +8,6 @@ from ..base import DataLayerFactory
 pytestmark = pytest.mark.django_db
 
 
-def getColors(elements):
-    return [
-        el.evaluate("e => window.getComputedStyle(e).backgroundColor")
-        for el in elements.all()
-    ]
-
-
 DATALAYER_DATA1 = {
     "type": "FeatureCollection",
     "features": [
@@ -96,6 +89,9 @@ DATALAYER_DATA2 = {
 
 
 def test_simple_equal_rule_at_load(live_server, page, map):
+    map.settings["properties"]["fields"] = [
+        {"key": "mytype", "type": "String"},
+    ]
     map.settings["properties"]["rules"] = [
         {"condition": "mytype=odd", "properties": {"color": "aliceblue"}}
     ]
@@ -103,13 +99,18 @@ def test_simple_equal_rule_at_load(live_server, page, map):
     DataLayerFactory(map=map, data=DATALAYER_DATA1)
     DataLayerFactory(map=map, data=DATALAYER_DATA2)
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
-    markers = page.locator(".leaflet-marker-icon .icon-container")
+    markers = page.locator(".leaflet-marker-icon")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
 
 
-def test_simple_not_equal_rule_at_load(live_server, page, map):
+def test_simple_not_equal_rule_at_load(live_server, page, map, wait_for_loaded):
+    map.settings["properties"]["fields"] = [
+        {"key": "mytype", "type": "String"},
+    ]
     map.settings["properties"]["rules"] = [
         {"condition": "mytype!=even", "properties": {"color": "aliceblue"}}
     ]
@@ -117,13 +118,19 @@ def test_simple_not_equal_rule_at_load(live_server, page, map):
     DataLayerFactory(map=map, data=DATALAYER_DATA1)
     DataLayerFactory(map=map, data=DATALAYER_DATA2)
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
-    markers = page.locator(".leaflet-marker-icon .icon-container")
+    wait_for_loaded(page)
+    markers = page.locator(".leaflet-marker-icon")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
 
 
 def test_gt_rule_with_number_at_load(live_server, page, map):
+    map.settings["properties"]["fields"] = [
+        {"key": "mynumber", "type": "Number"},
+    ]
     map.settings["properties"]["rules"] = [
         {"condition": "mynumber>10", "properties": {"color": "aliceblue"}}
     ]
@@ -133,11 +140,16 @@ def test_gt_rule_with_number_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 2
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(2)
 
 
 def test_lt_rule_with_number_at_load(live_server, page, map):
+    map.settings["properties"]["fields"] = [
+        {"key": "mynumber", "type": "Number"},
+    ]
     map.settings["properties"]["rules"] = [
         {"condition": "mynumber<14", "properties": {"color": "aliceblue"}}
     ]
@@ -147,8 +159,10 @@ def test_lt_rule_with_number_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 4
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(4)
 
 
 def test_lt_rule_with_float_at_load(live_server, page, map):
@@ -161,8 +175,10 @@ def test_lt_rule_with_float_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 4
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(4)
 
 
 def test_equal_rule_with_boolean_at_load(live_server, page, map):
@@ -175,8 +191,10 @@ def test_equal_rule_with_boolean_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 2
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(2)
 
 
 def test_equal_rule_with_boolean_not_true_at_load(live_server, page, map):
@@ -189,8 +207,10 @@ def test_equal_rule_with_boolean_not_true_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
 
 
 def test_equal_rule_with_boolean_false_at_load(live_server, page, map):
@@ -203,8 +223,10 @@ def test_equal_rule_with_boolean_false_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 1
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(1)
 
 
 def test_equal_rule_with_boolean_not_false_at_load(live_server, page, map):
@@ -217,8 +239,10 @@ def test_equal_rule_with_boolean_not_false_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 4
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(4)
 
 
 def test_empty_rule_at_load(live_server, page, map):
@@ -231,8 +255,10 @@ def test_empty_rule_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
 
 
 def test_not_empty_rule_at_load(live_server, page, map):
@@ -245,8 +271,10 @@ def test_not_empty_rule_at_load(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 2
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(2)
 
 
 def test_can_create_new_rule(live_server, page, openmap):
@@ -263,14 +291,18 @@ def test_can_create_new_rule(live_server, page, openmap):
     page.locator("input[name=condition]").fill("mytype=odd")
     page.locator(".umap-field-color .define").first.click()
     page.get_by_title("AliceBlue").first.click()
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
     page.locator(".edit-undo").click()
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 0
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(0)
 
 
-def test_can_deactive_rule_from_list(live_server, page, openmap):
+def test_can_deactivate_rule_from_list(live_server, page, openmap):
     openmap.settings["properties"]["rules"] = [
         {"condition": "mytype=odd", "properties": {"color": "aliceblue"}}
     ]
@@ -280,20 +312,36 @@ def test_can_deactive_rule_from_list(live_server, page, openmap):
     page.goto(f"{live_server.url}{openmap.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
     page.get_by_role("button", name="Edit").click()
     page.get_by_role("button", name="Map advanced properties").click()
     page.get_by_text("Conditional style rules").click()
     page.get_by_role("button", name="Toggle rule").click()
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 0
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(0)
     page.get_by_role("button", name="Toggle rule").click()
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
 
 
 def test_autocomplete_datalist(live_server, page, openmap):
+    openmap.settings["properties"]["fields"] = [
+        {"key": "myboolean", "type": "String"},
+        {"key": "mytype", "type": "String"},
+        {"key": "mynumber", "type": "String"},
+        {"key": "mydate", "type": "String"},
+        {"key": "name", "type": "String"},
+        {"key": "maybeempty", "type": "String"},
+        {"key": "onlyinone", "type": "String"},
+    ]
+    openmap.save()
     DataLayerFactory(map=openmap, data=DATALAYER_DATA1)
     page.goto(f"{live_server.url}{openmap.get_absolute_url()}?edit#6/48.948/1.670")
     page.get_by_role("button", name="Map advanced properties").click()
@@ -335,10 +383,12 @@ def test_can_combine_rules(live_server, page, map):
     drops = page.locator(".umap-drop-icon .icon-container")
     expect(markers).to_have_count(5)
     expect(drops).to_have_count(2)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
-    colors = getColors(drops)
-    assert colors.count("rgb(240, 248, 255)") == 2
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
+    drops = page.locator('.umap-drop-icon [style*="background-color: aliceblue"]')
+    expect(drops).to_have_count(2)
 
 
 def test_first_matching_rule_wins_on_given_property(live_server, page, map):
@@ -352,8 +402,10 @@ def test_first_matching_rule_wins_on_given_property(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
-    assert colors.count("rgb(240, 248, 255)") == 3
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(3)
 
 
 def test_rules_from_datalayer(live_server, page, map):
@@ -370,11 +422,14 @@ def test_rules_from_datalayer(live_server, page, map):
     page.goto(f"{live_server.url}{map.get_absolute_url()}#6/48.948/1.670")
     markers = page.locator(".leaflet-marker-icon .icon-container")
     expect(markers).to_have_count(5)
-    colors = getColors(markers)
     # Alice Blue should only affect layer 1
-    assert colors.count("rgb(240, 248, 255)") == 1
+    markers = page.locator(
+        '.leaflet-marker-icon [style*="background-color: aliceblue"]'
+    )
+    expect(markers).to_have_count(1)
     # Dark Red as for map global rules
-    assert colors.count("rgb(139, 0, 0)") == 2
+    markers = page.locator('.leaflet-marker-icon [style*="background-color: darkred"]')
+    expect(markers).to_have_count(2)
 
 
 def test_rules_in_caption(live_server, page, map):

umap/tests/integration/test_draw_polygon.py

@@ -176,6 +176,7 @@ def test_can_draw_hole(page, live_server, tilelayer):
     expect(polygons).to_have_count(1)
     expect(vertices).to_have_count(4)
 
+    page.wait_for_timeout(300)  # Time for the panel animation to finish
     # First vertex of the hole will be created here
     map.click(position={"x": 180, "y": 120}, button="right")
     page.get_by_role("button", name="Start a hole here").click()
@@ -373,6 +374,7 @@ def test_can_clone_polygon(live_server, page, tilelayer, settings):
     map.click(position={"x": 100, "y": 100})
     # Click again to finish
     map.click(position={"x": 100, "y": 100})
+    page.wait_for_timeout(300)  # Time for the panel animation to finish
     expect(polygons).to_have_count(1)
     polygons.first.click(button="right")
     page.get_by_role("button", name="Clone this feature").click()
@@ -399,6 +401,7 @@ def test_can_transform_polygon_to_line(live_server, page, tilelayer, settings):
     map.click(position={"x": 100, "y": 100})
     expect(polygons).to_have_count(1)
     expect(paths).to_have_count(1)
+    page.wait_for_timeout(300)  # Time for the panel animation to finish
     polygons.first.click(button="right")
     page.get_by_role("button", name="Transform to lines").click()
     # No more polygons (will fill), but one path, it must be a line
@@ -426,6 +429,8 @@ def test_can_draw_a_polygon_and_invert_it(live_server, page, tilelayer, settings
     page.get_by_text("Advanced properties").click()
     page.get_by_text("Display the polygon inverted").click()
     data = save_and_get_json(page)
+    # Close save message
+    page.get_by_role("dialog").get_by_role("button", name="Close").click()
     assert len(data["features"]) == 1
     assert data["features"][0]["geometry"]["type"] == "Polygon"
     assert data["features"][0]["geometry"]["coordinates"] == [
@@ -469,6 +474,7 @@ def test_vertexmarker_not_shown_if_too_many(live_server, map, page, settings):
     settings.UMAP_ALLOW_ANONYMOUS = True
     page.goto(f"{live_server.url}/en/map/new/#15/48.4395/3.3189")
     page.get_by_title("Import data").click()
+    page.wait_for_timeout(300)  # Time for the panel animation to finish
     page.locator(".umap-import textarea").fill(geojson)
     page.locator('select[name="format"]').select_option("geojson")
     page.get_by_role("button", name="Import data", exact=True).click()

umap/tests/integration/test_draw_polyline.py

@@ -261,6 +261,9 @@ def test_can_extract_shape(live_server, page, tilelayer):
     map.click(position={"x": 200, "y": 200})
     # Click again to finish
     map.click(position={"x": 200, "y": 200})
+    # Let the panel fully open, not to close the contextmenu (when
+    # refocus on panel input)
+    page.wait_for_timeout(300)
     expect(lines).to_have_count(1)
     lines.first.click(position={"x": 10, "y": 1}, button="right")
     extract_button.click()
@@ -279,6 +282,9 @@ def test_can_clone_polyline(live_server, page, tilelayer, settings):
     map.click(position={"x": 100, "y": 200})
     # Click again to finish
     map.click(position={"x": 100, "y": 200})
+    # Let the panel fully open, not to close the contextmenu (when
+    # refocus on panel input)
+    page.wait_for_timeout(300)
     expect(lines).to_have_count(1)
     lines.first.click(position={"x": 10, "y": 1}, button="right")
     page.get_by_role("button", name="Clone this feature").click()
@@ -304,6 +310,8 @@ def test_can_transform_polyline_to_polygon(live_server, page, tilelayer, setting
     map.click(position={"x": 100, "y": 200})
     # Click again to finish
     map.click(position={"x": 100, "y": 200})
+    page.wait_for_timeout(300)  # Time for the panel animation to finish
+
     expect(paths).to_have_count(1)
     expect(polygons).to_have_count(0)
     paths.first.click(position={"x": 10, "y": 1}, button="right")
@@ -324,6 +332,9 @@ def test_can_delete_shape_using_toolbar(live_server, page, tilelayer, settings):
     map.click(position={"x": 100, "y": 100})
     map.click(position={"x": 100, "y": 200})
     map.click(position={"x": 100, "y": 200})
+    # Let the panel fully open, not to close the contextmenu (when
+    # refocus on panel input)
+    page.wait_for_timeout(300)
 
     # Now split the line
     map.click(position={"x": 100, "y": 100}, button="right")