WS.HR.BrandLijst.BrandLijstService.txt

Er is nog een csp pagina met een query die verwijst naar de global die je op gaat kuisen. csp.HR.Medewerkers.cls
En er is nog wel wat code die naar die global verwijst.
Een 'search in files' gewoon naar 'protime' brengt nog wel wat code aan het licht.
Ik denk dat er ook nog wat onderzoek zal moeten gebeuren of deze overblijvende code al dan niet nog gebruikt wordt. er wordt nog gerefereerd naar protime nummers. van waar komen die dan?
Dat zal verder bepalen of er mss reeds uitgevoerde deletes moeten teruggedraaid worden of niet.

Is uw probleem natuurlijk, ge moet lezen wat er staat...

Bij AssertEquals() is het derde argument enkel zinvol als je daar een uitleg wil schrijven.

Met Asserts zoals hierboven lijkt het al gauw op

 	$$$AssertEquals(StreamSize, "") 

Dat staat er natuurlijk niet, maar mijn dislectisch brein interpretteert dit wel zo :-\
Ook met de andere Asserts in deze klasse heb ik moeite om snel te begrijpen wat er getest wordt.

Wil je de lege Comments in de Asserts (alle in de hele klasse) verwijderen?

Uw voorganger heeft niet proper gedaan
Vandaar mijn vraag of je hier het Boyscout principe wil toepassen? Merci Jo.

Deze assert (van uw voorganger) kan VEEL specifieker en veiliger :

	Do $$$AssertEquals(Stream.Size, StreamSize + (9*25 ))   ; i.e. "*" komt 9x per lijn voor, er zijn 25 lijnen.  

Wil je deze aanpassen? Thx.

Beter :
Replace naar "Line ... AA" of naar "Line ... 00" , want "Line ... 99" bestaat reeds vooraleer iets ge-replaced wordt --> werkt wel, maar is een beetje misleidend.
(wellicht ook in de volgende testmethods).