ExterneVerkoopPowerService.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ik had deze initieel bij Kris (als reviewer) gezet, maar dat was wellicht geen goed idee :-P

Ik had deze initieel bij Kris (als reviewer) gezet, maar dat was wellicht geen goed idee :-P

[ICT] Powertool ExterneVerkoopPowerservice
[ICT] Powertool ExterneVerkoopPowerservice
[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
[UW687] VH4: problemen IntractoMelder bij Haefele ConfirmOrder
[UW687] VH4: problemen IntractoMelder bij Haefele ConfirmOrder
Tja, snap uw bedenking wel. Tis hier nogal een speciaal geval. Ik wil echt een aparte log van deze melding met de juiste "Groep" en "Actie". Eigenlijk is dat omdat ik onze logging niet handig vind,...

Tja, snap uw bedenking wel.
Tis hier nogal een speciaal geval. Ik wil echt een aparte log van deze melding met de juiste "Groep" en "Actie". Eigenlijk is dat omdat ik onze logging niet handig vind, vandaar :-P

Er is consensus dat de "open accolade" meteen achter de If of Else moet staan, dus op dezelfde lijn. Er is volgens mij geen consensus of de Else/ElseIf op dezelfde lijn of op de volgende lijn moet...

Er is consensus dat de "open accolade" meteen achter de If of Else moet staan, dus op dezelfde lijn.

Er is volgens mij geen consensus of de Else/ElseIf op dezelfde lijn of op de volgende lijn moet staan.
Mijn voorkeur is op de volgende lijn, maar ik heb er geen problemen mee dat anderen de Else op dezelfde lijn zetten, i.e. achter de "}".
Idem Try-Catch

Voor mij is dit voldoende, heb geen nood aan geschreven consensus hiervoor

Na kleine rondvraag: er is een zekere consensus binnen vhict over de schrijfwijze van if else en try catch structuren nl.      if () { ... } else { ... } try { ... } catch { ... }     ...

Na kleine rondvraag: er is een zekere consensus binnen vhict over de schrijfwijze van if else en try catch structuren nl.

    
if () {
   ...
} else {
   ...
}

try {
   ...
} catch {
   ...
}
           
manueel ipv mauneel http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

manueel ipv mauneel

Kun je in dit geval de try ... catch niet gewoon weghalen?

Kun je in dit geval de try ... catch niet gewoon weghalen?

[VRB460] EDI: SFS: Integratie vrijgave in handel-ERP: Error met HTTP POST: kunnen hernemen, en verwittigd...
[VRB460] EDI: SFS: Integratie vrijgave in handel-ERP: Error met HTTP POST: kunnen hernemen, en verwittigd...