Activity CacheAdminA_trunk2010/DOM/PM/Product

1:58 pm

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronde...

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronder verifiëren?

8:01 am

Tom Vermeulen

More
Re: CR-1891
commented on review

overbodige comment?

2:35 pm

Jo Willems

More
CR-1891
started review

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten

9:42 am

Tommy Hebb

More
CR-1872
Closed

3:08 pm

Kurt Callebaut

More
CR-1872
finished reviewing

1:20 pm

Wim Vermeulen

More
Re: CR-1872
commented on review

Deze method voor TAX is volledig identiek http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/warning.gif aan die erboven voor LBX, op één woordje na : ..GeefProductU...

Deze method voor TAX is volledig identiek aan die erboven voor LBX, op één woordje na : ..GeefProductUpdater("TAX", ...
En ondertussen mogelijk ook nog eens t zelfde voor TBX (niet in deze review)

Zoals reeds eerder opgemerkt : Maak in de UT duidelijk waar de variatie zit, dus niet door de volledige impl te kopieren en één waarde te veranderen.
Oplossing hier is heel simpel : maak private method AssertProductUpdaterVoorLadeVariant (LadeVariant As ...enu)
en dan heb je 2 (of 3 indien TBX ook) testmethods die 1 lijntje code bevat met "LBX" of "TAX" of "TBX"