CacheAdminA_trunk2010/vhPowerTools/APPS

EC

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 8
Last Commit: 21 Mar 16
Commits this week: 0
Total Lines of Code (LoC): 1.517
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 01 Oct 2024
9:35 am

Peter Vercoutter

Tuesday 27 Aug 2024
4:21 pm

Jo Willems

Wednesday 11 Aug 2021
1:40 pm

Wim Vermeulen

Thursday 24 Jun 2021
8:16 am

Jo Willems

Wednesday 26 May 2021
3:50 pm

Wim Vermeulen

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

3:49 pm

Wim Vermeulen

3:49 pm

Wim Vermeulen

Monday 08 Feb 2021
11:22 am

Wim Vermeulen

[ICT] Powertool ExterneVerkoopPowerservice
[ICT] Powertool ExterneVerkoopPowerservice
Wednesday 01 Apr 2020
2:17 pm

Jo Willems

[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
Thursday 20 Feb 2020
12:15 pm

Jo Willems

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.

Tuesday 21 Jan 2020
2:13 pm

Peter Vercoutter

11:58 am

Peter Vercoutter

[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
Wednesday 15 Jan 2020
5:22 pm

Jo Willems

5:19 pm

Jo Willems

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

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

4:11 pm

Jo Willems

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

Tuesday 14 Jan 2020
3:02 pm

Peter Vercoutter

[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
Monday 07 Jan 2019
5:12 pm

Wim Vermeulen

10:54 am

Tom Vermeulen

Wednesday 19 Dec 2018
10:06 am

Wim Vermeulen

[UW687] VH4: problemen IntractoMelder bij Haefele ConfirmOrder
[UW687] VH4: problemen IntractoMelder bij Haefele ConfirmOrder
Thursday 19 Jul 2018
10:54 am

Martijn Vierstraete

Tuesday 17 Jul 2018
11:58 am

Wim Vermeulen

Tuesday 08 May 2018
6:09 pm

Wim Vermeulen

Uitleg zie in e-mail (vanwege syntax-highlighting) Concreet: In de UT doe je eerst ..ZetVerwachteExceptie() en daarna roep je ..AssertBewaarCorrecteKorting() op Binnen den Assert heb je dan uitera...

Uitleg zie in e-mail (vanwege syntax-highlighting)

Concreet:
In de UT doe je eerst ..ZetVerwachteExceptie() en daarna roep je ..AssertBewaarCorrecteKorting() op
Binnen den Assert heb je dan uiteraard ook de Verifieer.
Maar volgens mij (vreemd dat t spelleke ni crasht) kan je geen “Assert” en “Verwacht” en “Verifeer” doen als je een Error aan het testen bent.

en de rest staat in mijne mail ...

6:09 pm

Wim Vermeulen

Misschien niet voor nu meteen, maar eerder voor een volgende "Refactor-sessie" http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif dat we de KlantSessieData...

Misschien niet voor nu meteen, maar eerder voor een volgende "Refactor-sessie" dat we de KlantSessieDataBepalen en de RefSessieDataBepaler uitmockbaar maken, zodat in de UT geen encrypted DataOpt (en Ref) moet doorgegeven worden.
Dan staat daar gewoon in de dto.KlantSessieData ingevuld dto.Korting=0.25
Nu staat er naast de encrypted DataOpt hoeveel de waarde van de korting is.

Ik laat de keuze aan jouw, om ofwel aan te passen, ofwel te markeren, zodat een volgende persoon dat meeneemt. Deze comment in Crucible zal echter verloren gaan, dus is niet voldoende als reminder, hé.

Monday 07 May 2018
1:34 pm

Martijn Vierstraete

VH4: Kortingen ook voor TBX
VH4: Kortingen ook voor TBX
Friday 30 Mar 2018
2:21 pm

Wim Vermeulen

Thursday 15 Mar 2018
3:00 pm

Wim Vermeulen

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

2:49 pm

Wim Vermeulen

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

1:05 pm

Martijn Vierstraete

11:29 am

Martijn Vierstraete

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