Skip to content

Commit

Permalink
resolve jinja2 Markup deprecation
Browse files Browse the repository at this point in the history
fixes #150
  • Loading branch information
guruofgentoo committed Sep 10, 2021
1 parent d8489d1 commit 86cfe0e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions keg_elements/forms/validators.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

from decimal import Decimal

import jinja2
import markupsafe
from wtforms import ValidationError
import re

Expand Down Expand Up @@ -100,6 +100,6 @@ def __call__(self, form, field):
else:
text = _('This value must be unique but is already assigned to {link}.',
link=self.object_html_link(other))
msg = jinja2.Markup(text)
msg = markupsafe.Markup(text)
raise ValidationError(msg)
return True
8 changes: 4 additions & 4 deletions keg_elements/tests/test_forms/test_validators.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# coding=utf-8
import jinja2
import markupsafe
import pytest
import wtforms
from werkzeug.datastructures import MultiDict
Expand Down Expand Up @@ -28,13 +28,13 @@ def test_validation_fails(self):
form = UniqueForm(MultiDict({'uq_field': '1'}))
assert form.uq_field.validate(form) is False
assert form.uq_field.errors == [
jinja2.Markup('This value must be unique but is already assigned to link.')
markupsafe.Markup('This value must be unique but is already assigned to link.')
]

form = UniqueForm(MultiDict({'uq_field': '1'}), obj=2)
assert form.uq_field.validate(form) is False
assert form.uq_field.errors == [
jinja2.Markup('This value must be unique but is already assigned to link.')
markupsafe.Markup('This value must be unique but is already assigned to link.')
]

def test_no_object_link_provided(self):
Expand All @@ -47,7 +47,7 @@ def get_object_by_field(self, field):
form = Form(MultiDict({'uq_field': '1'}))
assert form.uq_field.validate(form) is False
assert form.uq_field.errors == [
jinja2.Markup('This value must be unique but is already assigned.')
markupsafe.Markup('This value must be unique but is already assigned.')
]

def test_get_obj(self):
Expand Down

0 comments on commit 86cfe0e

Please sign in to comment.