CacheAdminA_trunk2010/vhTest/Dummy

APPS

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 6
Last Commit: 15 Jul 15
Commits this week: 0
Total Lines of Code (LoC): 556
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
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
Thursday 05 Oct 2017
9:44 am

Tom Vermeulen

Wednesday 04 Oct 2017
2:36 pm

Jo Claes

  • More
  • CR-535
  • finished reviewing
Thursday 21 Sep 2017
8:49 am

Tom Vermeulen

Ja, want dat staat ook in de standaardDecrypteerRequest

Ja, want dat staat ook in de standaardDecrypteerRequest

Wednesday 20 Sep 2017
9:35 am

Tom Vermeulen

Het zou natuurlijk ook kunnen zijn dat ik, bij het overzetten van al die verschillende methods/properties/..., deze per ongeluk ben overgeschoten...

Het zou natuurlijk ook kunnen zijn dat ik, bij het overzetten van al die verschillende methods/properties/..., deze per ongeluk ben overgeschoten...

Tuesday 19 Sep 2017
3:44 pm

Jo Claes

'k weet niet of deze voorwaarde effectief wel klopt ; lijkt het mij niet beter te checken of het feit dat er moet gecontroleerd worden voor deze hostname ? 'k vermoed dat we naar een systeem moete...

'k weet niet of deze voorwaarde effectief wel klopt ; lijkt het mij niet beter te checken of het feit dat er moet gecontroleerd worden voor deze hostname ?

'k vermoed dat we naar een systeem moeten gaan om hostnames te koppelen en personalities en ervoor zorgen dat die hostnames per systeem eventueel anders kunnen ingesteld worden.

3:42 pm

Jo Claes

'k zou zeggen spel wat met de waarden die ja als dummy doorgeeft en zie welke rare effecten uw testen opeens zullen hebben. Als het dan echt dummies zijn zou ik er effectief ook dummy klassen van m...

'k zou zeggen spel wat met de waarden die ja als dummy doorgeeft en zie welke rare effecten uw testen opeens zullen hebben.
Als het dan echt dummies zijn zou ik er effectief ook dummy klassen van maken zodat ze niet de aandacht trekken in uw code en de nadruk ligt op hetgeen je wil benadrukken.

3:39 pm

Jo Claes

Hier zou ik geen dummy string gebruiken want je controleert er effectief op

Hier zou ik geen dummy string gebruiken want je controleert er effectief op

3:38 pm

Jo Claes

Wel raar dat hier opeens een stub staat ?? Is het zo belangrijk dat er "example.com" wordt teruggegeven

Wel raar dat hier opeens een stub staat ?? Is het zo belangrijk dat er "example.com" wordt teruggegeven

3:37 pm

Jo Claes

Niet echt nodig om response te hebben -> het is perfect mogelijk om gewoon de registreer uit te voeren aangezien je het toch niet kan controleren -> kwestie van het geen aandacht te geven

Niet echt nodig om response te hebben -> het is perfect mogelijk om gewoon de registreer uit te voeren aangezien je het toch niet kan controleren -> kwestie van het geen aandacht te geven

3:28 pm

Jo Claes

Als deze foutmelding naar de client gaat en zo lijkt het er op dan is deze verkeerd want de client weet niet dat wij intern in ons systeem de id prefixen om onderscheid te kunnen maken in de handel...

Als deze foutmelding naar de client gaat en zo lijkt het er op dan is deze verkeerd want de client weet niet dat wij intern in ons systeem de id prefixen om onderscheid te kunnen maken in de handelklantdata.

3:25 pm

Jo Claes

?? GeefID verwijderen maar GeefPrefix laten we staan ?? Hoe kan je dit verklaren ?

?? GeefID verwijderen maar GeefPrefix laten we staan ?? Hoe kan je dit verklaren ?

3:23 pm

Jo Claes

Reponse lijkt mij toch eerder een datastructuur dus al het moet genewed worden in de constructor is er iets mis. Ga ik nu echt moeten verplichten dat alle methods 2x moeten uitgevoerd worden in de...

Reponse lijkt mij toch eerder een datastructuur dus al het moet genewed worden in de constructor is er iets mis.

Ga ik nu echt moeten verplichten dat alle methods 2x moeten uitgevoerd worden in de testen en er gecontroleerd wordt dat het resultaat niet dezelfde instantie is dan die van de vorige call

Friday 15 Sep 2017
3:06 pm

Jo Claes

Dummies injecten

Dummies injecten

3:05 pm

Jo Claes

Dummy Decrypteerder meegeven - dat die test niet opeens begint te falen omdat een bepaalde decryptie niet meer geldig is

Dummy Decrypteerder meegeven - dat die test niet opeens begint te falen omdat een bepaalde decryptie niet meer geldig is

3:04 pm

Jo Claes

Dummy Registreerder meegeven

Dummy Registreerder meegeven

Thursday 14 Sep 2017
11:39 am

Tom Vermeulen

alleen maar wat code verplaatst, voor de rest niets

alleen maar wat code verplaatst, voor de rest niets

11:38 am

Tom Vermeulen

[UST2849] Vh4 Hein: HandelAuthenticator opkuisen
[UST2849] Vh4 Hein: HandelAuthenticator opkuisen
Tuesday 22 Aug 2017
8:37 am

Tom Vermeulen

Wednesday 28 Jun 2017
4:57 pm

Jo Claes

  • More
  • CR-282
  • finished reviewing
Thursday 22 Jun 2017
3:24 pm

Jo Claes

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aa...

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aan uw been

3:22 pm

Jo Claes

Factory method hoort eigenlijk niet thuis op een service klasse

Factory method hoort eigenlijk niet thuis op een service klasse

3:21 pm

Jo Claes

Opkuisen

Opkuisen

3:21 pm

Jo Claes

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

3:17 pm

Jo Claes

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce. Zodoende we maar één converter klasse hebben ...

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce.

Zodoende we maar één converter klasse hebben die converteert in beide richtingen.

3:07 pm

Tom Vermeulen

Ik heb dit kaartje al afgesloten, maar ik weet wel dat er nog een hoop opkuiswerk volgt, plus nog wat extra functionaliteit (maar die staat in volgende kaartjes) Ook net opgevallen dat ik nog niet...

Ik heb dit kaartje al afgesloten, maar ik weet wel dat er nog een hoop opkuiswerk volgt, plus nog wat extra functionaliteit (maar die staat in volgende kaartjes)

Ook net opgevallen dat ik nog niet met Override-objectje werk.