diff --git a/frontend/src/components/ActiveCitationInput.vue b/frontend/src/components/ActiveCitationInput.vue index b135c07fc8..b91c32662a 100644 --- a/frontend/src/components/ActiveCitationInput.vue +++ b/frontend/src/components/ActiveCitationInput.vue @@ -281,7 +281,7 @@ watch( { { { { }), ], }) - const button = screen.getByLabelText("Eintrag bearbeiten") - await user.click(button) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) expect(screen.getByLabelText("Art der Zitierung")).toBeVisible() expect( @@ -223,7 +223,7 @@ describe("Active Citations", () => { renderComponent({ modelValue: [generateActiveCitation()], }) - expect(screen.getByLabelText("Eintrag bearbeiten")).toBeInTheDocument() + expect(screen.getByLabelText("Listen Eintrag")).toBeInTheDocument() }) it("correctly updates value citation style input", async () => { @@ -241,8 +241,8 @@ describe("Active Citations", () => { expect(screen.queryByText(/Änderungen/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) await user.type( await screen.findByLabelText("Art der Zitierung"), @@ -264,8 +264,8 @@ describe("Active Citations", () => { expect(screen.queryByText(/EuGH-Vorlage/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) await user.type( await screen.findByLabelText("Dokumenttyp der Aktivzitierung"), @@ -287,8 +287,8 @@ describe("Active Citations", () => { expect(screen.queryByText(/AG Test/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) await user.type( await screen.findByLabelText("Gericht der Aktivzitierung"), @@ -309,8 +309,8 @@ describe("Active Citations", () => { }) expect(screen.queryByText(/new fileNumber/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileNumberInput = await screen.findByLabelText( "Aktenzeichen Aktivzitierung", @@ -330,8 +330,8 @@ describe("Active Citations", () => { }) expect(screen.queryByText(/02.02.2022/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileNumberInput = await screen.findByLabelText( "Entscheidungsdatum der Aktivzitierung", @@ -351,8 +351,8 @@ describe("Active Citations", () => { }) const activeCitations = screen.getAllByLabelText("Listen Eintrag") expect(activeCitations.length).toBe(2) - const buttonList = screen.getAllByLabelText("Eintrag löschen") - await user.click(buttonList[0]) + await user.click(activeCitations[0]) + await user.click(screen.getByLabelText("Eintrag löschen")) expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) @@ -364,8 +364,8 @@ describe("Active Citations", () => { const { user } = renderComponent({ modelValue }) const activeCitations = screen.getAllByLabelText("Listen Eintrag") expect(activeCitations.length).toBe(2) - const buttonList = screen.getAllByLabelText("Eintrag löschen") - await user.click(buttonList[0]) + await user.click(activeCitations[0]) + await user.click(screen.getByLabelText("Eintrag löschen")) expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) @@ -379,8 +379,8 @@ describe("Active Citations", () => { "Änderungen, label1, 01.02.2022, test fileNumber, documentType1", ), ).toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileNumberInput = await screen.findByLabelText( "Aktenzeichen Aktivzitierung", @@ -409,7 +409,8 @@ describe("Active Citations", () => { ], }) - await user.click(screen.getByLabelText("Eintrag bearbeiten")) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) expect(screen.getByLabelText("Art der Zitierung")).toBeVisible() ;[ @@ -453,8 +454,8 @@ describe("Active Citations", () => { it("displays error in list and edit component when fields missing", async () => { const modelValue: ActiveCitation[] = [generateActiveCitation()] const { user } = renderComponent({ modelValue }) - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileInput = await screen.findByLabelText( "Aktenzeichen Aktivzitierung", @@ -462,7 +463,7 @@ describe("Active Citations", () => { await user.clear(fileInput) await user.click(screen.getByLabelText("Aktivzitierung speichern")) expect(screen.getByLabelText(/Fehlerhafte Eingabe/)).toBeInTheDocument() - await user.click(editButton) + await user.click(itemHeader) expect(screen.getAllByText(/Pflichtfeld nicht befüllt/).length).toBe(1) }) @@ -494,8 +495,8 @@ describe("Active Citations", () => { }), ], }) - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) expect(screen.getByText("Art der Zitierung *")).toBeVisible() expect(screen.getAllByText(/Pflichtfeld nicht befüllt/).length).toBe(1) diff --git a/frontend/test/components/ensuingDecisions.spec.ts b/frontend/test/components/ensuingDecisions.spec.ts index e740bb1562..3dcdd8de96 100644 --- a/frontend/test/components/ensuingDecisions.spec.ts +++ b/frontend/test/components/ensuingDecisions.spec.ts @@ -184,7 +184,7 @@ describe("EnsuingDecisions", () => { expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) - it("click on edit icon, opens the list entry in edit mode", async () => { + it("click on list item, opens the list entry in edit mode", async () => { const { user } = renderComponent({ modelValue: [ generateEnsuingDecision({ @@ -197,8 +197,8 @@ describe("EnsuingDecisions", () => { screen.queryByLabelText("Nachgehende Entscheidung speichern"), ).not.toBeInTheDocument() - const button = screen.getByLabelText("Eintrag bearbeiten") - await user.click(button) + const entryHeader = screen.getByLabelText("Listen Eintrag") + await user.click(entryHeader) expect( screen.getByLabelText("Nachgehende Entscheidung speichern"), @@ -210,8 +210,8 @@ describe("EnsuingDecisions", () => { modelValue: [generateEnsuingDecision()], }) - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const entryHeader = screen.getByLabelText("Listen Eintrag") + await user.click(entryHeader) const checkbox = await screen.findByLabelText("Anhängige Entscheidung") @@ -228,8 +228,8 @@ describe("EnsuingDecisions", () => { expect(screen.queryByText(/AG Test/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const entryHeader = screen.getByLabelText("Listen Eintrag") + await user.click(entryHeader) await user.type( await screen.findByLabelText("Gericht Nachgehende Entscheidung"), @@ -250,8 +250,8 @@ describe("EnsuingDecisions", () => { }) expect(screen.queryByText(/new fileNumber/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const entryHeader = screen.getByLabelText("Listen Eintrag") + await user.click(entryHeader) const fileNumberInput = await screen.findByLabelText( "Aktenzeichen Nachgehende Entscheidung", @@ -271,8 +271,8 @@ describe("EnsuingDecisions", () => { }) expect(screen.queryByText(/02.02.2022/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const entryHeader = screen.getByLabelText("Listen Eintrag") + await user.click(entryHeader) const fileNumberInput = await screen.findByLabelText( "Entscheidungsdatum Nachgehende Entscheidung", @@ -292,8 +292,9 @@ describe("EnsuingDecisions", () => { }) const proceedingDecisions = screen.getAllByLabelText("Listen Eintrag") expect(proceedingDecisions.length).toBe(2) - const buttonList = screen.getAllByLabelText("Eintrag löschen") - await user.click(buttonList[0]) + + await user.click(proceedingDecisions[0]) + await user.click(screen.getByLabelText("Eintrag löschen")) expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) @@ -306,10 +307,8 @@ describe("EnsuingDecisions", () => { }), ], }) - expect(screen.getByLabelText("Eintrag bearbeiten")).toBeInTheDocument() - - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const entryHeader = screen.getByLabelText("Listen Eintrag") + await user.click(entryHeader) expect( screen.queryByLabelText("Anhängige Entscheidung"), @@ -338,7 +337,7 @@ describe("EnsuingDecisions", () => { await user.click(saveButton) expect( screen.getByText( - /nachgehend, label1, 01.02.2022, test fileNumber, documentType1, Vermerk, ABC/, + /nachgehend, label1, 01.02.2022, test fileNumber, documentType1, Vermerk/, ), ).toBeInTheDocument() }) @@ -373,8 +372,8 @@ describe("EnsuingDecisions", () => { it("displays error in list and edit component when fields missing", async () => { const modelValue: EnsuingDecision[] = [generateEnsuingDecision()] const { user } = renderComponent({ modelValue }) - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const entryHeader = screen.getByLabelText("Listen Eintrag") + await user.click(entryHeader) const fileInput = await screen.findByLabelText( "Aktenzeichen Nachgehende Entscheidung", @@ -384,7 +383,7 @@ describe("EnsuingDecisions", () => { screen.getByLabelText("Nachgehende Entscheidung speichern"), ) expect(screen.getByLabelText(/Fehlerhafte Eingabe/)).toBeInTheDocument() - await user.click(editButton) + await user.click(entryHeader) expect(screen.getAllByText(/Pflichtfeld nicht befüllt/).length).toBe(1) }) }) diff --git a/frontend/test/components/normReferences.spec.ts b/frontend/test/components/normReferences.spec.ts index 50b2c915b4..b9f037af6b 100644 --- a/frontend/test/components/normReferences.spec.ts +++ b/frontend/test/components/normReferences.spec.ts @@ -92,8 +92,6 @@ describe("Norm references", () => { ] renderComponent({ modelValue }) - expect(screen.getAllByLabelText("Eintrag bearbeiten").length).toBe(2) - expect(screen.getAllByLabelText("Eintrag löschen").length).toBe(2) expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(2) expect( screen.queryByLabelText("RIS-Abkürzung der Norm"), @@ -102,7 +100,7 @@ describe("Norm references", () => { expect(screen.getByText(/§ 345/)).toBeInTheDocument() }) - it("click on edit icon, opens the list entry in edit mode", async () => { + it("click on list item, opens the list entry in edit mode", async () => { vi.spyOn(documentUnitService, "validateSingleNorm").mockImplementation(() => Promise.resolve({ status: 200, data: "Ok" }), ) @@ -113,8 +111,8 @@ describe("Norm references", () => { }), ], }) - const button = screen.getByLabelText("Eintrag bearbeiten") - await user.click(button) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) expect(screen.getByLabelText("RIS-Abkürzung der Norm")).toBeInTheDocument() expect(screen.getByLabelText("Einzelnorm der Norm")).toBeInTheDocument() @@ -127,9 +125,10 @@ describe("Norm references", () => { modelValue: [generateNormReference(), generateNormReference()], }) - expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(2) - const buttonList = screen.getAllByLabelText("Eintrag löschen") - await user.click(buttonList[0]) + const norms = screen.getAllByLabelText("Listen Eintrag") + expect(norms.length).toBe(2) + await user.click(norms[0]) + await user.click(screen.getByLabelText("Eintrag löschen")) expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) @@ -154,8 +153,8 @@ describe("Norm references", () => { ] const { user } = renderComponent({ modelValue }) await screen.findByText(/CDE, 01.02.2022, 2022/) - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const abbreviationInput = await screen.findByLabelText( "RIS-Abkürzung der Norm", @@ -166,7 +165,7 @@ describe("Norm references", () => { await user.click(screen.getByLabelText("Norm speichern")) await screen.findByText(/01.02.2022, 2022/) expect(screen.getByLabelText(/Fehlerhafte Eingabe/)).toBeInTheDocument() - await user.click(editButton) + await user.click(itemHeader) expect(screen.getAllByText(/Pflichtfeld nicht befüllt/).length).toBe(1) }) }) diff --git a/frontend/test/components/previousDecisions.spec.ts b/frontend/test/components/previousDecisions.spec.ts index d40216ffbe..537d658555 100644 --- a/frontend/test/components/previousDecisions.spec.ts +++ b/frontend/test/components/previousDecisions.spec.ts @@ -181,7 +181,7 @@ describe("PreviousDecisions", () => { expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) - it("click on edit icon, opens the list entry in edit mode", async () => { + it("click on list item, opens the list entry in edit mode", async () => { const { user } = renderComponent({ modelValue: [ generatePreviousDecision({ @@ -189,8 +189,8 @@ describe("PreviousDecisions", () => { }), ], }) - const button = screen.getByLabelText("Eintrag bearbeiten") - await user.click(button) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) expect( screen.getByLabelText("Gericht Vorgehende Entscheidung"), @@ -213,7 +213,7 @@ describe("PreviousDecisions", () => { renderComponent({ modelValue: [generatePreviousDecision()], }) - expect(screen.getByLabelText("Eintrag bearbeiten")).toBeInTheDocument() + expect(screen.getByLabelText("Listen Eintrag")).toBeInTheDocument() }) it("correctly updates value court input", async () => { @@ -223,8 +223,8 @@ describe("PreviousDecisions", () => { expect(screen.queryByText(/AG Test/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) await user.type( await screen.findByLabelText("Gericht Vorgehende Entscheidung"), @@ -245,8 +245,8 @@ describe("PreviousDecisions", () => { }) expect(screen.queryByText(/new fileNumber/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileNumberInput = await screen.findByLabelText( "Aktenzeichen Vorgehende Entscheidung", @@ -265,8 +265,8 @@ describe("PreviousDecisions", () => { modelValue: [generatePreviousDecision()], }) - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const checkbox = await screen.findByLabelText( "Datum Unbekannt Vorgehende Entscheidung", @@ -284,8 +284,8 @@ describe("PreviousDecisions", () => { }) expect(screen.queryByText(/02.02.2022/)).not.toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileNumberInput = await screen.findByLabelText( "Entscheidungsdatum Vorgehende Entscheidung", @@ -305,8 +305,8 @@ describe("PreviousDecisions", () => { }) const proceedingDecisions = screen.getAllByLabelText("Listen Eintrag") expect(proceedingDecisions.length).toBe(2) - const buttonList = screen.getAllByLabelText("Eintrag löschen") - await user.click(buttonList[0]) + await user.click(proceedingDecisions[0]) + await user.click(screen.getByLabelText("Eintrag löschen")) expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) @@ -318,8 +318,8 @@ describe("PreviousDecisions", () => { const { user } = renderComponent({ modelValue }) const proceedingDecisions = screen.getAllByLabelText("Listen Eintrag") expect(proceedingDecisions.length).toBe(2) - const buttonList = screen.getAllByLabelText("Eintrag löschen") - await user.click(buttonList[0]) + await user.click(proceedingDecisions[0]) + await user.click(screen.getByLabelText("Eintrag löschen")) expect(screen.getAllByLabelText("Listen Eintrag").length).toBe(1) }) @@ -331,8 +331,8 @@ describe("PreviousDecisions", () => { expect( screen.getByText("label1, 01.02.2022, test fileNumber, documentType1"), ).toBeInTheDocument() - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileNumberInput = await screen.findByLabelText( "Aktenzeichen Vorgehende Entscheidung", @@ -358,7 +358,7 @@ describe("PreviousDecisions", () => { }), ], }) - expect(screen.getByLabelText("Eintrag bearbeiten")).toBeInTheDocument() + expect(screen.getByLabelText("Listen Eintrag")).toBeInTheDocument() }) it("lists search results", async () => { @@ -391,8 +391,8 @@ describe("PreviousDecisions", () => { it("displays error in list and edit component when fields missing", async () => { const modelValue: PreviousDecision[] = [generatePreviousDecision()] const { user } = renderComponent({ modelValue }) - const editButton = screen.getByLabelText("Eintrag bearbeiten") - await user.click(editButton) + const itemHeader = screen.getByLabelText("Listen Eintrag") + await user.click(itemHeader) const fileInput = await screen.findByLabelText( "Aktenzeichen Vorgehende Entscheidung", @@ -400,7 +400,7 @@ describe("PreviousDecisions", () => { await user.clear(fileInput) await user.click(screen.getByLabelText("Vorgehende Entscheidung speichern")) expect(screen.getByLabelText(/Fehlerhafte Eingabe/)).toBeInTheDocument() - await user.click(editButton) + await user.click(itemHeader) expect(screen.getAllByText(/Pflichtfeld nicht befüllt/).length).toBe(1) }) }) diff --git a/frontend/test/components/searchResultList.spec.ts b/frontend/test/components/searchResultList.spec.ts index 045612daf6..61f003e0bf 100644 --- a/frontend/test/components/searchResultList.spec.ts +++ b/frontend/test/components/searchResultList.spec.ts @@ -3,20 +3,19 @@ import { render, screen } from "@testing-library/vue" import SearchResultList, { SearchResults, } from "@/components/SearchResultList.vue" -import ActiveCitation from "@/domain/activeCitation" -import ProceedingDecision from "@/domain/previousDecision" +import RelatedDocumentation from "@/domain/relatedDocumentation" function renderSearchResults( - searchResults?: SearchResults, + searchResults?: SearchResults, isLoading?: boolean, ) { const props: { - searchResults: SearchResults + searchResults: SearchResults isLoading: boolean } = { searchResults: searchResults ?? [ { - decision: new ProceedingDecision({ + decision: new RelatedDocumentation({ ...{ court: { type: "fooType", @@ -44,7 +43,7 @@ function renderSearchResults( return { user, ...utils } } -describe("ProceedingDecision SearchResult List", () => { +describe("Search result list", () => { it("renders correctly", async () => { renderSearchResults() @@ -53,17 +52,17 @@ describe("ProceedingDecision SearchResult List", () => { expect(screen.queryByText(/Bereits hinzugefügt/)).not.toBeInTheDocument() }) - it("indicates not yet added proceeding decisions", async () => { + it("indicates not yet added previous decisions", async () => { renderSearchResults() expect(await screen.findByText("Übernehmen")).toBeVisible() expect(screen.queryByText(/Bereits hinzugefügt/)).not.toBeInTheDocument() }) - it("indicates already added proceeding decisions", async () => { + it("indicates already added previous decisions", async () => { renderSearchResults([ { - decision: new ProceedingDecision({ + decision: new RelatedDocumentation({ ...{ court: { type: "fooType", @@ -97,7 +96,7 @@ describe("ProceedingDecision SearchResult List", () => { it("renders search results for active citations", async () => { renderSearchResults([ { - decision: new ActiveCitation({ + decision: new RelatedDocumentation({ ...{ court: { type: "fooType", diff --git a/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts b/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts index e7eccbda62..1078929c8b 100644 --- a/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts +++ b/frontend/test/e2e/caselaw/categories/contentRelatedIndexing/active-citations.spec.ts @@ -38,7 +38,10 @@ test.describe("active citations", () => { await page.getByLabel("Aktivzitierung speichern").click() await expect(page.getByLabel("Fehlerhafte Eingabe")).toBeVisible() - await page.getByLabel("Eintrag bearbeiten").click() + await page + .getByLabel("Aktivzitierung", { exact: true }) + .getByLabel("Listen Eintrag") + .click() await expect( page.getByLabel("Aktivzitierung").getByText("Pflichtfeld nicht befüllt"), ).toHaveCount(3) diff --git a/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts b/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts index ab084b30a3..714652be11 100644 --- a/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts +++ b/frontend/test/e2e/caselaw/categories/ensuing-decisions.spec.ts @@ -129,15 +129,13 @@ test.describe("ensuing decisions", () => { //make sure to have citation style in list await expect( page.getByText( - `nachgehend, AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, ${prefilledDocumentUnit.documentNumber}`, + `nachgehend, AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil`, ), ).toBeVisible() - await expect(page.getByLabel("Eintrag löschen")).toBeVisible() - - //can not be edited - await expect(page.getByLabel("Eintrag bearbeiten")).toBeVisible() - - await page.getByLabel("Eintrag bearbeiten").click() + await page + .getByLabel("Nachgehende Entscheidung", { exact: true }) + .getByLabel("Listen Eintrag") + .click() await expect( page.getByLabel("Gericht Nachgehende Entscheidung"), ).toBeHidden() @@ -157,7 +155,7 @@ test.describe("ensuing decisions", () => { await page.getByLabel("Nachgehende Entscheidung speichern").click() await expect( page.getByText( - `nachgehend, AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, Vermerk, ${prefilledDocumentUnit.documentNumber}`, + `nachgehend, AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, Vermerk`, ), ).toBeVisible() }) @@ -175,7 +173,10 @@ test.describe("ensuing decisions", () => { await page.getByLabel("Nachgehende Entscheidung speichern").click() await expect(page.getByLabel("Fehlerhafte Eingabe")).toBeVisible() - await page.getByLabel("Eintrag bearbeiten").click() + await page + .getByLabel("Nachgehende Entscheidung", { exact: true }) + .getByLabel("Listen Eintrag") + .click() await expect( page .getByLabel("Nachgehende Entscheidung") diff --git a/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts b/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts index 431731d385..fdde757444 100644 --- a/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts +++ b/frontend/test/e2e/caselaw/categories/previous-decisions.spec.ts @@ -44,7 +44,10 @@ test.describe("previous decisions", () => { await page.getByLabel("Vorgehende Entscheidung speichern").click() await expect(page.getByLabel("Fehlerhafte Eingabe")).toBeVisible() - await page.getByLabel("Eintrag bearbeiten").click() + await page + .getByLabel("Vorgehende Entscheidung", { exact: true }) + .getByLabel("Listen Eintrag") + .click() await expect( page .getByLabel("Vorgehende Entscheidung") @@ -160,7 +163,10 @@ test.describe("previous decisions", () => { { clickSaveButton: true }, ) - await page.getByLabel("Eintrag bearbeiten").click() + await page + .getByLabel("Vorgehende Entscheidung", { exact: true }) + .getByLabel("Listen Eintrag") + .click() await page .locator( "[aria-label='Abweichendes Aktenzeichen Vorgehende Entscheidung anzeigen']", @@ -187,7 +193,10 @@ test.describe("previous decisions", () => { { clickSaveButton: true }, ) - await page.getByLabel("Eintrag bearbeiten").click() + await page + .getByLabel("Vorgehende Entscheidung", { exact: true }) + .getByLabel("Listen Eintrag") + .click() await page .locator( "[aria-label='Abweichendes Aktenzeichen Vorgehende Entscheidung anzeigen']", diff --git a/frontend/test/e2e/caselaw/search/small-search.spec.ts b/frontend/test/e2e/caselaw/search/small-search.spec.ts index b80bb69faf..8e5bd74ad6 100644 --- a/frontend/test/e2e/caselaw/search/small-search.spec.ts +++ b/frontend/test/e2e/caselaw/search/small-search.spec.ts @@ -44,7 +44,7 @@ test("manually added items are saved, can be edited and deleted", async ({ await expect(container.getByText(fileNumber1)).toBeVisible() // edit entry - await container.getByLabel("Eintrag bearbeiten").click() + await container.getByLabel("Listen Eintrag").click() await container .getByLabel("Aktenzeichen " + section, { exact: true }) .fill(fileNumber2) @@ -70,11 +70,10 @@ test("manually added items are saved, can be edited and deleted", async ({ await expect(container.getByLabel("Listen Eintrag")).toHaveCount(2) await page.reload() - await expect(container.getByLabel("Listen Eintrag")).toHaveCount(2) - await expect(container.getByLabel("Eintrag löschen")).toHaveCount(2) - await expect(container.getByLabel("Eintrag bearbeiten")).toHaveCount(2) - - await container.getByLabel("Eintrag löschen").first().click() + const listEntries = container.getByLabel("Listen Eintrag") + await expect(listEntries).toHaveCount(2) + await listEntries.first().click() + await container.getByLabel("Eintrag löschen").click() await expect(container.getByLabel("Listen Eintrag")).toHaveCount(1) }) } @@ -126,7 +125,7 @@ test("search for documentunits and link decision", async ({ await expect(container.getByText("1 Ergebnis gefunden")).toBeVisible() - let summary = `AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil, ${prefilledDocumentUnit.documentNumber}` + let summary = `AG Aachen, 31.12.2019, ${prefilledDocumentUnit.coreData.fileNumbers?.[0]}, Anerkenntnisurteil` const result = container.getByText(summary) await expect(result).toBeVisible() @@ -139,9 +138,7 @@ test("search for documentunits and link decision", async ({ const listItem = container.getByLabel("Listen Eintrag").last() await expect(listItem).toBeVisible() - await expect(listItem).toHaveText(summary, { useInnerText: true }) - - await expect(container.getByLabel("Eintrag löschen")).toBeVisible() + await expect(listItem).toContainText(summary, { useInnerText: true }) // search for same parameters gives same result, indication that decision is already added await container.getByLabel("Weitere Angabe").click() @@ -162,8 +159,7 @@ test("search for documentunits and link decision", async ({ await expect(container.getByText("Bereits hinzugefügt")).toBeVisible() //can be edited - await expect(container.getByLabel("Eintrag bearbeiten")).toBeVisible() - await container.getByLabel("Eintrag bearbeiten").click() + await container.getByLabel("Listen Eintrag").last().click() if (container === activeCitationContainer) { await fillActiveCitationInputs(page, { citationType: "Änderung" }) @@ -186,10 +182,9 @@ test("search for documentunits and link decision", async ({ .click() } - await expect(container.getByText(summary)).toBeVisible() - //can be deleted - await container.getByLabel("Eintrag löschen").first().click() + await container.getByLabel("Listen Eintrag").last().click() + await container.getByLabel("Eintrag löschen").click() await expect(container.getByLabel("Listen Eintrag")).toHaveCount(1) await expect(