Activity CacheAdminA_trunk2010/vhUnitTest/WSimpl/Vhisie4/Auth

8:34 am

Joannes Lemmens

More
CR-2623
started review

[ICT-3740] RVG: Data OPT extra parameter

12:25 pm

Joannes Lemmens

More
71279
changed 4 files
root: CR-2623

[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

- -2
- +6
./HandelAuthenticator/Decrypteerder/HandelsKlantKortingsPercentageBepaler/Bepaal/Test.cls.xml
- History
- Source
- Diff
./HandelAuthenticator/.../Test.cls.xml
… 3 more files in changeset.

10:06 am

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!)

2:58 pm

Tom Vermeulen

More
CR-2460
finished reviewing

2:58 pm

Tom Vermeulen

More
Re: CR-2460
commented on review

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

10:30 am

Joannes Lemmens

More
CR-2460
started review

[ICT-2976] MVX Kortingen Instellen

8:46 am

Joannes Lemmens

More
68230
changed 3 files
root: CR-2460

[ICT-2976] MVX Kortingen Instellen

- Als kortingen op de webshop worden ingegeven, komen ze langs dit pad. Merivobox erbij gezet zodat deze ook wordt opgeslagen

- -0
- +2
./HandelAuthenticator/ContactGegevensRegistreerder/Registreer/Test.cls.xml
- History
- Source
- Diff
./HandelAuthenticator/.../Test.cls.xml
… 2 more files in changeset.

1:10 pm

Joannes Lemmens

More
68174
changed 5 files
root: CR-2460

[ICT-2976] MVX Kortingen Instellen

- MVX toegevoegd aan KortingBewaarder

- Decrypteerder uitgebreid om korting MVX terug te geven

- Unit tests uitgebreid om korting MVX te testen

- -1
- +5
./HandelAuthenticator/Decrypteerder/HandelsKlantKortingsPercentageBepaler/Bepaal/Test.cls.xml
- History
- Source
- Diff
./HandelAuthenticator/.../Test.cls.xml
… 4 more files in changeset.

2:17 pm

Tom Vermeulen

More
CR-2420
finished reviewing

10:36 am

Joannes Lemmens

More
Re: CR-2420
replied to Tom Vermeulen

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

10:23 am

Joannes Lemmens

More
Re: CR-2420
replied to Tom Vermeulen

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.

9:11 am

Tom Vermeulen

More
Re: CR-2420
commented on review

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, ....)

9:09 am