Skip to content

Commit 5459adf

Browse files
committed
fix: add spaces around operators in f-strings for consistency
The changes add spaces around the `+` operator in f-strings to improve code readability and maintain consistency with PEP 8 style guidelines. This affects test assertions in both `beneficiary_service_test.py` and `group_beneficiary_service_test.py` where beneficiary counts are verified.
1 parent 371a5b5 commit 5459adf

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

social_protection/tests/beneficiary_service_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ def test_add_beneficiary(self):
9595
for i, individual in enumerate([self.individual, self.individual2]):
9696
uuid = self.add_beneficiary_return_uuid(individual, self.benefit_plan_no_max, status="ACTIVE")
9797
self.check_beneficiary_exists(uuid, with_status="ACTIVE")
98-
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i+1} beneficiaries should be added and active")
98+
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i + 1} beneficiaries should be added and active")
9999

100100
def test_update_beneficiary(self):
101101
def create_and_update_to_active(individual, benefit_plan):
@@ -126,7 +126,7 @@ def create_and_update_to_active(individual, benefit_plan):
126126
result, uuid = create_and_update_to_active(individual, self.benefit_plan_no_max)
127127
self.assertTrue(result.get('success', False), result.get('detail', "No details provided"))
128128
self.check_beneficiary_exists(uuid, "ACTIVE")
129-
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i+1} beneficiaries should be added and active")
129+
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i + 1} beneficiaries should be added and active")
130130

131131
def test_delete_beneficiary(self):
132132
uuid = self.add_beneficiary_return_uuid(self.individual)

social_protection/tests/group_beneficiary_service_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ def test_add_group_beneficiary(self):
8888
for i, group in enumerate([self.group, self.group2]):
8989
uuid = self.add_beneficiary_return_uuid(group, self.benefit_plan_no_max, status="ACTIVE")
9090
self.check_beneficiary_exists(uuid, with_status="ACTIVE")
91-
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i+1} beneficiaries should be added and active")
91+
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i + 1} beneficiaries should be added and active")
9292

9393
def test_update_group_beneficiary(self):
9494
def create_and_update_to_active(group, benefit_plan):
@@ -135,7 +135,7 @@ def create_and_update_to_active(group, benefit_plan):
135135
result, uuid = create_and_update_to_active(group, self.benefit_plan_no_max)
136136
self.assertTrue(result.get('success', False), result.get('detail', "No details provided"))
137137
self.check_beneficiary_exists(uuid, with_status="ACTIVE")
138-
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i+1} beneficiaries should be added and active")
138+
self.check_active_beneficiaries_count_eq(i + 1, self.benefit_plan_no_max, f"{i + 1} beneficiaries should be added and active")
139139

140140
def test_delete_group_beneficiary(self):
141141
uuid = self.add_beneficiary_return_uuid(self.group)

0 commit comments

Comments
 (0)