Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
De lijn hieronder is een uitbreiding op deze en omvat deze case volledig + de A/B case.

De lijn hieronder is een uitbreiding op deze en omvat deze case volledig + de A/B case.

gaat dit nooit meer $$$True mogen zijn (geen idee hoor, just wondering)?

gaat dit nooit meer $$$True mogen zijn (geen idee hoor, just wondering)?

dé moment om te laten zien dat ik wel naar gekeken heb: 'Do' ipv 'do' (hieronder nog ergens)

dé moment om te laten zien dat ik wel naar gekeken heb: 'Do' ipv 'do' (hieronder nog ergens)

Hierin zit dus ook het verwijderen van alles wat met LPN-code te maken heet (aan de Caché-kant). De tests zouden hopelijk duidelijk moeten maken wat de PalletIDBepaler allemaal moet doen.

Hierin zit dus ook het verwijderen van alles wat met LPN-code te maken heet (aan de Caché-kant). De tests zouden hopelijk duidelijk moeten maken wat de PalletIDBepaler allemaal moet doen.

[ICT-4246] Manco: Backend ProScan
[ICT-4246] Manco: Backend ProScan
gaat deze zoek nog even snel gaan? een "=" gaat indexed zoeken, maar een "like" gaat mogelijks natural door de data. Het gaat hier wel om een tabel met enkele miljoenen records. Zeker dubbelchecken...

gaat deze zoek nog even snel gaan? een "=" gaat indexed zoeken, maar een "like" gaat mogelijks natural door de data. Het gaat hier wel om een tabel met enkele miljoenen records. Zeker dubbelchecken! (idem voor de andere queries natuurlijk)

Gisteren hier nog even met WV over gebabbeld en we zouden hier toch 'adviseren' om de grotere private methods in (een) aparte helperklasse(s) te steken, elk met hun eigen testen. Op die manier zijn...

Gisteren hier nog even met WV over gebabbeld en we zouden hier toch 'adviseren' om de grotere private methods in (een) aparte helperklasse(s) te steken, elk met hun eigen testen. Op die manier zijn de verschillende codepaden beter te testen (ipv sommige testen in de UT van deze klasse)
Dit valt onder het "Single Responsibility" principe. Ik ga dat hier niet allemaal verder uittypen, we praten of bellen wel

opeetcode!!! zeker aanpassen

opeetcode!!! zeker aanpassen

[ICT-4242] Manco: DnaCodeService uitbreiding voor A/B
[ICT-4242] Manco: DnaCodeService uitbreiding voor A/B
Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt. Ik dacht dat een OnBeforeOne...

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt.
Ik dacht dat een OnBeforeOneTest bedoeld is voor als er een gevaar is dat hij door eerdere UTs zou kunnen bevuild worden, wat hier niet het geval is.

Bwa, is een beetje zo gegroeid door refactoren... Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests? Daarna de afweging: Waarom dan nog de call on...

Bwa, is een beetje zo gegroeid door refactoren...
Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests?
Daarna de afweging: Waarom dan nog de call onder test speciaal gaan afzonderen? En bovendien: Waarom beschikken we anders over een $$$AssertTrue en $$$AssertFalse?
Het was bovendien daardoor ook niet meer nodig om een betekenisvolle, extra lokale variabele te voorzien in de UnitTesten (VerwachtInOpstartfase), dus ook al een regel minder per UT, zonder aan leesbaarheid in te boeten (eerder het tegendeel).
Doe de nieuwe versie eens open in studio en zie eens hoe leesbaar die is
Misschien moeten we dan eerder (mettertijd, als die klassen eens onder change komen) die van LBX en TBX aanpassen naar dit model?

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval. Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (pro...

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval.
Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (productieSequentie) toe te voegen.
Na al je inspanningen, keur ik deze oplossing goed

Ik begrijp niet echt waarom je deze "Assert..." method hebt weggewerkt. Dit is immers een algemeen gebruikt principe, o.a. om aan te geven dat de test-methods in deze klasse dezelfde logica oproepe...

Ik begrijp niet echt waarom je deze "Assert..." method hebt weggewerkt.
Dit is immers een algemeen gebruikt principe, o.a. om aan te geven dat de test-methods in deze klasse dezelfde logica oproepen. Hoe eenvoudig deze ook is.

En voor zover consistentie een argument i : de implementatie bij TAX is nu minder consistent met die van LBX en TBX.

Ik vind dit geen drama, hoor. Als je wilt kunnen we hierover van gedachten wisselen

Kleine suggestie : OpstartfaseBepaler voor iedere test(method) newen, dat is ietsje properder --> dus in de OnBeforeOneTest() is beter. Let op: parameter (aTestCase) !

Kleine suggestie :
OpstartfaseBepaler voor iedere test(method) newen, dat is ietsje properder --> dus in de OnBeforeOneTest() is beter.
Let op: parameter (aTestCase) !

ineens een Set van maken, lijn wordt korter en leesbaarder

ineens een Set van maken, lijn wordt korter en leesbaarder

[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
  • More
  • CR-195
  • finished reviewing
[VRB417] Geen bundeling productiedetailbladen TBX over toeleveringlijnen heen
[VRB417] Geen bundeling productiedetailbladen TBX over toeleveringlijnen heen