From 031a8599aece2b1fbb7b2e573014e0364f5e76fe Mon Sep 17 00:00:00 2001 From: Nicola Soranzo Date: Mon, 4 Mar 2024 14:39:55 +0000 Subject: [PATCH] Fix ``test_page_encoding`` test Don't assume that the history has unencoded id 1. Broken by commit 85630e2ca90211bfaea459192a90b7070c0a024a . Fix the following error: ``` ______________ TestPageJsonEncodingIntegration.test_page_encoding ______________ self = history_id = '529fd61ab1c6cc36' def test_page_encoding(self, history_id: str): request = dict( slug="mypage", title="MY PAGE", content=f"""

Page!

""", ) page_response = self._post("pages", request, json=True) api_asserts.assert_status_code_is_ok(page_response) sa_session = self._app.model.session page_revision = sa_session.scalars(select(model.PageRevision).filter_by(content_format="html")).all()[0] > assert '''id="History-1"''' in page_revision.content, page_revision.content E AssertionError:

Page!

Embedded Galaxy History - 'test_history'

[Do not edit this block; Galaxy will fill it in with the annotated History when it is displayed]

E assert 'id="History-1"' in '

Page!

Embedded Galaxy History - \'test_history\'

[Do not edit this block; Galaxy will fill it in with the annotated History when it is displayed]

' E + where '

Page!

Embedded Galaxy History - \'test_history\'

[Do not edit this block; Galaxy will fill it in with the annotated History when it is displayed]

' = .content test/integration/test_page_revision_json_encoding.py:33: AssertionError ``` --- test/integration/test_page_revision_json_encoding.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/test/integration/test_page_revision_json_encoding.py b/test/integration/test_page_revision_json_encoding.py index 1e197b30d88..f112b282ae2 100644 --- a/test/integration/test_page_revision_json_encoding.py +++ b/test/integration/test_page_revision_json_encoding.py @@ -5,6 +5,8 @@ the security parameter to encode IDs may be changed by admins). Test case also v exported API values are encoded though. """ +import re + from sqlalchemy import select from galaxy import model @@ -21,6 +23,7 @@ class TestPageJsonEncodingIntegration(integration_util.IntegrationTestCase): self.dataset_populator = DatasetPopulator(self.galaxy_interactor) def test_page_encoding(self, history_id: str): + history_num_re = re.compile(r'id="History-\d+"') request = dict( slug="mypage", title="MY PAGE", @@ -30,13 +33,13 @@ class TestPageJsonEncodingIntegration(integration_util.IntegrationTestCase): api_asserts.assert_status_code_is_ok(page_response) sa_session = self._app.model.session page_revision = sa_session.scalars(select(model.PageRevision).filter_by(content_format="html")).all()[0] - assert '''id="History-1"''' in page_revision.content, page_revision.content + assert history_num_re.search(page_revision.content), page_revision.content assert f'''id="History-{history_id}"''' not in page_revision.content, page_revision.content show_page_response = self._get("pages/{}".format(page_response.json()["id"])) api_asserts.assert_status_code_is_ok(show_page_response) content = show_page_response.json()["content"] - assert '''id="History-1"''' not in content, content + assert not history_num_re.search(content), content assert f'''id="History-{history_id}"''' in content, content def test_page_encoding_markdown(self, history_id: str):