From 497dea04fd457b080d2c539f3dc83088af16201e Mon Sep 17 00:00:00 2001 From: Martin <git@mail.de> Date: Tue, 11 Feb 2025 16:07:58 +0100 Subject: [PATCH 1/8] OZG-7474 fix data test id for zuruecholen button --- .../zurueckholen-button/zurueckholen-button.component.html | 4 ++-- .../zurueckholen-button.component.spec.ts | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html index 3fd57ccdb2..4f41c6a5a1 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html @@ -27,7 +27,7 @@ @if (showAsIconButton) { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="zurueckholen-icon-button" + dataTestId="zurueckholen-icon-button" variant="ghost" size="fit" tooltip="Vorgang zurückholen" @@ -38,7 +38,7 @@ } @else { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="zurueckholen-button" + dataTestId="zurueckholen-button" text="Vorgang zurückholen" variant="outline" (clickEmitter)="zurueckholen()" diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.spec.ts b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.spec.ts index 3a1cd90115..f91b5423f7 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.spec.ts +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.spec.ts @@ -28,7 +28,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { UndoIconComponent } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; -import { getDataTestIdOf } from 'libs/tech-shared/test/data-test'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createVorgangWithEingangResource } from 'libs/vorgang-shared/test/vorgang'; import { MockComponent } from 'ng-mocks'; import { of } from 'rxjs'; @@ -38,8 +38,8 @@ describe('ZurueckholenButtonComponent', () => { let component: ZurueckholenButtonComponent; let fixture: ComponentFixture<ZurueckholenButtonComponent>; - const zurueckholenButton: string = getDataTestIdOf('zurueckholen-button'); - const zurueckholenIconButton: string = getDataTestIdOf('zurueckholen-icon-button'); + const zurueckholenButton: string = getDataTestIdAttributeOf('zurueckholen-button'); + const zurueckholenIconButton: string = getDataTestIdAttributeOf('zurueckholen-icon-button'); const vorgangCommandService = { ...mock(VorgangCommandService), -- GitLab From d8b394f112d8382d8b6e3350772289c18e13787f Mon Sep 17 00:00:00 2001 From: Martin <git@mail.de> Date: Tue, 11 Feb 2025 16:09:37 +0100 Subject: [PATCH 2/8] OZG-7474 fix verwerfen button data test id --- .../verwerfen-button/verwerfen-button.component.html | 4 ++-- .../verwerfen-button/verwerfen-button.component.spec.ts | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.html index 8d663e5e9b..784ddfbbf3 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.html @@ -27,7 +27,7 @@ @if (showAsIconButton) { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="verwerfen-icon-button" + dataTestId="verwerfen-icon-button" tooltip="Vorgang verwerfen" variant="ghost" size="fit" @@ -38,7 +38,7 @@ } @else { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="verwerfen-button" + dataTestId="verwerfen-button" text="Verwerfen" variant="outline" (clickEmitter)="verwerfen()" diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.spec.ts b/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.spec.ts index ebdb6df88b..75b739a230 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.spec.ts +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/verwerfen-button/verwerfen-button.component.spec.ts @@ -28,7 +28,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { DiscardVorgangIconComponent } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; -import { getDataTestIdOf } from 'libs/tech-shared/test/data-test'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createVorgangWithEingangResource } from 'libs/vorgang-shared/test/vorgang'; import { MockComponent } from 'ng-mocks'; import { of } from 'rxjs'; @@ -38,8 +38,8 @@ describe('VerwerfenButtonComponent', () => { let component: VerwerfenButtonComponent; let fixture: ComponentFixture<VerwerfenButtonComponent>; - const verwerfenButton: string = getDataTestIdOf('verwerfen-button'); - const verwerfenIconButton: string = getDataTestIdOf('verwerfen-icon-button'); + const verwerfenButton: string = getDataTestIdAttributeOf('verwerfen-button'); + const verwerfenIconButton: string = getDataTestIdAttributeOf('verwerfen-icon-button'); const vorgangCommandService = { ...mock(VorgangCommandService), -- GitLab From 08f6036be936b3253c3ce8c7dd79856e8a94331b Mon Sep 17 00:00:00 2001 From: Alexander Reifschneider <alexander.reifschneider@mgm-tp.com> Date: Tue, 11 Feb 2025 16:09:47 +0100 Subject: [PATCH 3/8] OZG-7474 Fix data test ids - postfach mail button - wiederoeffnen button --- .../postfach-mail-button.component.html | 12 +++++++----- .../postfach-mail-button.component.spec.ts | 8 ++++---- .../wiedereroeffnen-button.component.html | 4 ++-- .../wiedereroeffnen-button.component.spec.ts | 6 +++--- 4 files changed, 16 insertions(+), 14 deletions(-) diff --git a/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.html b/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.html index 8a29c815b6..440eb0b7fe 100644 --- a/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.html +++ b/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.html @@ -27,7 +27,7 @@ <!-- TODO Aufteilen in 3 einzelne Komponenten --> @if (showAsIconButton) { <ods-button - data-test-id="send-mail-icon-button" + dataTestId="send-mail-icon-button" [isLoading]="pendingSendPostfachMailCommand.resource | hasLink: commandLinkRel.UPDATE" tooltip="Neue Nachricht erstellen" variant="ghost" @@ -36,9 +36,10 @@ > <ods-mailbox-icon icon /> </ods-button> - } @else if (text && !toolTip) { + } + @if (!showAsIconButton && text && !toolTip) { <ods-button - data-test-id="create-mail-button-with-text" + dataTestId="create-mail-button-with-text" [isLoading]="pendingSendPostfachMailCommand.resource | hasLink: commandLinkRel.UPDATE" [text]="text" variant="outline" @@ -46,9 +47,10 @@ > <ods-plus-icon icon /> </ods-button> - } @else if (!text && toolTip) { + } + @if (!showAsIconButton && !text && toolTip) { <ods-button - data-test-id="create-mail-button-without-text" + dataTestId="create-mail-button-without-text" [isLoading]="pendingSendPostfachMailCommand.resource | hasLink: commandLinkRel.UPDATE" [tooltip]="toolTip" variant="ghost" diff --git a/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.spec.ts b/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.spec.ts index d02565a847..10a405d105 100644 --- a/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.spec.ts +++ b/alfa-client/libs/postfach/src/lib/postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component.spec.ts @@ -28,7 +28,7 @@ import { faker } from '@faker-js/faker'; import { ButtonComponent, MailboxIconComponent, PlusIconComponent, TooltipDirective } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; import { createPostfachMailListResource } from 'libs/postfach-shared/test/postfach'; -import { getDataTestIdOf } from 'libs/tech-shared/test/data-test'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { MockComponent, MockDirective } from 'ng-mocks'; import { PostfachMailButtonComponent } from './postfach-mail-button.component'; @@ -36,9 +36,9 @@ describe('PostfachMailButtonComponent', () => { let component: PostfachMailButtonComponent; let fixture: ComponentFixture<PostfachMailButtonComponent>; - const sendMailButtonWithText: string = getDataTestIdOf('create-mail-button-with-text'); - const sendMailButtonWithoutText: string = getDataTestIdOf('create-mail-button-without-text'); - const sendMailIconButton: string = getDataTestIdOf('send-mail-icon-button'); + const sendMailButtonWithText: string = getDataTestIdAttributeOf('create-mail-button-with-text'); + const sendMailButtonWithoutText: string = getDataTestIdAttributeOf('create-mail-button-without-text'); + const sendMailIconButton: string = getDataTestIdAttributeOf('send-mail-icon-button'); beforeEach(async () => { await TestBed.configureTestingModule({ diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.html index 51b70df609..0e3bba543b 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.html @@ -27,7 +27,7 @@ @if (showAsIconButton) { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="wiedereroeffnen-icon-button" + dataTestId="wiedereroeffnen-icon-button" tooltip="Vorgang wiedereröffnen" variant="ghost" size="fit" @@ -38,7 +38,7 @@ } @else { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="wiedereroeffnen-button" + dataTestId="wiedereroeffnen-button" text="Wiedereröffnen" variant="outline" (clickEmitter)="wiedereroeffnen()" diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.spec.ts b/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.spec.ts index 3ee20a3ec3..a522800052 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.spec.ts +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/wiedereroeffnen-button/wiedereroeffnen-button.component.spec.ts @@ -28,7 +28,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { UndoIconComponent } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; -import { getDataTestIdOf } from 'libs/tech-shared/test/data-test'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createVorgangWithEingangResource } from 'libs/vorgang-shared/test/vorgang'; import { MockComponent } from 'ng-mocks'; import { of } from 'rxjs'; @@ -37,8 +37,8 @@ import { WiedereroeffnenButtonComponent } from './wiedereroeffnen-button.compone describe('WiedereroeffnenButtonComponent', () => { let component: WiedereroeffnenButtonComponent; let fixture: ComponentFixture<WiedereroeffnenButtonComponent>; - const wiedereroeffnenButton: string = getDataTestIdOf('wiedereroeffnen-button'); - const wiedereroeffnenIconButton: string = getDataTestIdOf('wiedereroeffnen-icon-button'); + const wiedereroeffnenButton: string = getDataTestIdAttributeOf('wiedereroeffnen-button'); + const wiedereroeffnenIconButton: string = getDataTestIdAttributeOf('wiedereroeffnen-icon-button'); const vorgangCommandService = { ...mock(VorgangCommandService), -- GitLab From 6733afe93ddf8f5395cddbd709a4c1c42fbbdb50 Mon Sep 17 00:00:00 2001 From: Martin <git@mail.de> Date: Tue, 11 Feb 2025 16:11:24 +0100 Subject: [PATCH 4/8] OZG-7474 fix bearbeiten button data test id --- .../bearbeiten-button/bearbeiten-button.component.html | 4 ++-- .../bearbeiten-button/bearbeiten-button.component.spec.ts | 5 +++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.html index d1de12ab8c..107e3f67c2 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.html @@ -28,7 +28,7 @@ @if (showAsIconButton) { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="bearbeiten-icon-button" + dataTestId="bearbeiten-icon-button" tooltip="Vorgang bearbeiten" variant="ghost" size="fit" @@ -39,7 +39,7 @@ } @else { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="bearbeiten-button" + dataTestId="bearbeiten-button" text="Bearbeiten" variant="outline" (clickEmitter)="bearbeiten()" diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.spec.ts b/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.spec.ts index 627aa0b059..9691ba2813 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.spec.ts +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/bearbeiten-button/bearbeiten-button.component.spec.ts @@ -28,6 +28,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { EditIconComponent } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createVorgangWithEingangResource } from 'libs/vorgang-shared/test/vorgang'; import { MockComponent } from 'ng-mocks'; import { of } from 'rxjs'; @@ -37,8 +38,8 @@ describe('BearbeitenButtonComponent', () => { let component: BearbeitenButtonComponent; let fixture: ComponentFixture<BearbeitenButtonComponent>; - const bearbeitenButton: string = '[data-test-id="bearbeiten-button"]'; - const bearbeitenIconButton: string = '[data-test-id="bearbeiten-icon-button"]'; + const bearbeitenButton: string = getDataTestIdAttributeOf('bearbeiten-button'); + const bearbeitenIconButton: string = getDataTestIdAttributeOf('bearbeiten-icon-button'); const vorgangCommandService = { ...mock(VorgangCommandService), -- GitLab From 50abe80812e6c9ad5acf55fbd907623292544724 Mon Sep 17 00:00:00 2001 From: Martin <git@mail.de> Date: Tue, 11 Feb 2025 16:13:00 +0100 Subject: [PATCH 5/8] OZG-7474 fix annehmen button data test id --- .../buttons/annehmen-button/annehmen-button.component.html | 4 ++-- .../annehmen-button/annehmen-button.component.spec.ts | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.html index e3113ba3b2..7e55b9639e 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.html @@ -31,7 +31,7 @@ tooltip="Vorgang annehmen" variant="ghost" size="fit" - data-test-id="annehmen-icon-button" + dataTestId="annehmen-icon-button" > <ods-check-circle-icon icon class="fill-text" /> </ods-button-with-spinner> @@ -41,7 +41,7 @@ (clickEmitter)="annehmen()" text="Annehmen" variant="outline" - data-test-id="annehmen-button" + dataTestId="annehmen-button" > <ods-check-circle-icon icon /> </ods-button-with-spinner> diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.spec.ts b/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.spec.ts index 8e531074e2..0b4059086f 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.spec.ts +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/annehmen-button/annehmen-button.component.spec.ts @@ -29,7 +29,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { CheckCircleIconComponent, TooltipDirective } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; -import { getDataTestIdOf } from 'libs/tech-shared/test/data-test'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createVorgangWithEingangResource } from 'libs/vorgang-shared/test/vorgang'; import { MockComponent } from 'ng-mocks'; import { of } from 'rxjs'; @@ -39,8 +39,8 @@ describe('AnnehmenButtonComponent', () => { let component: AnnehmenButtonComponent; let fixture: ComponentFixture<AnnehmenButtonComponent>; - const annehmenButton: string = getDataTestIdOf('annehmen-button'); - const annehmenIconButton: string = getDataTestIdOf('annehmen-icon-button'); + const annehmenButton: string = getDataTestIdAttributeOf('annehmen-button'); + const annehmenIconButton: string = getDataTestIdAttributeOf('annehmen-icon-button'); const vorgangCommandService = { ...mock(VorgangCommandService), -- GitLab From 4404de34e278542363f8e9fd5d343d3ddacb564c Mon Sep 17 00:00:00 2001 From: Martin <git@mail.de> Date: Tue, 11 Feb 2025 16:14:49 +0100 Subject: [PATCH 6/8] OZG-7474 fix abschliessen data test id --- .../abschliessen-button/abschliessen-button.component.html | 4 ++-- .../abschliessen-button.component.spec.ts | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.html index f9670cbee8..758515c533 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.html @@ -28,7 +28,7 @@ @if (showAsIconButton) { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="abschliessen-icon-button" + dataTestId="abschliessen-icon-button" tooltip="Vorgang abschließen" variant="ghost" size="fit" @@ -39,7 +39,7 @@ } @else { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="abschliessen-button" + dataTestId="abschliessen-button" text="Abschließen" variant="outline" (clickEmitter)="abschliessen()" diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.spec.ts b/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.spec.ts index 964474db25..a7334741a0 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.spec.ts +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/abschliessen-button/abschliessen-button.component.spec.ts @@ -29,7 +29,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { CheckIconComponent } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; -import { getDataTestIdOf } from 'libs/tech-shared/test/data-test'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createVorgangWithEingangResource } from 'libs/vorgang-shared/test/vorgang'; import { MockComponent } from 'ng-mocks'; import { Observable, of } from 'rxjs'; @@ -39,8 +39,8 @@ describe('AbschliessenButtonComponent', () => { let component: AbschliessenButtonComponent; let fixture: ComponentFixture<AbschliessenButtonComponent>; - const abschliessenButton: string = getDataTestIdOf('abschliessen-button'); - const abschliessenIconButton: string = getDataTestIdOf('abschliessen-icon-button'); + const abschliessenButton: string = getDataTestIdAttributeOf('abschliessen-button'); + const abschliessenIconButton: string = getDataTestIdAttributeOf('abschliessen-icon-button'); const vorgangCommandService = { ...mock(VorgangCommandService), -- GitLab From af3b03471ac1134a9bced864942f8c9aadca4333 Mon Sep 17 00:00:00 2001 From: Alexander Reifschneider <alexander.reifschneider@mgm-tp.com> Date: Tue, 11 Feb 2025 16:16:46 +0100 Subject: [PATCH 7/8] OZG-7474 Add more data test ids --- .../zurueckstellen-button.component.html | 4 ++-- .../zurueckstellen-button.component.spec.ts | 6 +++--- .../erledigen-button-container.component.html | 2 +- .../erledigen-button-container.component.spec.ts | 3 ++- 4 files changed, 8 insertions(+), 7 deletions(-) diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.html index c595747b98..843b20963e 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.html @@ -28,7 +28,7 @@ @if (showAsIconButton) { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="zurueckstellen-icon-button" + dataTestId="zurueckstellen-icon-button" tooltip="Vorgang zurückstellen" variant="ghost" size="fit" @@ -39,7 +39,7 @@ } @else { <ods-button-with-spinner [stateResource]="commandStateResource$ | async" - data-test-id="zurueckstellen-button" + dataTestId="zurueckstellen-button" text="Zurückstellen" variant="outline" (clickEmitter)="zurueckstellen()" diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.spec.ts b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.spec.ts index b17b7c5a7c..4f9a74edc4 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.spec.ts +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckstellen-button/zurueckstellen-button.component.spec.ts @@ -28,7 +28,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { UndoIconComponent } from '@ods/system'; import { createCommandResource } from 'libs/command-shared/test/command'; -import { getDataTestIdOf } from 'libs/tech-shared/test/data-test'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createVorgangWithEingangResource } from 'libs/vorgang-shared/test/vorgang'; import { MockComponent } from 'ng-mocks'; import { of } from 'rxjs'; @@ -38,8 +38,8 @@ describe('ZurueckstellenButtonComponent', () => { let component: ZurueckstellenButtonComponent; let fixture: ComponentFixture<ZurueckstellenButtonComponent>; - const zurueckstellenButton: string = getDataTestIdOf('zurueckstellen-button'); - const zurueckstellenIconButton: string = getDataTestIdOf('zurueckstellen-icon-button'); + const zurueckstellenButton: string = getDataTestIdAttributeOf('zurueckstellen-button'); + const zurueckstellenIconButton: string = getDataTestIdAttributeOf('zurueckstellen-icon-button'); const vorgangCommandService = { ...mock(VorgangCommandService), diff --git a/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.html b/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.html index 5099a02204..71fb146636 100644 --- a/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.html +++ b/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.html @@ -25,7 +25,7 @@ --> @if (wiedervorlage | hasLink: linkRel.ERLEDIGEN) { <ods-button-with-spinner - data-test-id="erledigen-icon-button" + dataTestId="erledigen-icon-button" [stateResource]="erledigenCommand$ | async" tooltip="Als erledigt markieren" variant="ghost" diff --git a/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.spec.ts b/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.spec.ts index 69aaf5f00d..81eec11065 100644 --- a/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.spec.ts +++ b/alfa-client/libs/wiedervorlage/src/lib/erledigen-button-container/erledigen-button-container.component.spec.ts @@ -27,6 +27,7 @@ import { WiedervorlageLinkRel, WiedervorlageService } from '@alfa-client/wiederv import { ComponentFixture, TestBed } from '@angular/core/testing'; import { ButtonWithSpinnerComponent } from '@ods/component'; import { CheckCircleIconComponent } from '@ods/system'; +import { getDataTestIdAttributeOf } from 'libs/tech-shared/test/data-test'; import { createWiedervorlageResource } from 'libs/wiedervorlage-shared/test/wiedervorlage'; import { MockComponent } from 'ng-mocks'; import { ErledigenButtonContainerComponent } from './erledigen-button-container.component'; @@ -35,7 +36,7 @@ describe('ErledigenButtonContainerComponent', () => { let component: ErledigenButtonContainerComponent; let fixture: ComponentFixture<ErledigenButtonContainerComponent>; - const erledigenIconButton = '[data-test-id="erledigen-icon-button"]'; + const erledigenIconButton = getDataTestIdAttributeOf('erledigen-icon-button'); const wiedervorlageService = mock(WiedervorlageService); -- GitLab From 7e7e392e680bc3a503f045196c09a775e5723c16 Mon Sep 17 00:00:00 2001 From: Martin <git@mail.de> Date: Tue, 11 Feb 2025 16:26:01 +0100 Subject: [PATCH 8/8] OZG-7474 fix test --- .../zurueckholen-button/zurueckholen-button.component.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html index 4f41c6a5a1..d2ac600887 100644 --- a/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html +++ b/alfa-client/libs/vorgang-detail/src/lib/buttons/zurueckholen-button/zurueckholen-button.component.html @@ -39,7 +39,7 @@ <ods-button-with-spinner [stateResource]="commandStateResource$ | async" dataTestId="zurueckholen-button" - text="Vorgang zurückholen" + text="Zurückholen" variant="outline" (clickEmitter)="zurueckholen()" > -- GitLab