event

De specs waren hier mss niet duidelijk genoeg, maar per productlijn genereer je hier identieke lijnen (wat vroeger niet het geval was vanwege de inventsync die uniek is)
Al de informatie die je in het bericht moet steken kan je bepalen adhv bon, je hebt de lijnen niet nodig.
Je moet dus eigenlijk maar één lijn per bon voor zien.
AX kan er tegen, maar ze geen x keer de zelfde bon markeren dat geboekt mag worden.
Dit gaat de opzet van de test mocks data etc ook wat eenvoudiger maken

Algemene opmerking code conventions volgorde van elementen van een class
1 parameters
2 properties
3 Constructor (%OnNew)
4 Public methods
5 Private methods

voor Testcases onbefore* en onafter* methods voor de eigenlijke test methods

zie alg opmerking codeconventions

Meegeven van GeneratePickingCompleetResult kan nu nog.
Voor de volgende fase, het uitgesteld raisen van de event heb je dat resultaat niet meer...
Test op pickingcompleetresultaat dus in deze method steken.
Of anders refactoren bij implementatie 2e fase

zie alg opmerking coding conventions

Waarom zou je hier een spatie zetten? een spatie is niet 'niet ingevuld'
Idem voor qty