BuildBufferViaDnaCode

Goed opgemerkt, Jelle.
Tom zal wel meteen zien dat de .ProductNummer binnen de Tech.Equals() moet staan.

@ iedereen (vooral de jonkies) :
Bovenal is de vraag : is er geen UT die dit aan het licht brengt?
Zal zelf al deels antwoorden :
Deze klasse BL.MB.UP.Receptie.cls bevat stokoude implementatie, en er is nauwelijks sprake van "scheiding van dependencies".
Dit laatste maakt het dan ook zeer moeilijk om UT hiervoor te schrijven.
@Tom : misschien toch een systeemke bedenken om de Dna-complexiteit af te zonderen in een helper-klasse, zodat die alsnog kan ge-UT-ed worden :
bvb. DnaCodeSoortBepaler .IsVoorKad() .IsVoorVhoss(), .IsVoorSpaceStep()
( *) woordkeuze "...Soort..." kan vast beter!

Geen Asserts na een "VerwachteExceptie".
Deze worden toch niet uitgevoerd, kunnen daarom zelfs foute waarden tonen en niet falen. Dus sowieso de asserts hier weghalen.

@iedereen : deze opmerking altijd maken als reviewer, wanneer iemand hiertegen zondigt. En als developer hier best zelf proberen op te letten :-P

Waarvoor wordt DnaCode4 gebruikt?

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

#dim IngegevenKenmerken As DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken = ##class(vhTest.Dummy.DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken).DummyLade()

Productnummer wordt vergeleken met boolean?

Toevoegen dat de method een 'boolean' teruggeeft

Door het toevoegen van deze method is er wel wat duplicated code ontstaan om een dna code te maken. Ik laat de keuze aan jou of je een soort van dnaCodeInvullen method maakt of niet. Voor mij kan dit er nog mee door omdat het maar 2 methods zijn uiteindelijk

had ik bewust niet gedaan omdat al de rest van deze klasse procedureblock=0 is en ik geen extra can of worms wou openen

Wat als de tweede toelevering wel reeds volledig gescand is? Kan het zijn dat deze dan nog in de buffer zit, en getoond wordt met aantalGescand : 1/1 ?
Of is de toelevering dan weg, als in "niet meer vermeld in het Antwoord-object ?
--> geen idee of het zinvol is om deze situatie als TestMethod toe te voegen.

VH-naming conventies : ook in Delphi ?
--> RondAfKaderdeurDag

Olaahh ...
dit is een randgeval, kwestie van de code in UI of in de Backend te schrijven.

"ZoakGebr.... " is dat een sint-niklaases dialect?
Bij de WSimpl heb ik tevens een opmerking gemaakt over de naam van deze method. Zie WSimpl.

Deze naam is eerder verwarrend, vind ik : "Zoek..." en je geeft geen zoek-criteria op, er wordt dus ook niks gefilterd (lees: gezocht)
Wat dacht je van "GeefAlleAangemeldeGebruikers" of zoiets?
Al dan niet met gevolgen voor de klasse WS.Halux.AAP.GebruikerService en het WSDL-gedoe.

Eens effe overleggen?
.

Ik zie het nu pas, deze method is not procedureblock (want inherited from class).
Maar alle nieuwe methods kan je toch beter als

[ProcedureBlock = 1]

implementeren.
Hopelijk geeft dat geen problemen met ..AddToBuffer() enzo.