test_update_users KeyError: 'sn'

isort-check from invenio_users_resources.services.users.tasks import reindex_users from cds_rdm.ldap.client import LdapClient -from cds_rdm.ldap.user_importer import LdapUserImporter, update_or_create_names_vocabularies +from cds_rdm.ldap.user_importer import ( + LdapUserImporter, + update_or_create_names_vocabularies, +) from cds_rdm.ldap.utils import InvenioUser, serialize_ldap_user, user_exists

Black format check --- /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/ldap/api.py 2024-09-19 13:38:27.583855+00:00 +++ /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/ldap/api.py 2024-09-19 13:43:00.064553+00:00 @@ -16,11 +16,14 @@ from invenio_db import db from invenio_oauthclient.models import RemoteAccount from invenio_users_resources.services.users.tasks import reindex_users from cds_rdm.ldap.client import LdapClient -from cds_rdm.ldap.user_importer import LdapUserImporter, update_or_create_names_vocabularies +from cds_rdm.ldap.user_importer import ( + LdapUserImporter, + update_or_create_names_vocabularies, +) from cds_rdm.ldap.utils import InvenioUser, serialize_ldap_user, user_exists def get_ldap_users(log_func): """Create and return a map of all LDAP users."""

isort-check from flask import current_app from flask_security.confirmable import confirm_user +from invenio_access.permissions import system_identity from invenio_accounts.models import User from invenio_db import db from invenio_oauthclient.models import RemoteAccount, UserIdentity +from invenio_records_resources.proxies import current_service_registry from invenio_userprofiles.models import UserProfile -from invenio_records_resources.proxies import current_service_registry -from invenio_access.permissions import system_identity from sqlalchemy.orm.exc import NoResultFound

Black format check --- /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/ldap/user_importer.py 2024-09-19 13:38:27.583855+00:00 +++ /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/ldap/user_importer.py 2024-09-19 13:43:00.906780+00:00 @@ -34,11 +34,13 @@ }, "affiliations": [{"name": "CERN"}], } try: - fetched_name = names_service.read(system_identity, ldap_user["remote_account_person_id"]) + fetched_name = names_service.read( + system_identity, ldap_user["remote_account_person_id"] + ) # Determine if any updates are necessary fetched_name_dict = fetched_name.to_dict() update_needed = False for key, value in name_data.items(): if key not in fetched_name_dict or fetched_name_dict[key] != value: @@ -50,10 +52,11 @@ name = names_service.update(system_identity, fetched_name.id, name_data) except NoResultFound: name = names_service.create(system_identity, name_data) return name + class LdapUserImporter: """Import ldap users to Invenio RDM records. Expected input format for ldap users: @@ -106,11 +109,11 @@ user_id=user_id, extra_data=dict( keycloak_id=keycloak_id, person_id=employee_id, department=department ), ) - + def import_user(self, ldap_user): """Create Invenio users from LDAP export.""" user = self.create_invenio_user(ldap_user) user_id = user.id

isort-check """CDS-RDM ldap serializers.""" +import hashlib from functools import partial from invenio_accounts.models import User from invenio_userprofiles import UserProfile from cds_rdm.ldap.errors import InvalidLdapUser -import hashlib def serialize_ldap_user(ldap_user_data, log_func=None):

Black format check --- /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/ldap/utils.py 2024-09-19 13:38:27.583855+00:00 +++ /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/ldap/utils.py 2024-09-19 13:43:01.248128+00:00 @@ -131,11 +131,11 @@ def hash_value(value, length=16): """Return a hashed version of the value.""" # TODO: we should use a secret to encrypt the value (the repo is public) - value_bytes = value.encode('utf-8') + value_bytes = value.encode("utf-8") hash_object = hashlib.sha256() hash_object.update(value_bytes) hashed_id = hash_object.hexdigest()[:length] - - return hashed_id \ No newline at end of file + + return hashed_id

Process completed with exit code 1.