This is a list of all comments for CR-2083.

Review Summary: No summary


----------------------------------------
File: APPS/VKP/Klantspecifiek/Haefele/KortingParser.cls.xml
Revision Comment by Tom Vermeulen on 22 June 2020, 12:58 
http://subversion02:8060/cru/CR-2083#c3453

  Grammarnazicomment: invoerbestand is 1 woord. :)

Revision Comment by Tom Vermeulen on 22 June 2020, 12:57 defect
http://subversion02:8060/cru/CR-2083#c3452

  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.

----------------------------------------
File: vhUnitTest/APPS/VKP/Klantspecifiek/Haefele/KortingParser/Parse/Test.cls.xml
Revision Comment by Tom Vermeulen on 22 June 2020, 12:48 defect
http://subversion02:8060/cru/CR-2083#c3450

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

Revision Comment by Tom Vermeulen on 22 June 2020, 12:50 defect
http://subversion02:8060/cru/CR-2083#c3451

  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)

----------------------------------------
File: vhUnitTest/APPS/VKP/Korting/impl/KortingService/GeefKortingConfiguratieLijst/Test.cls.xml
Revision Comment by Tom Vermeulen on 22 June 2020, 13:25 defect
http://subversion02:8060/cru/CR-2083#c3454

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


---

ID: CR-2083 http://subversion02:8060/cru/CR-2083

Title: [ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten
Statement of Objectives:


State: Closed

Summary: 

Author: Sam Van Hoey
Moderator: Sam Van Hoey
Reviewers: (0 active, 1 completed*)
     Tom Vermeulen   (*)