Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3804] VH4: Kortingen ook voor TBX

- KlantSessieData- en RefSessieDataBepaler via constructor + aangepaste tests

  1. … 5 more files in changeset.
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 ...

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

VH4: Kortingen ook voor TBX
VH4: Kortingen ook voor TBX
VH4: Kortingen ook voor TBX
  1. … 5 more files in changeset.
VH4: Kortingen ook voor TBX

- tests

  1. … 14 more files in changeset.
Aha; ik had hem bewust niet in het rijtje van Verwerk-calls gezet met échte verwerking, maar bij de VerwerkVanHoecke, die ook géén echte verwerking heeft. Wat is onze conventie hiervoor? Eerst alle...

Aha; ik had hem bewust niet in het rijtje van Verwerk-calls gezet met échte verwerking, maar bij de VerwerkVanHoecke, die ook géén echte verwerking heeft.
Wat is onze conventie hiervoor? Eerst alle echte Visitor-Verwerk-methods, en dan alle meer interne methods?

  • More
  • CR-14
  • finished reviewing
Niet dat het zou belangrijk is maar om toch een goed voorbeeld te stellen zou ik de verwerk methods toch een beetje bundelen en niet Verwerk .. nog wat anders en daarna nog wat Verwerk

Niet dat het zou belangrijk is maar om toch een goed voorbeeld te stellen zou ik de verwerk methods toch een beetje bundelen en niet Verwerk .. nog wat anders en daarna nog wat Verwerk

  • More
  • CR-14
  • started review
[DEF570] Kortingen voor Haefele BE werden niet toegepast voor LBX en TAX.
[DEF570] Kortingen voor Haefele BE werden niet toegepast voor LBX en TAX.
[DEF570] Kortingen voor Haefele BE werden niet toegepast voor LBX en TAX.
    • -0
    • +59
    ./TestHaefeleBe.cls.xml
  1. … 1 more file in changeset.