Skip to content

Commit 5b3284b

Browse files
authored
Remove use of lxml (#4133)
Also marks test introduced in #4133 as flaky.
1 parent cc268b9 commit 5b3284b

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

app/tests/documentation_tests/test_views.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,7 @@ def nested_docpages():
188188

189189

190190
@pytest.mark.django_db
191+
@pytest.mark.flaky(reruns=3)
191192
@pytest.mark.parametrize(
192193
"level, num_queries",
193194
(

app/tests/groups_tests/test_views.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import pytest
2-
from lxml.html.diff import html_escape
2+
from django.utils.html import escape
33

44
from tests.algorithms_tests.factories import AlgorithmFactory
55
from tests.archives_tests.factories import ArchiveFactory
@@ -53,7 +53,7 @@ def get_user_autocomplete():
5353
assert response.status_code == 200
5454
assert str(u.pk) in response.json()["results"][0]["id"]
5555
assert (
56-
html_escape(str(u.user_profile.get_mugshot_url()))
56+
escape(str(u.user_profile.get_mugshot_url()))
5757
in response.json()["results"][0]["text"]
5858
)
5959
assert u.username in response.json()["results"][0]["text"]
@@ -122,7 +122,7 @@ def get_user_autocomplete():
122122

123123
assert str(u.pk) in response.json()["results"][0]["id"]
124124
assert (
125-
html_escape(str(u.user_profile.get_mugshot_url()))
125+
escape(str(u.user_profile.get_mugshot_url()))
126126
in response.json()["results"][0]["text"]
127127
)
128128
assert u.username in response.json()["results"][0]["text"]

0 commit comments

Comments
 (0)