Activity CacheAdminA_trunk2010/vhUnitTest/WSimpl/Unishop/Login/KortingVoorVariantBepaler

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

- -10
- +10
./Test.cls.xml
- History
- Source
- Diff
./Test.cls.xml
… 3 more files in changeset.

10:06 am

Joannes Lemmens

More
Re: CR-2420
replied to Tom Vermeulen

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

10:23 am

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