DataMMATTAORCreator.cls.xml

De timings moeten niet mee tot in productie, lijkt me. Dus de TimerUtils-lijnen terug uit de code halen vooraleer te mergen naar Prod,
tenzij dit expliciet in de opdracht (jira) staat.

De timings moeten niet mee tot in productie, lijkt me. Dus de TimerUtils-lijnen terug uit de code halen vooraleer te mergen naar Prod,
tenzij dit expliciet in de opdracht (jira) staat.

Private methods onderaan plaatsen.
(zie zelfde opm bij andere UT)

Ik heb enkele bedenkingen, die we gemakkelijker mondeling kunnen bespreken.
Let's skype

Private methods onderaan plaatsen.

Alternatief is om er een OnBeforeOneTest() van te maken, maar dat zal hier niet lukken aangezien je data meegeeft (Kenm en variant)

Dus toch private en onderaan
En dan zou ik er een Function van maken i.p.v. "Do ..." van maken,
Bijvoorbeeld : GeefDataMMatCreator() met dezelfde args.

Hier zou ik de "StandaardLade" method oproepen en dan enkel de properties Kleur en Materiaal overschrijven.
Da's 4 lijnen code-duplicatie minder.

Anderszijds wou ik opmerken dat je de kleur-variaties beter in de UT zelf implementeert.
De Utils.MatKenmerken klasse is niet bedoeld om alle mogelijke variaties van kleuren, die we ooit ergens (in een UT) zouden nodig hebben, hier te gaan toevoegen.
1 kleur-variatie lijkt me nog wel te kunnen in deze klasse (is al schemerzone), maar 3 toch liever niet.

Ter illustratie : in deze klasse kan bijvoorbeeld wel : TaorIndelingMat , ExtraBredeMat, ... (die zijn nu uiteraard niet nodig)