TestmaakttoeleveringenGEENreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct

Resultaat hangt af van leverancieer, is dus essentieel.

Het feit dat er één aanpassing is en dat er 5 testen moeten aangepast worden dit moet toch één of andere bellentje doen rinkelen dat de structuur toch niet zo 'bien' is -> niet voor nu op te lossen , maar te aanschouwen als een verbeterpuntje

Geen goed idee om dit iedere keer te herhalen -> beter één maal in basis test , die het standaard gedrag test , maar voor de rest een stub gebruiken , met eventueel een dummy product repo -> aandacht gaat teveel naar zaken die er niet toe doen in de testen.

!!! Testen moeten duidelijk zijn -> werken is maar een bijzaak , indien ze niet werken moeten ze makkelijk aan te passen zijn -> dus hoe duidelijker hoe beter;

Van een MaakReceptieVoorToelevering zou ik eerder een toelevering object / toeleveringid verwachten en geen differse parameters -> misschien is MaakReceptie voldoende

GeefProductIdVoorLeverancierId lijkt mij toch wel een rare naamgeving ; misschien is een GeefProductId al voldoende

Lijkt mij beter om de productlijn in een variabele te steken en deze hier te gebruiken ipv ProductLijnen.GetAt(1) -> verhoogt de leesbaar -> begrijpbaarheid

StandaardToelevering hoofding met flexibele in te stellen data lijkt mij niet echt standaard te zijn -> lijkt mij eerder iets in om in .Fake. te zetten

Opletten met negatieve resultaten in functies ; beter IsReceptieNodig -> dubbele ontkenningen etc..