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
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.
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.

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....

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....

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

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

[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

- 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.

  1. … 4 more files in changeset.
[VRB509][UST3835] Webshop: Try-Catch voor IntractoMelder WebClient

- Refactor na review (om Logger te kunnen uitmocken) - Lastig geval!!! extra kanban-kaartje gemaakt (ICT397)

    • -0
    • +17
    ./LoggerMock.cls.xml
You got me there boy http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

You got me there boy

Logger uitmocken en injecteren, dan kan je expliciet een verwacht van de LogEnMail doen (wat in testomschrijving staat)

Logger uitmocken en injecteren, dan kan je expliciet een verwacht van de LogEnMail doen (wat in testomschrijving staat)

[VRB509] Webshop: Try-Catch voor IntractoMelder WebClient
[VRB509] Webshop: Try-Catch voor IntractoMelder WebClient
[VRB509][UST3835] Webshop: Try-Catch voor IntractoMelder WebClient

- UT voor IntractoMelder UpdateCartStatus()

  1. … 1 more file in changeset.