Korting

Deze moet dan False teruggeven. Khad inderdaad wel wat moeite met een goeie naam te kiezen... en deze lijkt nogal afhankelijk van hoe ge het leest.
Het idee was: "Niet-BE Haefele Personality", dus een Haefele personality die niet HaefeleBE is. Ander idee is bijvoorbeeld "IsHaefelePersonalityMaarNietHaefeleBE" ofzo?

Nuja, het is volledig afhankelijk van hoe verschillende bedrijven die data doorsturen. Als er bvb een nieuwe dat exact zoals Haefele wilt doen dan moet die hier ook toegevoegd worden en dus moet de naam veranderd worden.

wat moet deze doen als bv SFS binnenkomt (of een andere personality die niets met Häfele te maken heeft?

Grammarnazicomment: invoerbestand is 1 woord.

klein detail, maar het kan zeker geen kwaad om single quotes in de output rond Bestandsnaam te voorzien. Maakt het duidelijker dat er iets had moeten staan indien BestandsNaam leeg is. In dit concreet geval maakt het niet zoveel uit, maar het is gewoon een goede gewoonte.

Het verschil met de test hieronder is dat je in dit geval de VerwerkCustomer maar 1 gaat oproepen, hieronder 2x. Je kan dat gedrag afdwingen door nog een .ExactAantalKeer(1) (of (2) natuurlijk) toe te voegen aan bovenstaande lijn. Als je dat niet doet, zijn de beide testen virtueel hetzelfde (behalve de maxnumberofcustomers) en weet je als lezer niet echt waar het grote verschil zit.

(idem voor de testen hieronder natuurlijk)

Test met 0 klanten niet vergeten. Het zou kunnen dat klant crappy data doorstuurt.