test: Vorgang-Detail Seite: Share (mail-to) Link enthält URL zum Vorgang

Reviewed-on: #41

src/lib/components/NameItemEditor.svelte

@@ -15,7 +15,7 @@
 	export let currentName: string;
 	export let vorgangToken: string | null;
 	export let onSave: (n: string, o: string, t?: string) => unknown = () => {};
-	export let onDelete: (n: string) => unknown | null = () => {};
+	export let onDelete: ((n: string) => unknown) | null = () => {};
 
 	let localName = currentName;
 	let isEditing = false;

src/lib/server/vorgangService.ts

@@ -238,11 +238,11 @@ export const vorgangPINValidation = function (vorgangToken: string, vorgangPIN:
 /**
  * Change VorgangName or VorgangPIN
  * @param vorgangToken
- * @param newName
+ * @param newValue
  * @returns {int} number of affected lines
  */
 export const updateVorgangAttrByToken = function (vorgangToken: string,
-											newName: string,
+											newValue: string,
 											column: string) {
 	const renameSQLStmt = `UPDATE cases set ${column} = ? WHERE token = ?`;
 	const statement = db.prepare(renameSQLStmt);
@@ -250,7 +250,7 @@ export const updateVorgangAttrByToken = function (vorgangToken: string,
 	let info;
 	
 	try {
-		info = statement.run(newName, vorgangToken);
+		info = statement.run(newValue, vorgangToken);
 	} catch (err) {
 		console.log(`error: ${err}`)
 		return 0;

src/routes/(angemeldet)/+layout.server.ts

@@ -1,8 +1,6 @@
 import { type ServerLoadEvent } from '@sveltejs/kit';
 import type { PageServerLoad } from '../anmeldung/$types';
 
-import { ROUTE_NAMES } from '..';
-
 export const load: PageServerLoad = (event: ServerLoadEvent) => {
 	if (event.locals.user) {
 		return {

src/routes/(angemeldet)/upload/+page.server.ts

@@ -1,10 +1,8 @@
 import { Readable } from 'stream';
 import { BUCKET, client } from '$lib/minio';
 import { fail, error } from '@sveltejs/kit';
-import { v4 as uuidv4 } from 'uuid';
 
-import { db } from '$lib/server/dbService';
-import { getVorgangByName, vorgangNameExists } from '$lib/server/vorgangService';
+import { getVorgangByName } from '$lib/server/vorgangService';
 
 const isRequiredFieldValid = (value: unknown) => {
 	if (value == null) return false;

src/routes/api/users/[user]/+server.ts

@@ -1,4 +1,3 @@
-import { json } from '@sveltejs/kit';
 import { deleteUser } from '$lib/server/userService';
 
 export async function DELETE({ params }) {

tests/components/NameItemEditor.test.ts

@@ -18,7 +18,13 @@ describe('NameItemEditor - Funktionalität', () => {
 		onDelete
 	};
 
-	test.todo('FocusIn nach Klick auf edit');
+	test('Focus Input nach Klick auf edit', async () => {
+        render(NameItemEditor, { props: baseProps });
+        await fireEvent.click(screen.getByTestId('edit-button'));
+        const input = screen.getByTestId('test-input');
+
+        expect(document.activeElement).toBe(input);
+	});
 
 	it('zeigt initial Edit/Delete Buttons und aktuellen Namen', () => {
 		render(NameItemEditor, { props: baseProps });
@@ -87,7 +93,7 @@ describe('NameItemEditor - Funktionalität', () => {
 		expect(onSave).not.toHaveBeenCalled();
 	});
 
-	it('ruft onSave korrekt auf bei gültigem Namen: Tatort/Crime (ohne Vorgang)', async () => {
+	it('ruft onSave korrekt auf bei gültigem Namen: Tatort/Crime', async () => {
 		render(NameItemEditor, { props: baseProps });
 		await fireEvent.click(screen.getByTestId('edit-button'));
 
@@ -117,7 +123,7 @@ describe('NameItemEditor - Funktionalität', () => {
 		expect(screen.getByTestId('edit-button')).toBeInTheDocument();
 	});
 
-	it('triggert Save bei Enter-Taste: Tatort/Crime (ohne Vorgang)', async () => {
+	it('triggert Save bei Enter-Taste: Tatort/Crime', async () => {
 		render(NameItemEditor, { props: baseProps });
 		await fireEvent.click(screen.getByTestId('edit-button'));
 

tests/fixtures.ts

@@ -41,7 +41,7 @@ export const baseData = {
 	vorgang: testVorgangsList[0],
 	vorgangList: testVorgangsList,
 	crimesList: testCrimesList,
-	url: `https://example.com/list/${testVorgangsList[0].vorgangToken}`,
+	url: new URL(`https://example.com/list/${testVorgangsList[0].vorgangToken}`),
 	crimeNames: ['modell-A', 'Fall-A']
 };
 

tests/views/Anmeldung.test.ts

@@ -77,7 +77,39 @@ describe('Vorgang Anzeige via Token', () => {
 		// Cookie wird nicht gesetzt
 		expect(cookiesSet).not.toHaveBeenCalled();
 	});
-	it.todo('Überprüfe was passiert, wenn Eingabe falsch, bzw. nicht im System passend gefunden');
+
+	it('Falsche PIN', async () => {
+		// Mock formData
+		const vorgObj = baseData.vorgang;
+
+		const formData = new FormData();
+		formData.set('vorgang-token', vorgObj.vorgangToken);
+		formData.set('vorgang-pin', vorgObj.vorgangPIN);
+
+		const mockRequest = {
+			formData: vi.fn().mockResolvedValue(formData)
+		};
+
+		// PIN-Validierung nicht erfolgreich
+		vi.mocked(vorgangPINValidation).mockReturnValueOnce(false);
+
+		const cookiesSet = vi.fn();
+
+		const event = {
+			request: mockRequest,
+			cookies: {
+				set: cookiesSet
+			}
+		};
+
+		const result = await actions.default(event);
+		expect(result.status).toBe(400);
+		expect(result.data.message).toMatch(/Falsch/i);
+	});
+
+	// Nicht vorhandener Vorgang-Token nicht notwendig, da PIN-Check
+	// entsprechend fehlerhaft
+	it.skip('Nicht vorhandener Vorgang-Token', () => {});
 });
 
 describe('Teste Guard', () => {

tests/views/TatortList.test.ts

@@ -26,9 +26,29 @@ async function clickPlusButton() {
 }
 
 describe('Seite: Vorgangsansicht', () => {
-  test.todo('Share Link disabled wenn Liste leer');
+  test('Share Link disabled wenn Liste leer', () => {
+    const testData = { ...baseData, crimesList: [] };
+    render(TatortListPage, { props: { data: testData } });
+
+    const button = screen.getByRole('button', { name: /share link/i });
+    expect(button).toBeInTheDocument()
+    expect(button).toBeDisabled();
+  });
+
   describe('Szenario: Admin + Liste gefüllt - Funktionalität', () => {
-    test.todo('Share Link Link generierung richtig');
+    test('Share Link Link generierung richtig', () => {
+        const testData = { ...baseData};
+
+        render(TatortListPage, { props: { data: testData } });
+
+        const link = screen.getByRole('link', { name: /share link/i });
+        expect(link).toBeInTheDocument()
+
+        // const vorgangTokenFirstUUIDGroup = testData.vorgangList[0].vorgangToken.split('-')[0]
+        const vorgangURL = testData.url.toString()
+        const vorgangURLEncoded = encodeURIComponent(vorgangURL)
+        expect(link).toHaveAttribute('href', expect.stringContaining(vorgangURLEncoded));
+    });
 
     it('führt PUT-Request aus und aktualisiert UI nach onSave', async () => {
       const data = structuredClone(baseData);

tests/views/TatortList.view.test.ts

@@ -100,4 +100,16 @@ describe('Seite: Vorgangsansicht', () => {
 			expect(linkElement).toHaveAttribute('href', expectedURL);
 		});
 	});
+
+	describe('PIN Anzeige & Button', () => {
+		it('Teste korrekte Anzeige von PIN Komponente', () => {
+			const testData = { ...baseData};
+			render(TatortListPage, { props: { data: testData } });
+			const vorgObj = baseData.vorgangList[0]
+
+			// PIN is being displayed within ´NameItemEditor´
+			let label = screen.queryByText(vorgObj.vorgangPIN);
+			expect(label).toBeInTheDocument();
+		});
+	});
 });

tests/views/VorgangList.view.test.ts

@@ -171,4 +171,14 @@ describe('Hinzufügen Buton', () => {
 		const result = await actions.default(event);
 		expect(result).toEqual({ token: testVorgangToken });
 	});
+});
+
+describe('Vorgang-Operationen', () => {
+	it('Teste korrekte Anzeige von Vorgang-Input Komponente', () => {
+		const testData = { ...baseData};
+		const { getAllByTestId } = render(VorgangListPage, { props: { data: testData } });
+
+		let buttons = getAllByTestId('edit-button')
+		expect(buttons.length).toBeGreaterThan(1);
+	});
 });