Skip to content

Commit 85da08c

Browse files
committed
rename Sign in/out to Log in/out
This is the copy used in the prototype and design system, so we should follow it.
1 parent ddd839d commit 85da08c

File tree

4 files changed

+18
-18
lines changed

4 files changed

+18
-18
lines changed

manage_breast_screening/auth/jinja2/auth/sign_in.jinja

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@
33
{% block page_content %}
44
<div class="nhsuk-grid-row">
55
<div class="nhsuk-grid-column-two-thirds">
6-
<h1 class="nhsuk-heading-l">Sign in</h1>
6+
<h1 class="nhsuk-heading-l">Log in</h1>
77

88
<a class="nhsuk-button nhsuk-button--secondary" href="{{ url('auth:cis2_sign_in') }}">
9-
Sign in with CIS2
9+
Log in with CIS2
1010
</a>
1111
</div>
1212
</div>

manage_breast_screening/auth/views.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ def sign_in(request):
1616
request,
1717
"auth/sign_in.jinja",
1818
{
19-
"page_title": "Sign in",
19+
"page_title": "Log in",
2020
"navActive": "sign_in",
2121
},
2222
)

manage_breast_screening/core/jinja2/layout-app.jinja

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919

2020
{% do account_items.append({
2121
"href": url('auth:sign_out') if request and request.user.is_authenticated else url('auth:sign_in'),
22-
"text": "Sign out" if request and request.user.is_authenticated else "Sign in",
22+
"text": "Log out" if request and request.user.is_authenticated else "Log in",
2323
}) %}
2424

2525
{{ header({

manage_breast_screening/tests/system/test_cis2_sign_in.py

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -42,31 +42,31 @@ def userinfo(self, token=None):
4242
)
4343

4444
def test_sign_in_and_sign_out_via_cis2(self):
45-
self.given_i_am_on_the_sign_in_page()
46-
self.when_i_sign_in_via_cis2()
45+
self.given_i_am_on_the_log_in_page()
46+
self.when_i_log_in_via_cis2()
4747
self.then_i_am_redirected_to_home()
48-
self.then_header_shows_sign_out()
49-
self.when_i_click_sign_out()
50-
self.then_header_shows_sign_in()
48+
self.then_header_shows_log_out()
49+
self.when_i_click_log_out()
50+
self.then_header_shows_log_in()
5151

52-
def given_i_am_on_the_sign_in_page(self):
52+
def given_i_am_on_the_log_in_page(self):
5353
self.page.goto(self.live_server_url + reverse("auth:sign_in"))
5454

55-
def when_i_sign_in_via_cis2(self):
56-
self.page.get_by_text("Sign in with CIS2").click()
55+
def when_i_log_in_via_cis2(self):
56+
self.page.get_by_text("Log in with CIS2").click()
5757

5858
def then_i_am_redirected_to_home(self):
5959
home_path = reverse("clinics:index")
6060
expect(self.page).to_have_url(re.compile(home_path))
6161

62-
def then_header_shows_sign_out(self):
62+
def then_header_shows_log_out(self):
6363
header = self.page.get_by_role("navigation")
64-
expect(header.get_by_text("Sign out")).to_be_visible()
64+
expect(header.get_by_text("Log out")).to_be_visible()
6565

66-
def when_i_click_sign_out(self):
66+
def when_i_click_log_out(self):
6767
header = self.page.get_by_role("navigation")
68-
header.get_by_text("Sign out").click()
68+
header.get_by_text("Log out").click()
6969

70-
def then_header_shows_sign_in(self):
70+
def then_header_shows_log_in(self):
7171
header = self.page.get_by_role("navigation")
72-
expect(header.get_by_text("Sign in")).to_be_visible()
72+
expect(header.get_by_text("Log in")).to_be_visible()

0 commit comments

Comments
 (0)