CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Method IsLiRe() mag wel blijven bestaan in de HalffabItemUtils-klasse (want is algemeen bruikbaar) Daarentegen method ZijnAfdekkappenInbegrepen() is een specifieke method voor LBX-Zijkanten halffab...

Method IsLiRe() mag wel blijven bestaan in de HalffabItemUtils-klasse (want is algemeen bruikbaar)
Daarentegen method ZijnAfdekkappenInbegrepen() is een specifieke method voor LBX-Zijkanten halffab --> dus deze method hoort niet thuis is deze Utils-klasse.

Conclusie:
method IsLiRe() terug herstellen met juiste impl. en de nieuwe method verplaatsen naar klasse HalffabItemVoorV1Maker

Ik zou de mock durven newen (en meegeven in de constructor want die voorziet dat al) ipv te mock-instancen. Hoe minder testen rekenen op een mock op een context, hoe beter

Ik zou de mock durven newen (en meegeven in de constructor want die voorziet dat al) ipv te mock-instancen. Hoe minder testen rekenen op een mock op een context, hoe beter

#dimmen én type definiëren

#dimmen én type definiëren

Ik zou geneigd zijn om een enumeratie aan te maken

Ik zou geneigd zijn om een enumeratie aan te maken

[BUS722] PM: LBX: zijkanten zonder afdekkappen:
[BUS722] PM: LBX: zijkanten zonder afdekkappen:
Het aanmaken van de SSL-configuratie zelf die hier gebruikt wordt, moet manueel op elke omgeving gebeuren. Daar ga je niets van terugvinden in de commits/review

Het aanmaken van de SSL-configuratie zelf die hier gebruikt wordt, moet manueel op elke omgeving gebeuren. Daar ga je niets van terugvinden in de commits/review

[BUS720]MP: SFS: andere server: impact op "Integratie webshop-vrijgave in handel-ERP"
[BUS720]MP: SFS: andere server: impact op "Integratie webshop-vrijgave in handel-ERP"
idem: Casing?

idem: Casing?

idem: Casing?

idem: Casing?

Casing? RunTaxTesten() ik vind niks anders om Review-opmerkingen te geven :-D

Casing?
RunTaxTesten()

ik vind niks anders om Review-opmerkingen te geven :-D

Ik zou de parameter niet linken aan LadeType. Wat denk je van "Sleutel" of "Keyword"? Bvb. een toekomstige case zou kunnen zijn If Sleutel = "Vhisie4" Do ..RunVhisie4Testen()

Ik zou de parameter niet linken aan LadeType.
Wat denk je van "Sleutel" of "Keyword"?

Bvb. een toekomstige case zou kunnen zijn

If Sleutel = "Vhisie4" Do ..RunVhisie4Testen()
ja, maar code vliegt er over twee weken toch weer uit

ja, maar code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
Eén instantie maken van legacypartij lijkt me voldoende.

Eén instantie maken van legacypartij lijkt me voldoende.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvr...

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvragen van klantsessiedata.

Duplicate testcode op klantid na ?

Duplicate testcode op klantid na ?