Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3740] RVG: Data OPT extra parameter
[ICT-3740] RVG: Data OPT extra parameter
[ICT-3740] RVG: Data OPT extra parameter

- TestBase voor UT BewaarKortingMetVariant uitgebreid met Revego

Misschien best wel, ja: deze worden enkel gebruikt voor kortingen met DataOpt (die kijkt naar de toegelaten ladetypes). Door ze erbij te zetten weet de dataopt dat OL een ding is, maar hoeft er nog...

Misschien best wel, ja: deze worden enkel gebruikt voor kortingen met DataOpt (die kijkt naar de toegelaten ladetypes). Door ze erbij te zetten weet de dataopt dat OL een ding is, maar hoeft er nog niet mee om te gaan. Er is ook al iets in productie dat deze OL gewoon kan negeren indien nodig.
Plus, op CacheAccept staat het al zoals het hier is (openstaande wijziging, mag ik niet vergeten als ik het merge/commit!)

hier juist de bedenking of het klopt dat al die "OL" lijnen hier al mee moeten of niet?

hier juist de bedenking of het klopt dat al die "OL" lijnen hier al mee moeten of niet?

[ICT-2976] MVX Kortingen Instellen
[ICT-2976] MVX Kortingen Instellen
[ICT-2976] MVX Kortingen Instellen

- TestBase vergeten mee te updaten

Unit test toegevoegd die gedrag voor niet-Haefele personality test.

Unit test toegevoegd die gedrag voor niet-Haefele personality test.

Deze moet dan False teruggeven. Khad inderdaad wel wat moeite met een goeie naam te kiezen... en deze lijkt nogal afhankelijk van hoe ge het leest. Het idee was: "Niet-BE Haefele Personality", dus ...

Deze moet dan False teruggeven. Khad inderdaad wel wat moeite met een goeie naam te kiezen... en deze lijkt nogal afhankelijk van hoe ge het leest.
Het idee was: "Niet-BE Haefele Personality", dus een Haefele personality die niet HaefeleBE is. Ander idee is bijvoorbeeld "IsHaefelePersonalityMaarNietHaefeleBE" ofzo?

Nuja, het is volledig afhankelijk van hoe verschillende bedrijven die data doorsturen. Als er bvb een nieuwe dat exact zoals Haefele wilt doen dan moet die hier ook toegevoegd worden en dus moet de naam veranderd worden.

zie ook opmerking in kortingservicehelper zelf: ge gaat hier moeten duidelijk maken (met een of meerdere extra testjes) wat het gedrag gaat zijn als er met een willekeurige personality deze method ...

zie ook opmerking in kortingservicehelper zelf: ge gaat hier moeten duidelijk maken (met een of meerdere extra testjes) wat het gedrag gaat zijn als er met een willekeurige personality deze method wordt opgeroepen (leeg, error, defaultgedrag, ....)

wat moet deze doen als bv SFS binnenkomt (of een andere personality die niets met Häfele te maken heeft?

wat moet deze doen als bv SFS binnenkomt (of een andere personality die niets met Häfele te maken heeft?

[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:

- UnitTest aangepast zodat Orgalux verwacht wordt

[UST3804] VH4: Kortingen ook voor TBX

- KlantSessieData- en RefSessieDataBepaler via constructor + aangepaste tests

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

- reviewopmerkingen verwerkt

  1. … 2 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.