UGLYPicking

Block van Toelevering etc. afzonderen in private method.
Of eventueel meteen method AssertLijnReferentie( ... ) van maken. Geen idee of dit properder zal zijn.

Block van OrderLijnen/OrderMock afzonderen in private method

Refactor UT aub.
Bij Tommy geef ik soms wel de opmerking dat de meerdere testmethods gelijke blokken code bevatten, waardoor het overzicht van de setup verloren gaat, en dus veel minder duidelijk is waar de variatiepunten in de tests zitten.
... dus moet ik dat ook bij u doen.

ProductInOrder is %Boolean --> $$$False i.p.v. ""

opgepast met haakjes.
Deze komt in de gevarenzone, vind ik.
Extra haakjes en spaties lijkt me niet overbodig. Tevens ook voor de leesbaarheid.

Zelfde opmerking :
ConfiguratorService naar constructor.

Zelfde opmerking :
LijnReferentieBepaler naar constructor.

Naar constructor verplaatsen.
zeker aangezien deze binnen de Loop Lijnen zit.

Opgelet : "Probox" is toevallig dezelfde enumwaarde.
Zou beter zijn om een VerpakkingNaarDragerConverter te maken (voor TBX+LBX+TAX).

Bij ontbreken van een converter : eigenlijk op z'n minst dit vermelden in een lijntje commentaar.
Of toch maar ineens ne converter bijmaken? Wat vind jij?

Opgelet met ProcedureBlock en aanroepen van Routines waarin de variabelen niet genewed zijn !

Alternatief : code verplaatsen naar een (class)method en de gebruikers best mee aanpassen (obviously )

Gewoonte van maken om de $Name() functie te gebruiken :

 " ... '"_$Name(^UPRCP("B",bfrRGID,bfrVolgNr))_"' ..." 

dit is veel gemakkelijker, veel juister, en automatisch het juiste aantal qoutes.
--> UT zal dit aantonen