Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Opgelet: wanneer de While() over meerdere items loopt, dan geldt het Result van de laatste. Wellicht is de veronderstelling dat de ZoekViaKlantId() slechts één item oplevert. Dan toch beter geen Wh...

Opgelet: wanneer de While() over meerdere items loopt, dan geldt het Result van de laatste.
Wellicht is de veronderstelling dat de ZoekViaKlantId() slechts één item oplevert.
Dan toch beter geen While() uitvoeren, maar enkel een If HasNext() then ...
Ultiem kan je nog controleren of een nadien nog een HasNext() is en dan een Warning loggen of een error smijten.

Good one! Merci voor de hint!!

Good one! Merci voor de hint!!

Dat leek mij wat overkill, omdat de relevante change slechts op 1 plek in 1 klasse zit. Ik dacht dat een ProjectSetting gebruikt werd als je met vele changes in vele klassen zit, die tegelijkertijd...

Dat leek mij wat overkill, omdat de relevante change slechts op 1 plek in 1 klasse zit. Ik dacht dat een ProjectSetting gebruikt werd als je met vele changes in vele klassen zit, die tegelijkertijd aan/af moeten kunnen gezet worden.

Is voor het stukje hieronder eigenlijk de ProjectSettingsApi niet bruikbaar?

Is voor het stukje hieronder eigenlijk de ProjectSettingsApi niet bruikbaar?

klein detail (wrschl niet de moeite om aan te passen): In bovenstaand geval is het soms interessanter/properder om het volgende te doen: If $$$AssertEquals(..OpmerkingenLijst.Count(),2) {{ ... de ...

klein detail (wrschl niet de moeite om aan te passen): In bovenstaand geval is het soms interessanter/properder om het volgende te doen:

If $$$AssertEquals(..OpmerkingenLijst.Count(),2) {{
... de andere asserts
}} (dubbele accolades omdat die Jira anders denkt dat het een macro is)

Stel dat het aantal 1 is, krijg je maar 1 test-errormelding, anders gaat de .GetAt(2) ook nog eens een melding geven die je misschien op het verkeerde been zet bij het proberen fixen

[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:
[ICT-1591] PA-ub-125: wegwerppallet op basis van klantinstellingen
[ICT-1591] PA-ub-125: wegwerppallet op basis van klantinstellingen
Telegramstijl, omdat ze anders te lang worden en de variaties nu eenmaal soms ingewikkeld zijnhttp://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Telegramstijl, omdat ze anders te lang worden en de variaties nu eenmaal soms ingewikkeld zijn

Idem reply hierboven.

Idem reply hierboven.

Zoals besproken... Dit ging over de property-namen, maar... toch liever niet. In de properties kunnen soms mocks, soms stubs gestoken worden, afhankelijk van wat de UnitTest nodig heeft.

Zoals besproken... Dit ging over de property-namen, maar... toch liever niet. In de properties kunnen soms mocks, soms stubs gestoken worden, afhankelijk van wat de UnitTest nodig heeft.

Done!

Done!

Done

Done

Done! (smartass http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif )

Done! (smartass )

Done!

Done!

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done en done en done! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Done en done en done!

Normaal binnen de scope blijven, ik weet het, maar ik dacht de while op deze manier zo proper en leesbaar mogelijk te houden. Uiteindelijk wordt elk van die dim'ekes toch in die while gebruikt en o...

Normaal binnen de scope blijven, ik weet het, maar ik dacht de while op deze manier zo proper en leesbaar mogelijk te houden. Uiteindelijk wordt elk van die dim'ekes toch in die while gebruikt en omhelst die while de volledige method.

Denk het niet. Nu krijgt ge alle OrderLijnIterators netjes op een rijtje via code completion. Eentje is dan de volledige, sommige zijn "via" dit, andere "via" dat, ...

Denk het niet. Nu krijgt ge alle OrderLijnIterators netjes op een rijtje via code completion. Eentje is dan de volledige, sommige zijn "via" dit, andere "via" dat, ...

Had ik al aangepast inmiddels, zit al in volgende review http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Had ik al aangepast inmiddels, zit al in volgende review

test namen zeggen niet duidelijk wat er getest wordt.

test namen zeggen niet duidelijk wat er getest wordt.

(Is bestaande code natuurlijk) hoeft geen lokale var te zijn. mag rechtstreeks op ..ProductService(Mock)

(Is bestaande code natuurlijk) hoeft geen lokale var te zijn. mag rechtstreeks op ..ProductService(Mock)

Refact nodig zodat namen van bestaande code ook ProductServiceMock worden, etc..

Refact nodig zodat namen van bestaande code ook ProductServiceMock worden, etc..

in deze situatie gebruiken we veelal if $$asserttrue(.. hasnext){ assert.. next() } omdat de .next hiervan afhankelijk is. ..

in deze situatie gebruiken we veelal
if $$asserttrue(.. hasnext)

Unknown macro: { assert.. next() }


omdat de .next hiervan afhankelijk is. ..