From a0b9f6dc801934a1bd00f35f35c7a91b596f2f75 Mon Sep 17 00:00:00 2001 From: Andrei Laas Date: Tue, 29 Sep 2026 19:24:58 +0200 Subject: [PATCH] Anker aus der laufenden Definition, und ein Test, der sich selbst durchwinkt Zwei Fehler aus dem ersten Lauf des Rauchtests, beide von mir. Der Anker fuer transfer_employee stammte aus 20260727120200 und fand nichts: die Funktion ist seither dynamisch gepatcht worden, die Belegungsabfrage ist zweizeilig und beruecksichtigt den Stichtag. Dieselbe Falle wie immer, nur diesmal nicht im Rumpf einer Funktion, sondern im Anker auf sie -- fuer hire_employee und rehire_employee stimmten die Anker, weil sie aus der zuletzt erzeugten Fassung kamen. Die Migration war nie angewendet (sie bricht als Ganzes ab), deshalb die Datei korrigiert statt eine neue geschrieben. Und der Rauchtest hat sich an einer Stelle selbst durchgewunken: das `raise` fuer den Fehlschlag stand innerhalb des Blocks mit exception-Zweig, wurde also vom eigenen Handler gefangen -- und weil seine Meldung das Wort "vorgemerkte" enthielt, bestand die Pruefung auf die erwartete Fehlermeldung. Der Schritt galt als bestanden, obwohl die Migration gar nicht angewendet war. Jetzt merkt sich der Block nur, ob es ging, und ausgewertet wird danach. --- .../20260929200000_vorgemerkte_planstelle.sql | 14 ++++++++++-- db/tests/rauchtest.sql | 22 ++++++++++++++----- 2 files changed, 28 insertions(+), 8 deletions(-) diff --git a/db/migrations/20260929200000_vorgemerkte_planstelle.sql b/db/migrations/20260929200000_vorgemerkte_planstelle.sql index f2680b2..05d72fe 100644 --- a/db/migrations/20260929200000_vorgemerkte_planstelle.sql +++ b/db/migrations/20260929200000_vorgemerkte_planstelle.sql @@ -115,10 +115,20 @@ end $migration$; -- ═══ transfer_employee ════════════════════════════════════════════ +-- +-- Der Anker steht auf der zweiten Zeile der Belegungsabfrage. Die erste +-- Fassung dieser Migration zielte auf +-- +-- where pa.position_id = v_target_position and pa.valid_to is null; +-- +-- also auf den Wortlaut aus 20260727120200 — und fand nichts. Die Funktion ist +-- seither dynamisch gepatcht worden: die Abfrage ist zweizeilig und +-- berücksichtigt den Stichtag. Dieselbe Falle wie immer, nur diesmal nicht im +-- Rumpf einer Funktion, sondern im Anker auf sie. do $migration$ declare - v_alt constant text := $anker$where pa.position_id = v_target_position and pa.valid_to is null;$anker$; - v_neu constant text := $anker$where pa.position_id = v_target_position and pa.valid_to is null; + v_alt constant text := $anker$and (pa.valid_to is null or pa.valid_to > v_effective);$anker$; + v_neu constant text := $anker$and (pa.valid_to is null or pa.valid_to > v_effective); declare v_vorgemerkt date := planstelle_vorgemerkt(v_target_position, v_effective, v_employee_id); diff --git a/db/tests/rauchtest.sql b/db/tests/rauchtest.sql index ddf39bd..12056ad 100644 --- a/db/tests/rauchtest.sql +++ b/db/tests/rauchtest.sql @@ -54,6 +54,7 @@ declare v_stelle uuid; v_offen int; v_meldung text; + v_ging boolean; begin select id into v_wurzel from org_units where parent_id is null; select id into v_ort from locations order by name limit 1; @@ -120,19 +121,28 @@ begin perform transfer_employee(jsonb_build_object( 'employee_id', v_a, 'target_position_id', v_p4, 'effective_date', (current_date + 10)::text)); + -- Der Fehlschlag wird **ausserhalb** des Blocks ausgewertet. Ein `raise` + -- innerhalb landet im eigenen exception-Zweig — und weil seine Meldung das + -- Wort „vorgemerkte" enthielt, bestand der Schritt beim ersten Lauf, obwohl + -- die Migration gar nicht angewendet war. Ein Test, der sich selbst + -- durchwinkt, ist schlimmer als keiner. + v_ging := false; begin perform hire_employee(jsonb_build_object( 'personnel_number', v_nr + 1, 'first_name', 'Rauch', 'last_name', 'Test-B', 'gender', 'm', 'birth_date', '1990-01-01', 'location_id', v_ort, 'position_id', v_p4, 'entry_date', current_date::text)); - raise exception 'Die vorgemerkte Planstelle liess sich besetzen.'; + v_ging := true; exception - when others then - v_meldung := SQLERRM; - if v_meldung not like '%vorgemerkt%' then - raise exception 'Erwartet wurde die Meldung zur Vormerkung, kam aber: %', v_meldung; - end if; + when others then v_meldung := SQLERRM; end; + + if v_ging then + raise exception 'Die vorgemerkte Planstelle liess sich besetzen — die Pruefung greift nicht.'; + end if; + if v_meldung not like '%bereits eine Versetzung vorgemerkt%' then + raise exception 'Erwartet wurde die Meldung zur Vormerkung, kam aber: %', v_meldung; + end if; raise notice '5/8 Vorgemerkte Planstelle wird abgewiesen'; -- Die eigene Vormerkung darf nicht im Weg stehen: dieselbe Versetzung noch