CacheAdminA_trunk2010/vhUnitTest/DOM/common/impl

VanHoeckeGebruikerBeheerder

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 09 Nov 16
Commits this week: 0
Total Lines of Code (LoC): 164
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
Friday 05 Apr 2019
1:06 pm

Tom Vermeulen

9:53 am

Tom Vermeulen

het tegenvoorbeeld is: zolang de test binnen 1 pagina past, is het niet altijd nodig om af te zonderen

het tegenvoorbeeld is: zolang de test binnen 1 pagina past, is het niet altijd nodig om af te zonderen

Thursday 04 Apr 2019
4:16 pm

Wim Vermeulen

4:11 pm

Wim Vermeulen

Stub, indien geen .Verifieer()

Stub, indien geen .Verifieer()

3:58 pm

Wim Vermeulen

idem opeten

idem opeten

3:56 pm

Wim Vermeulen

Mock moet stub zijn, maar ik zie dat deze 4 lijnen telkens opnieuw terugkeren, alvast in 3 van de 5 testmethods --> afzonderen in private method.

Mock moet stub zijn, maar ik zie dat deze 4 lijnen telkens opnieuw terugkeren, alvast in 3 van de 5 testmethods --> afzonderen in private method.

3:51 pm

Wim Vermeulen

i.p.v. error opeten : beter als innerexception toevoegen.

i.p.v. error opeten : beter als innerexception toevoegen.

3:49 pm

Wim Vermeulen

computernaam etc moet vermeld worden

computernaam etc moet vermeld worden

3:47 pm

Wim Vermeulen

LoginCode en Initialen toevoegen aan de foutmelding + Computernaam (en tijdstip) ook tonen.

LoginCode en Initialen toevoegen aan de foutmelding + Computernaam (en tijdstip) ook tonen.

3:47 pm

Wim Vermeulen

LoginCode en Initialen toevoegen aan de foutmelding

LoginCode en Initialen toevoegen aan de foutmelding

3:36 pm

Wim Vermeulen

detail : deze is een VanHoeckeGebruikerBeheerderStub i.p.v. een Mock. --> Martijn had een leuk ezelsbruggetje om deze uit mekaar te houden --> Check MVI http://subversion02.vanhoecke.be/static/ogdo...

detail : deze is een VanHoeckeGebruikerBeheerderStub i.p.v. een Mock.
--> Martijn had een leuk ezelsbruggetje om deze uit mekaar te houden --> Check MVI

3:28 pm

Wim Vermeulen

Opeet-code? Op z'n minst iets loggen? of andere actie.

Opeet-code?
Op z'n minst iets loggen? of andere actie.

3:20 pm

Wim Vermeulen

Ik zou eerst de .Bewaar() uitvoeren en dan checken op volgende HasNext()+exception. Anderszijds zou de UpdateAanmelding() een method moeten zijn op de service en niet thuishoren in de Validator (ma...

Ik zou eerst de .Bewaar() uitvoeren en dan checken op volgende HasNext()+exception.
Anderszijds zou de UpdateAanmelding() een method moeten zijn op de service en niet thuishoren in de Validator (maar dan vervalt wellicht mijn opmerking hierboven).

2:53 pm

Wim Vermeulen

Deze lijn bovenaan in de method zetten, dan moet je slecht 1x de ZoekViaInitialen() oproepen. Nu staat er 2x hetzelfde

Deze lijn bovenaan in de method zetten, dan moet je slecht 1x de ZoekViaInitialen() oproepen.
Nu staat er 2x hetzelfde

2:19 pm

Wim Vermeulen

Index definieren op LoginCode en op Initialen --> performantie bij ZoekViaFilter()

Index definieren op LoginCode en op Initialen --> performantie bij ZoekViaFilter()

Friday 22 Mar 2019
8:59 am

Tom Vermeulen

[UST4165] PA-UB-003: Service voorzien om gebruikers aan en af te melden
[UST4165] PA-UB-003: Service voorzien om gebruikers aan en af te melden
Friday 15 Dec 2017
5:02 pm

Wim Vermeulen

2:38 pm

Kurt Callebaut

  • More
  • CR-823
  • finished reviewing
9:28 am

Wim Vermeulen

[UW422] Vhisie4 MaakOrderVanOfferte faalt voor onze Nederlandse collega's
[UW422] Vhisie4 MaakOrderVanOfferte faalt voor onze Nederlandse collega's
8:51 am

Wim Vermeulen

8:45 am

Kurt Callebaut

  • More
  • CR-822
  • finished reviewing
8:43 am

Kurt Callebaut

[UW422] Vhisie4 MaakOrderVanOfferte faalt voor onze Nederlandse collega's
[UW422] Vhisie4 MaakOrderVanOfferte faalt voor onze Nederlandse collega's
Wednesday 22 Feb 2017
3:28 pm

Kurt Callebaut

Wednesday 15 Feb 2017
10:58 am

Tom Vermeulen

  • More
  • CR-65
  • finished reviewing
10:57 am

Tom Vermeulen

Zou je hier ook niet wat id's naar buiten schrijven in de foutmelding? Kwestie van later het forensisch onderzoek simpeler te maken?

Zou je hier ook niet wat id's naar buiten schrijven in de foutmelding? Kwestie van later het forensisch onderzoek simpeler te maken?

10:53 am

Tom Vermeulen

Moet deze ook uitgevoerd worden indien er een error geraised is? (geen idee, vraag het met gewoon af) (desnoods ook de testen aanpassen vaneigest)

Moet deze ook uitgevoerd worden indien er een error geraised is? (geen idee, vraag het met gewoon af) (desnoods ook de testen aanpassen vaneigest)

Tuesday 14 Feb 2017
8:08 am

Kurt Callebaut

  • More
  • CR-65
  • started review
[UST2038] Vh4_SVC: HerberekenMaatwerk
[UST2038] Vh4_SVC: HerberekenMaatwerk