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
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
ik weet het, maar dan is ie op zijn minst conform de rest voor het geval het weer ingeschakeld zou worden.

ik weet het, maar dan is ie op zijn minst conform de rest voor het geval het weer ingeschakeld zou worden.

Wijzigingen in deze test maken niet veel uit als deze uitgeschakeld is http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Wijzigingen in deze test maken niet veel uit als deze uitgeschakeld is

[UST4014] VH4 HFLN Referenties bij doorbestellingen
[UST4014] VH4 HFLN Referenties bij doorbestellingen
OnNew voorzien die location zet zodat in via een deploy de juiste server gezet kan worden. (Productie, test, accept, development) Set ..Location = ##class(TECH.Config.ConfigMgr).Instance().GetStri...

OnNew voorzien die location zet zodat in via een deploy de juiste server gezet kan worden. (Productie, test, accept, development)

Set ..Location = ##class(TECH.Config.ConfigMgr).Instance().GetString("xxxxxxxxxxxxx")

[UST3713][EPIC90]: ITR melden dat status order gewijzigd is
[UST3713][EPIC90]: ITR melden dat status order gewijzigd is
zie hierboven

zie hierboven

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebru...

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.

aangezien deploy wordt er geen aanpassing doorgevoerd.

aangezien deploy wordt er geen aanpassing doorgevoerd.

  • More
  • CR-227
  • finished reviewing
Wat met klanten zonder transportkeuze?

Wat met klanten zonder transportkeuze?

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

ExactAantalKeer(1) om intentie duidelijk te maken

ExactAantalKeer(1) om intentie duidelijk te maken

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen