Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ik zou hier precies ook nog een test schrijven met waarde true voor IsSalesMedewerker. Kwestie van zeker te zijn dat de waarde niet verloren gaat na encrypt/decrypt.

Ik zou hier precies ook nog een test schrijven met waarde true voor IsSalesMedewerker.
Kwestie van zeker te zijn dat de waarde niet verloren gaat na encrypt/decrypt.

[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele
[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele
[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele

- Fake + Mock ook gecommit

- Unit tests uitgebreid

  1. … 4 more files in changeset.
Mijn eerste idee was om $Number(Korting) te gebruiken, maar dit werkt niet omdat kortingen "?" en "" ook toegelaten moeten zijn. Daarom heb ik heb ervoor gekozen om de logica lichtjes te veranderen...

Mijn eerste idee was om $Number(Korting) te gebruiken, maar dit werkt niet omdat kortingen "?" en "" ook toegelaten moeten zijn.
Daarom heb ik heb ervoor gekozen om de logica lichtjes te veranderen, dit leek ook de meest cleane oplossing te zijn.

In plaats van waardes 0 en 1-100 toe te laten, laten we nu waardes 0-100 toe (dus ook een halve procent korting bijvoorbeeld). Dit omdat:

  • Niemand ooit een fractie van een procent korting gaat geven
  • Dit volgens Dries ook gewoon zo hoort te zijn
[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error

- Unit tests mee aangepast sinds waarden tussen 0% en 1% wel toegelaten zijn nu

    • -9
    • +0
    ./Login/KortingBewaarder/TestKisch.cls.xml
    • -9
    • +0
    ./Login/KortingBewaarder/TestDaro.cls.xml
[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error
[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error
[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

    • -0
    • +3
    ./Login/KortingBewaarder/TestBase.cls.xml
[ICT-3740] RVG: Data OPT extra parameter

- Unit test KortingVoorVariantBepaler aangepast (sinds error verhuisd is)

- Unit test HandelsKlantKortingsPercentageBepaler uitgebreid met RVG

- Unit test voor KortingBewaarder.BewaarMeegegevenKorting aangemaakt

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

    • -0
    • +3
    ./Login/KortingBewaarder/TestBase.cls.xml
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-2829] Haefele NO multipersonality:

- Overige bestanden gewijzigd (zoals in kaartje ICT-2223)

  1. … 65 more files in changeset.
[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:

- UnitTest aangepast zodat Orgalux verwacht wordt

    • -0
    • +3
    ./Login/KortingBewaarder/TestBase.cls.xml