Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
geen idee. ook voor 3 letterige afkorting ?

geen idee. ook voor 3 letterige afkorting ?

ofcourse

ofcourse

DateTimeApi gebruiken

DateTimeApi gebruiken

zou in principe toch PascalCase moeten zijn? Dus gewoon wat er stond

zou in principe toch PascalCase moeten zijn? Dus gewoon wat er stond

kan je hier dan niet beter ineens de nieuwe functie op de DateTimeApi gebruiken?

kan je hier dan niet beter ineens de nieuwe functie op de DateTimeApi gebruiken?

Is het zinvol om hier een paar kleine testjes toe te voegen die laten zien wat het gedrag van deze functie is?

Is het zinvol om hier een paar kleine testjes toe te voegen die laten zien wat het gedrag van deze functie is?

ICT-1199: Stap 3 : Dueout
ICT-1199: Stap 3 : Dueout
Dat is review CR-1616 geworden, voor zover ik kan oprakelen.

Dat is review CR-1616 geworden, voor zover ik kan oprakelen.

Kopiëren naar list is gedaan omdat de opkuis zelf ook een resultset nodig heeft. En caché leek niet altijd goed overweg te kunnen met meerdere actieve resultsets (MARS).

Kopiëren naar list is gedaan omdat de opkuis zelf ook een resultset nodig heeft. En caché leek niet altijd goed overweg te kunnen met meerdere actieve resultsets (MARS).

Ik ben het eigenlijk niet eens met de default-waarde True.

Ik ben het eigenlijk niet eens met de default-waarde True.

[UW732] Refactor HttpPost code voor MP's
[UW732] Refactor HttpPost code voor MP's
is het nodig om settransactionmode(1) te zetten na de delete actie ? (als het al iets uitmaakt. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

is het nodig om settransactionmode(1) te zetten na de delete actie ? (als het al iets uitmaakt.

Zou Deletelogging() ook in review moeten ? ik denk dat deze gewijzigd is. Als't goed is enkel nog delete voor logging en niet meer voor getalindex ?

Zou Deletelogging() ook in review moeten ? ik denk dat deze gewijzigd is. Als't goed is enkel nog delete voor logging en niet meer voor getalindex ?

Nederlands en Engelse taal door elkaar. Is het kopiëren van resultset naar list nog nodig als bulkaantal 100 is ?

Nederlands en Engelse taal door elkaar.
Is het kopiëren van resultset naar list nog nodig als bulkaantal 100 is ?

onAfterAllTests sowieso de aangemaakte logs verwijderen?

onAfterAllTests sowieso de aangemaakte logs verwijderen?