Joint degree functionaliteit

[mdemare]

In het hoger onderwijs bestaan er joint degrees: opleidingen die door meer dan één onderwijsintelling verzorgd worden.
In RIO is de eigenaar van de OpleidingsEenheid dan de “penvoerende” instelling. Maar de andere deelnemende instellingen mogen daar dan ook AangebodenOpleidingen voor aanleveren.
DUO vertelde me er het volgende over:
Wanneer een instelling een aangeboden opleiding aanlevert voor een opleidingseenheid die niet van "hem" is, dan bepalen we of de opleidingseenheid (HoOpleiding) een opleidingssamenwerking betreft (Joint Degree).Wanneer dat het geval is bepalen we obv de licenties die gekoppeld zijn aan die HO-opleidingserkenning welke instellingen de opleiding zouden mogen verzorgen. Staat hun instelling in het lijstje voorkomt keuren we de aanlevering goed. Wanneer dat niet zo is ontvangt de instelling een foutcode "N09461 Het onderwijsbestuur maakt geen deel uit van de vastgestelde opleidingssamenwerking".

Het doel is dat onze mapper deze functionaliteit gaat ondersteunen. Na wat uitzoekwerk en navragen kom ik tot het volgende ontwerp:
We voegen twee nieuwe velden aan de RIO consumer voor een Program toe:
jointProgram: een boolean. Als die true is, gaan we kijken naar het tweede veld en nemen we een “nieuwe afslag” in de verwerking van het program. Als jointProgram, false is of het hele veld ontbreekt, dan volgen we de oude logica.
educationUnitCode: een string die moet voldoen aan de regex ^\d{4}O\d{4}$ , namelijk een OpleidingsEenheidCode. Als dit veld niet matcht met de regex of ontbreekt terwijl jointProgram true is, moeten we een foutmelding teruggeven.
als jointProgram niet true is word een educationUnitCode niet verwerkt (helemaal genegeerd)
Als een instelling een upsert job aanlevert voor een program en deze beide nieuwe velden zijn correct ingevuld, gaan we het program aanleveren als een AangebodenHoOpleiding, horende bij de OpleidingsEenheid zoals die in educationUnitCode staat.
We slaan het ophalen van de EducationSpecification helemaal over.
Wellicht moet de validatie van Programs aangepast worden: het veld educationSpecification is niet verplicht, als het veld jointProgram true is.

Opmerkingen:
We kunnen ervan uitgaan dat een Program wat aangeleverd wordt als jointProgram altijd een AangebodenHoOpleiding is. In RIO bestaan er geen andere vormen die “joint” kunnen zijn.

[remvee]

Ziet er goed uit maar toch een paar opmerkingen.

[remvee]

Ik denk dat het duidelijker wordt als je van hier 1 deftest maakt met testing blokken erin en misschien wat van de let duplicatie eruit haalt. Op dit moment kost het me wat moeite om te zien wat de verschillende tussen deze tests zijn omdat er een hoop ge-copy-paste is.

[remvee]

Goed idee om deze wrappers te verplaatsen als ze alleen in tests gebruikt worden! Ben wel nieuwsgierig geworden hoe deze functionaliteit in de applicatie aan elkaar geknoopt wordt. Weten we zeker dat het dan nog werkt? Kunnen we bijvoorbeeld een e2e test hiervoor toevoegen?

[remvee]

Misschien moet we voor :always gaan ipv true en deze onderaan zetten. Net als dat we bij cond een :else hebben.

[remvee]

:always?