Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT - 610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob

- refactoring van testen

  1. … 2 more files in changeset.
In alle GeefOrderServiceInternal een BackGroundCartStatusUpdaterMock meegeven, anders wordt effectief een achtergrond process opgestart. Met indienMethodcalls().DoeNiets() processing verder laten g...

In alle GeefOrderServiceInternal een BackGroundCartStatusUpdaterMock meegeven, anders wordt effectief een achtergrond process opgestart.
Met indienMethodcalls().DoeNiets() processing verder laten gaan, indien er niets van wat in achtergrond uitgevoerd wordt moet gecontroleerd worden.

andere test method ook waar dit nog niet is.
geef een Mock mee en zie wat er aan fouten uit komt tijdens het uitvoeren van de testen.

[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob

- Bestaande testen voor ConfirmOrder herwerkt omdat UpdateCartStatus nu gedefinieerd staat als een backgroundjob onder een andere klasse (namelijk WSimpl.EC.Unishop.OrderConfirmation.Service.BackgroundCartStatusUpdater)

  1. … 13 more files in changeset.
[DEF694] Vh4: BestelWinkelkar: VHIP504: BestelOpmerking 8 cijfers; HFL NL bestelling toch tegengehouden...
[DEF694] Vh4: BestelWinkelkar: VHIP504: BestelOpmerking 8 cijfers; HFL NL bestelling toch tegengehouden...
Deze assert moet $$$False zijn (zie opm in de Validator-klasse zelf http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

Deze assert moet $$$False zijn (zie opm in de Validator-klasse zelf )

Bovenaan plaatsen aub. Of misschien is deze zelfs niet nodig???

Bovenaan plaatsen aub.
Of misschien is deze zelfs niet nodig???

De refactor van deze UT-klasse krijg je van mij cadeau http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

De refactor van deze UT-klasse krijg je van mij cadeau

OnBefore en OnAfter...xTest() steeds bovenaan plaatsen, conventie zoals den %OnNew()

OnBefore en OnAfter...xTest()
steeds bovenaan plaatsen, conventie zoals den %OnNew()

In geval van Haefele BE zal nog steeds hier passeren, en zo zal voor BE alsnog de AutomatischeVerwerking getriggered worden. Oepsie http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wik...

In geval van Haefele BE zal nog steeds hier passeren, en zo zal voor BE alsnog de AutomatischeVerwerking getriggered worden.
Oepsie

[DEF694] Vh4: BestelWinkelkar: VHIP504: BestelOpmerking 8 cijfers; HFL NL bestelling toch tegengehouden

- Automatisch order maken en sluiten inbouwen in OrderConfirmatie Service.cls.xml

  1. … 8 more files in changeset.
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject

- (nieuwe) OrderConfirmation Service : UT opmerking toegevoegd, bij gebrek aan tijd :-P

  1. … 1 more file in changeset.
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject

- (nieuwe) OrderConfirmation Service : UT's aangepast

[UW687] VH4: problemen IntractoMelder bij Haefele ConfirmOrder

- IntractoMelder alleen oproepen indien Vhisie4 : UT aangepast

  1. … 1 more file in changeset.
Ik zou de bepaling van de leverdatum afzonderen, dit zou de testsetup veel korter en duidelijker maken.

Ik zou de bepaling van de leverdatum afzonderen, dit zou de testsetup veel korter en duidelijker maken.

En Haefele NL? Zie bijlage kaartje To be:Voor HFL NL en BE: datum van eindklant op winkelkar zetten, na confirmatie

En Haefele NL?

Zie bijlage kaartje
To be:Voor HFL NL en BE: datum van eindklant op winkelkar zetten, na confirmatie

Wat is een geldig formaat? Zou je dit kunnen verduidelijken in de naamgeving (bv. HeeftGeldigEuropeesFormaat)? (of als het echt niet anders kan via commentaar) Bestaat in er in de codebase nog geen...

Wat is een geldig formaat? Zou je dit kunnen verduidelijken in de naamgeving (bv. HeeftGeldigEuropeesFormaat)? (of als het echt niet anders kan via commentaar)
Bestaat in er in de codebase nog geen dergelijke controle (bv. DateTimeApi)? Dit lijkt me generieke code.

FormatApi wordt niet gebruikt (bovenstaande lijn + property verwijderen)

FormatApi wordt niet gebruikt (bovenstaande lijn + property verwijderen)

Naamgeving: Api ipv API

Naamgeving: Api ipv API

Bij introductie van nieuwe API's in een klasse mag je deze newen ipv van de context te halen

Bij introductie van nieuwe API's in een klasse mag je deze newen ipv van de context te halen