fix

hpi-schul-cloud · Nov 28, 2023 · 4607e09 · 4607e09
1 parent 9e82c45
commit 4607e09
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/apps/server/src/modules/provisioning/strategy/oidc/service/oidc-provisioning.service.spec.ts b/apps/server/src/modules/provisioning/strategy/oidc/service/oidc-provisioning.service.spec.ts
@@ -487,7 +487,7 @@ describe('OidcProvisioningService', () => {
 						externalId: 'externalId',
 						name: 'existingName',
 						officialSchoolNumber: 'existingOfficialSchoolNumber',
-						systems: [],
+						systems: undefined,
 						features: [SchoolFeatures.OAUTH_PROVISIONING_ENABLED],
 					});
 
@@ -882,7 +882,7 @@ describe('OidcProvisioningService', () => {
 			});
 		});
 
-		describe('when provisioning an existing group with no other group members', () => {
+		describe('when provisioning an existing group with empty other group members', () => {
 			const setup = () => {
 				const student: UserDO = userDoFactory
 					.withRoles([{ id: new ObjectId().toHexString(), name: RoleName.STUDENT }])

diff --git a/apps/server/src/modules/provisioning/strategy/oidc/service/oidc-provisioning.service.ts b/apps/server/src/modules/provisioning/strategy/oidc/service/oidc-provisioning.service.ts
@@ -186,7 +186,7 @@ export class OidcProvisioningService {
 	}
 
 	private async getFilteredGroupUsers(externalGroup: ExternalGroupDto, systemId: string): Promise<GroupUser[]> {
-		if (!externalGroup.otherUsers) {
+		if (!externalGroup.otherUsers?.length) {
 			return [];
 		}
 

diff --git a/apps/server/src/modules/provisioning/strategy/sanis/sanis.strategy.spec.ts b/apps/server/src/modules/provisioning/strategy/sanis/sanis.strategy.spec.ts
@@ -90,6 +90,9 @@ describe('SanisStrategy', () => {
 					vorname: 'Hans',
 					familienname: 'Peter',
 				},
+				geburt: {
+					datum: '2023-11-17',
+				},
 			},
 			personenkontexte: [
 				{
@@ -99,6 +102,9 @@ describe('SanisStrategy', () => {
 						id: new UUID('df66c8e6-cfac-40f7-b35b-0da5d8ee680e').toString(),
 						name: 'schoolName',
 						kennung: 'Kennung',
+						anschrift: {
+							ort: 'Hannover',
+						},
 					},
 					gruppen: [
 						{