enu

Zelfde opmerkingen vorige method

GeefDoosLade() beter in een variabele steken, want nu wordt die 7x herhaald/berekend in één testmethod.

F-lade --> dat mag je gerust vermelden in de methodnaam;
bvb. "Test: Hoge ondiepe niet-binnenlade verpakt in ... " of "Test: niet-binnenlade F-hoogte verpakt in ... "

Hier in de UT zou ik de eerder "VerwachteAfmetingK.." noemen i.p.v. DefaultAfm...
Je gaat deze immers asserten.

Ik denk zelfs dat "K1=100" geen default waarde is, maar effectief een berekende waarden.

Beter: GeefIngegevenKenmerken() kan je direct aan MaatwerkProduct vragen.

Ter info :
1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls
2) Testklasse bevat meestal ook de naam van de public method die je aan 't testen bent. Maar misschien werd de naam dan te lang.
Je hoeft dit hier niet te veranderen, maar is taak van reviewer om dit op te merken

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk()
Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan dubbel? --> dubbel onderhoud.
Misschien kan je hier (BOG) de implementatie van LosCompInBulkk laten oproepen.