From 3fa57c97d764a4d8f9bd251bf50141378e2abccb Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Tue, 29 Oct 2024 12:31:17 +0100
Subject: [PATCH] OZG-6719 OZG-7044 show syncResult according to AKs
---
.../organisationseinheit-list.component.html | 13 +++++++++--
...rganisationseinheit-list.component.spec.ts | 22 +------------------
.../organisationseinheit-list.component.ts | 6 ++---
3 files changed, 14 insertions(+), 27 deletions(-)
diff --git a/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.html b/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.html
index a42709e0e2..0887f0c89c 100644
--- a/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.html
+++ b/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.html
@@ -2,7 +2,10 @@
<ods-list-item
*ngFor="let organisationsEinheitResource of organisationsEinheitResources"
[routerLink]="getEncodedSelfLink(organisationsEinheitResource)"
- [class.text-red-500]="syncResultIsNotOk(organisationsEinheitResource)"
+ [class.text-red-500]="
+ organisationsEinheitResource.syncResult === AdminOrganisationsEinheitSyncResult.NOT_FOUND_IN_PVOG ||
+ organisationsEinheitResource.syncResult === AdminOrganisationsEinheitSyncResult.ORGANISATIONSEINHEIT_ID_NOT_UNIQUE
+ "
data-test-id="organisations-einheit-list-item"
>
<dl class="flex-1 basis-3/4 font-semibold" [class.pl-4]="organisationsEinheitResource.isChild">
@@ -19,10 +22,16 @@
<dt class="sr-only">Synchronisationsergebnis</dt>
<dd class="mt-1">
<ods-exclamation-icon
- *ngIf="syncResultIsNotOk(organisationsEinheitResource)"
+ *ngIf="organisationsEinheitResource.syncResult === AdminOrganisationsEinheitSyncResult.NOT_FOUND_IN_PVOG"
matTooltip="Organisationseinheit wurde nicht in den PVOG-Daten gefunden."
size="small"
/>
+ <ods-exclamation-icon
+ *ngIf="
+ organisationsEinheitResource.syncResult === AdminOrganisationsEinheitSyncResult.ORGANISATIONSEINHEIT_ID_NOT_UNIQUE
+ "
+ size="small"
+ />
</dd>
</dl>
</ods-list-item>
diff --git a/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.spec.ts b/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.spec.ts
index fa4d2943bd..e39d5d9c2c 100644
--- a/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.spec.ts
+++ b/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.spec.ts
@@ -48,7 +48,7 @@ describe('OrganisationsEinheitListComponent', () => {
describe('input', () => {
describe('organisationsEinheitResources', () => {
const organisationsEinheitResource: AdminOrganisationsEinheitResource = createAdminOrganisationsEinheitResource(
- AdminOrganisationsEinheitSyncResult.NAME_MISMATCH,
+ AdminOrganisationsEinheitSyncResult.NOT_FOUND_IN_PVOG,
);
beforeEach(() => {
@@ -107,26 +107,6 @@ describe('OrganisationsEinheitListComponent', () => {
expect(component.organisationsEinheitResources[2].isChild).toBeTruthy();
});
});
- describe('syncResultIsNotOk', () => {
- it('should return true', () => {
- const organisationsEinheitResource: AdminOrganisationsEinheitResource = createAdminOrganisationsEinheitResource(
- AdminOrganisationsEinheitSyncResult.NAME_MISMATCH,
- );
- const result: boolean = component.syncResultIsNotOk(organisationsEinheitResource);
-
- expect(result).toBeTruthy();
- });
-
- it('should return false', () => {
- const organisationsEinheitResource: AdminOrganisationsEinheitResource = createAdminOrganisationsEinheitResource(
- AdminOrganisationsEinheitSyncResult.OK,
- );
- const result: boolean = component.syncResultIsNotOk(organisationsEinheitResource);
-
- expect(result).toBeFalsy();
- });
- });
-
describe('getEncodedSelfLink', () => {
it('should return encoded self link', () => {
const organisationsEinheitResource: AdminOrganisationsEinheitResource = createAdminOrganisationsEinheitResource();
diff --git a/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.ts b/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.ts
index aab7414ed0..9faaabdd00 100644
--- a/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.ts
+++ b/alfa-client/libs/admin/settings/src/lib/organisationseinheit/organisationseinheit-container/organisationseinheit-list/organisationseinheit-list.component.ts
@@ -20,6 +20,8 @@ export class OrganisationsEinheitListComponent {
this.moveChildrenIntoParentLevel(list);
}
+ public readonly AdminOrganisationsEinheitSyncResult = AdminOrganisationsEinheitSyncResult;
+
moveChildrenIntoParentLevel(list: AdminOrganisationsEinheitResource[]): void {
list.forEach((parent) => {
this._organisationsEinheitResources.push(parent);
@@ -33,10 +35,6 @@ export class OrganisationsEinheitListComponent {
});
}
- syncResultIsNotOk(organisationsEinheitResource: AdminOrganisationsEinheitResource): boolean {
- return organisationsEinheitResource.syncResult !== AdminOrganisationsEinheitSyncResult.OK;
- }
-
getEncodedSelfLink(organisationsEinheitResource: AdminOrganisationsEinheitResource): ResourceUri {
const resourceUri: ResourceUri = getLink(organisationsEinheitResource, OrganisationsEinheitLinkRel.SELF).href;
return encodeUrlForEmbedding(resourceUri);
--
GitLab