Activity CacheAdminA_trunk2010/vhUnitTest/WSimpl/EC/Unishop/OrderConfirmation/Service/IntractoMelder

9:35 am

Jo Willems

More
CR-1930
started review

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

12:15 pm

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.

2:13 pm

Peter Vercoutter

More
CR-1920
Closed

11:58 am

Peter Vercoutter

More
CR-1930
started review

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

9:20 am

Peter Vercoutter

More
59354
changed 14 files
root: CR-1930

[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
- +1
./UpdateCartStatus/Test.cls.xml
- History
- Source
- Diff
./UpdateCartStatus/Test.cls.xml
… 13 more files in changeset.

5:22 pm

Jo Willems

More
CR-1920
finished reviewing

5:19 pm

Jo Willems

More
Re: CR-1920
commented on review

Injecteren, en op zijn minst in de bestaande testen van deze klasse een fake gebruiken zodat deze geen background job starten die fout mails genereert. nog beter : nog een test schrijven zodat get...

Injecteren, en op zijn minst in de bestaande testen van deze klasse een fake gebruiken zodat deze geen background job starten die fout mails genereert.

nog beter : nog een test schrijven zodat getest wordt dat de BackgroundProcessStarter met de juiste argumenten aangeroepen wordt.

5:14 pm

Jo Willems

More
Re: CR-1920
commented on review

Er is op zijn minst nog één test waar een indienmethodcal op een stub/mock staat. Die faalt natuurlijk niet, maar staat daar niets meer te doen....

4:11 pm

Jo Willems

More
Re: CR-1920
commented on review

zet in één keer het volledige command in de foutmelding zodat een copy/paste gedaan kan worden

3:02 pm

Peter Vercoutter

More
CR-1920
started review

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

1:29 pm

Peter Vercoutter

More
59264
changed 5 files
root: CR-1920 CR-1930

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

- UpdateCartStatus() definiêren als een backgroundjob

- ErrorHandling aangepast zodat deze in UpdateCartStatus() zelf gebeurd. De LogEnMail werd weggehaald. De exceptie zal de cartId en status noteren en de UpdateCartStatus dient dan via de powertool 'IntractoMelderBestelStatusOpnieuwDoorgeven' opnieuw uitgevoerd te worden.