bcgov
diff --git a/‎legal-api/pyproject.toml‎
Lines changed: 1 addition & 1 deletion b/‎legal-api/pyproject.toml‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎legal-api/src/legal_api/services/filings/validations/registration.py‎
Lines changed: 9 additions & 6 deletions b/‎legal-api/src/legal_api/services/filings/validations/registration.py‎
Lines changed: 9 additions & 6 deletions
diff --git a/‎legal-api/tests/unit/services/filings/validations/test_registration.py‎
Lines changed: 2 additions & 1 deletion b/‎legal-api/tests/unit/services/filings/validations/test_registration.py‎
Lines changed: 2 additions & 1 deletion
@@ -1,6 +1,6 @@
 [project]
 name = "legal-api"
-version = "3.1.26"
+version = "3.1.27"
 description = ""
 authors = [
     {name = "thor",email = "[email protected]"}
 
@@ -171,18 +171,19 @@ def validate_party(filing: dict, legal_type: str, filing_type="registration") ->
 def validate_completing_party_name(filing: dict, filing_type="registration") -> list:
     """Validate the completing party name matches the submitting user's name.
 
-    Clients cannot edit the completing party (it is pre-populated from their login),
-    so the submitted name must match their user record; registry and SBC staff enter it
-    manually from a paper form and API gateway users have their own workflow, so those
-    are skipped.
+    BC Services Card clients cannot edit the completing party (it is pre-populated from
+    their verified login), so the submitted name must match their user record. Registry and
+    SBC staff enter it manually from a paper form, API gateway users have their own workflow
+    and other login sources (e.g. BCeID) carry an editable display name rather than a verified
+    legal name, so those are skipped.
     """
-    api_login_source = "API_GW"  # jwt loginSource of an API gateway user
+    bcsc_login_source = "BCSC"  # jwt loginSource of a BC Services Card user
     msg = []
     current_user = getattr(request_ctx, "current_user", None) if has_request_context() else None
     if (not current_user
             or jwt.validate_roles(current_user, [STAFF_ROLE])
             or jwt.validate_roles(current_user, [SBC_STAFF_ROLE])
-            or current_user.get("loginSource") == api_login_source):
+            or current_user.get("loginSource") != bcsc_login_source):
         return msg
 
     officer = None
@@ -201,6 +202,8 @@ def validate_completing_party_name(filing: dict, filing_type="registration") ->
     user_name = " ".join(" ".join(
         filter(None, [user.firstname, user.middlename, user.lastname])
     ).split())
+    if not user_name:
+        return msg  # nothing on record to compare against
     filing_name = " ".join(" ".join(
         filter(None, [officer.get("firstName"), officer.get("middleName"), officer.get("lastName")])
     ).split())
 
@@ -455,7 +455,8 @@ def mock_check(permissions_msg, filing_type):
         ('client_first_filing_user_created_from_token', [BASIC_USER], 'BCSC', None, ('Joe P', 'Swanson'), None),
         ('client_first_filing_token_name_mismatch', [BASIC_USER], 'BCSC', None, ('Different', 'Person'),
          MISMATCH_ERROR),
-        ('client_user_record_no_name', [BASIC_USER], 'BCSC', (None, None, None), None, MISMATCH_ERROR),
+        ('client_user_record_no_name_skipped', [BASIC_USER], 'BCSC', (None, None, None), None, None),
+        ('bceid_display_name_skipped', [BASIC_USER], 'BCEID', ('Nevneet Grewal', None, None), None, None),
         ('staff_skipped', [STAFF_ROLE], 'IDIR', ('Different', None, 'Person'), None, None),
         ('sbc_staff_skipped', [SBC_STAFF_ROLE], 'IDIR', ('Different', None, 'Person'), None, None),
         ('api_gw_skipped', [BASIC_USER], 'API_GW', ('Different', None, 'Person'), None, None),