Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:

- KortingService: Kortingen die via data dumps worden ingelezen hetzelfde gemaakt zoals Intracto het doet

- Unit test aangepast om dit gedrag weer te geven

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

- Enkele overbodige lijnen uit Unit Test gehaald

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

- KortingService: unit test uitgebreid met 'Tbx niet meegegeven'

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

- KortingService: Unit test uitgebreid

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

- KortingService uitgebreid zodat varianten die niet in data dump staan ook korting krijgen (afhankelijk van degenen die er wel in staan)

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

- KortingService: unit test aangepast t.g.v. eerste veralgemening

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

- KortingService: unit test opgeschoond (met oog op veralgemening)

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

- Unit tests toegevoegd voor KortingService Helper plus kleine change unit test KortingService

  1. … 3 more files in changeset.
[ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten

-Parse test uitgebreid en aanpassingen toegepast over hele test + review opmerkingen verwerkt

  1. … 2 more files in changeset.
[ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten

-testen van de exceptie uitwerken.

-Fake Kortingreader gerefactored

-SamV vult de andere testen aan.

  1. … 5 more files in changeset.
normaalgezien geen early quits in een method (tenzij helemaal bovenaan)

normaalgezien geen early quits in een method (tenzij helemaal bovenaan)

Grammarnazicomment: invoerbestand is 1 woord. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Grammarnazicomment: invoerbestand is 1 woord.

klein detail, maar het kan zeker geen kwaad om single quotes in de output rond Bestandsnaam te voorzien. Maakt het duidelijker dat er iets had moeten staan indien BestandsNaam leeg is. In dit concr...

klein detail, maar het kan zeker geen kwaad om single quotes in de output rond Bestandsnaam te voorzien. Maakt het duidelijker dat er iets had moeten staan indien BestandsNaam leeg is. In dit concreet geval maakt het niet zoveel uit, maar het is gewoon een goede gewoonte.

Het verschil met de test hieronder is dat je in dit geval de VerwerkCustomer maar 1 gaat oproepen, hieronder 2x. Je kan dat gedrag afdwingen door nog een .ExactAantalKeer(1) (of (2) natuurlijk) toe...

Het verschil met de test hieronder is dat je in dit geval de VerwerkCustomer maar 1 gaat oproepen, hieronder 2x. Je kan dat gedrag afdwingen door nog een .ExactAantalKeer(1) (of (2) natuurlijk) toe te voegen aan bovenstaande lijn. Als je dat niet doet, zijn de beide testen virtueel hetzelfde (behalve de maxnumberofcustomers) en weet je als lezer niet echt waar het grote verschil zit.

(idem voor de testen hieronder natuurlijk)

Test met 0 klanten niet vergeten. Het zou kunnen dat klant crappy data doorstuurt.

Test met 0 klanten niet vergeten. Het zou kunnen dat klant crappy data doorstuurt.

[ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten
[ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten
[ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten

-Parse test uitgebreid, VerwerkCustomer test aangemaakt, StelInkortingen test opgekuist

-Exception handlin in de while loop aangepast

  1. … 10 more files in changeset.