Skip to content

Commit

Permalink
GH-85: Upgrade GOV.UK Frontend to 5.4.0
Browse files Browse the repository at this point in the history
  • Loading branch information
markhobson committed Jun 18, 2024
1 parent e728cf2 commit aa6975e
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 11 deletions.
8 changes: 4 additions & 4 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
"author": "",
"license": "ISC",
"dependencies": {
"govuk-frontend": "~5.3.0",
"govuk-frontend": "~5.4.0",
"govuk-one-login-service-header": "govuk-one-login/service-header#f9ce92e4e9f5a8c0311e77bce211520a785e4ebc"
},
"devDependencies": {
Expand Down
4 changes: 2 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ dependencies = [
"dataclass-wizard~=0.22.0",
"flask~=3.0.0",
"flask-session[sqlalchemy]~=0.8.0",
"govuk-frontend-jinja~=3.0.0",
"govuk-frontend-wtf~=3.0.0",
"govuk-frontend-jinja~=3.1.0",
"govuk-frontend-wtf~=3.1.0",
"gunicorn~=22.0.0",
"inject~=5.2.0",
"pg8000~=1.31.0",
Expand Down
4 changes: 2 additions & 2 deletions tests/e2e/pages.py
Original file line number Diff line number Diff line change
Expand Up @@ -138,8 +138,8 @@ def __init__(self, banner: Locator):
self._heading = banner.get_by_role("paragraph")

@property
def heading(self) -> str | None:
return self._heading.text_content()
def heading(self) -> str:
return (self._heading.text_content() or "").strip()

@classmethod
def for_important(cls, page: Page) -> NotificationBannerComponent:
Expand Down
2 changes: 1 addition & 1 deletion tests/integration/pages.py
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ def __iter__(self) -> Iterator[str]:

class NotificationBannerComponent:
def __init__(self, banner: Tag):
self.heading = one(banner.select("p")).string
self.heading = (one(banner.select("p")).string or "").strip()

@classmethod
def for_important(cls, soup: BeautifulSoup) -> NotificationBannerComponent | None:
Expand Down
2 changes: 1 addition & 1 deletion tests/integration/test_scheme_funding.py
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ def test_cannot_spend_to_date_when_error(
assert (
change_spend_to_date_page.form.amount.is_errored
and change_spend_to_date_page.form.amount.error == "Error: Enter spend to date"
and change_spend_to_date_page.form.amount.value is None
and change_spend_to_date_page.form.amount.value == ""
)
actual_scheme = schemes.get(1)
assert actual_scheme
Expand Down

0 comments on commit aa6975e

Please sign in to comment.