diff options
-rw-r--r-- | CHANGES.md | 2 | ||||
-rw-r--r-- | archaeological_operations/tests.py | 29 | ||||
-rw-r--r-- | ishtar_common/tests.py | 14 |
3 files changed, 33 insertions, 12 deletions
diff --git a/CHANGES.md b/CHANGES.md index 6068371fb..28f5fa6b0 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -12,6 +12,8 @@ Ishtar changelog - syndication ui: better color for external sources - Search: sort by ID by default +### Bug fixes ### +- Fix geo finds for external sources: do not display v4.0.32 - 2022-12-12 -------------------- diff --git a/archaeological_operations/tests.py b/archaeological_operations/tests.py index a9fe8760e..7bc51539d 100644 --- a/archaeological_operations/tests.py +++ b/archaeological_operations/tests.py @@ -16,6 +16,7 @@ # along with this program. If not, see <http://www.gnu.org/licenses/>. # See the file COPYING for details. +from bs4 import BeautifulSoup import json import datetime from subprocess import Popen, PIPE @@ -4041,10 +4042,16 @@ class DocumentTest(OperationInitTest, TestCase): c.login(username=self.username, password=self.password) response = c.get(url, {"operation": self.operation.pk}) self.assertEqual(response.status_code, 200) - self.assertIn( - 'option value="{}" selected'.format(self.operation.pk), - response.content.decode(), - ) + + result = response.content.decode() + soup = BeautifulSoup(result, "lxml") + ok = False + for field in soup.findAll("option"): + keys = list(field.attrs.keys()) + if "selected" in keys and "value" in keys and field["value"] == str(self.operation.pk): + ok = True + break + self.assertTrue(ok, msg="Operation not selected in Document form") posted = {"authors": []} for related_key in models.Document.RELATED_MODELS: @@ -4079,10 +4086,16 @@ class DocumentTest(OperationInitTest, TestCase): c.login(username=self.username, password=self.password) response = c.get(url) self.assertEqual(response.status_code, 200) - self.assertIn( - 'option value="{}" selected'.format(self.operation.pk), - response.content.decode(), - ) + + result = response.content.decode() + soup = BeautifulSoup(result, "lxml") + ok = False + for field in soup.findAll("option"): + keys = list(field.attrs.keys()) + if "selected" in keys and "value" in keys and field["value"] == str(self.operation.pk): + ok = True + break + self.assertTrue(ok, msg="Operation not selected in Document form") posted = {"authors": [], "title": "hop2-is-back"} for related_key in models.Document.RELATED_MODELS: diff --git a/ishtar_common/tests.py b/ishtar_common/tests.py index f4e372770..d15675769 100644 --- a/ishtar_common/tests.py +++ b/ishtar_common/tests.py @@ -3918,10 +3918,16 @@ class DocumentTest(TestCase): c.login(username=self.username, password=self.password) response = c.get(url) self.assertEqual(response.status_code, 200) - self.assertIn( - 'option value="{}" selected'.format(self.ope1.pk), - response.content.decode() - ) + + result = response.content.decode() + soup = Soup(result, "lxml") + ok = False + for field in soup.findAll("option"): + keys = list(field.attrs.keys()) + if "selected" in keys and "value" in keys and field["value"] == str(self.ope1.pk): + ok = True + break + self.assertTrue(ok, msg="Operation not selected in Document form") self.assertIn(doc.title, response.content.decode()) posted = {"authors": [], "title": "A child document"} |