CacheAdminA_trunk2010/vhUnitTest/WSimpl/Unishop/Login

KortingBewaarder

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 22 Mar 16
Commits this week: 0
Total Lines of Code (LoC): 504
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 11 Apr 2024
3:14 pm

Joannes Lemmens

2:16 pm

Jelle Baele

1:45 pm

Joannes Lemmens

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
1:31 pm

Joannes Lemmens

[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error

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

Options
    • -9
    • +0
    ./TestKisch.cls.xml
    • -9
    • +0
    ./TestVanRaemdonck.cls.xml
    • -9
    • +0
    ./TestDaro.cls.xml
8:31 am

Joannes Lemmens

[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error
[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error
Monday 24 Jul 2023
8:34 am

Joannes Lemmens

Thursday 20 Jul 2023
9:20 am

Peter Vercoutter

Wednesday 19 Jul 2023
2:34 pm

Joannes Lemmens

[ICT-3740] RVG: Data OPT extra parameter
[ICT-3740] RVG: Data OPT extra parameter
Friday 14 Jul 2023
1:11 pm

Joannes Lemmens

[ICT-3740] RVG: Data OPT extra parameter

- TestBase voor UT BewaarKortingMetVariant uitgebreid met Revego

Options
    • -0
    • +3
    ./TestBase.cls.xml
12:25 pm

Joannes Lemmens

[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

Options
  1. ./BewaarMeegegevenKorting
    • -0
    • +211
    ./BewaarMeegegevenKorting/Test.cls.xml
  3. … 2 more files in changeset.
Monday 18 Jul 2022
10:06 am

Joannes Lemmens

10:06 am

Joannes Lemmens

Thursday 07 Jul 2022
8:54 am

Joannes Lemmens

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

Wednesday 06 Jul 2022
2:58 pm

Tom Vermeulen

2:58 pm

Tom Vermeulen

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?

Tuesday 05 Jul 2022
10:30 am

Joannes Lemmens

[ICT-2976] MVX Kortingen Instellen
[ICT-2976] MVX Kortingen Instellen
Tuesday 28 Jun 2022
1:38 pm

Joannes Lemmens

[ICT-2976] MVX Kortingen Instellen

- TestBase vergeten mee te updaten

Options
    • -0
    • +3
    ./TestBase.cls.xml
Wednesday 18 May 2022
2:17 pm

Tom Vermeulen

Monday 09 May 2022
10:36 am

Joannes Lemmens

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

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

10:23 am

Joannes Lemmens

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

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

Tom Vermeulen

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?

Monday 04 Apr 2022
12:05 pm

Joannes Lemmens

[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
Wednesday 16 Mar 2022
9:59 am

Joannes Lemmens

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

- UnitTest aangepast zodat Orgalux verwacht wordt

Options
    • -0
    • +3
    ./TestBase.cls.xml
Tuesday 19 May 2020
7:54 am

LauraL

[ICT-1473] SFS kortingen meegeven via DataOpt - Mogelijkheid voorzien om korting mee te geven per variant

Bestaande unit testen aanpassen, indien foute input gooien ze andere errors

Options
    • -2
    • +2
    ./TestDaro.cls.xml
    • -2
    • +2
    ./TestVanRaemdonck.cls.xml
    • -2
    • +2
    ./TestKisch.cls.xml
Thursday 19 Jul 2018
10:54 am

Martijn Vierstraete

Tuesday 17 Jul 2018
11:58 am

Wim Vermeulen

Monday 16 Jul 2018
9:29 am

MartijnV

[UST3804] VH4: Kortingen ook voor TBX

- KlantSessieData- en RefSessieDataBepaler via constructor + aangepaste tests

Options
    • -16
    • +10
    ./TestHaefeleBe.cls.xml
    • -19
    • +82
    ./TestBase.cls.xml
    • -26
    • +12
    ./TestVanRaemdonck.cls.xml
    • -26
    • +12
    ./TestKisch.cls.xml
    • -26
    • +14
    ./TestDaro.cls.xml
  6. … 1 more file in changeset.
Thursday 12 Jul 2018
4:42 pm

MartijnV

[UST3804] VH4: Kortingen ook voor TBX

- reviewopmerkingen verwerkt

Options
    • -2
    • +2
    ./TestDaro.cls.xml
    • -0
    • +8
    ./TestBase.cls.xml
    • -2
    • +2
    ./TestKisch.cls.xml
Tuesday 08 May 2018
6:09 pm

Wim Vermeulen

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