Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)

- gelijk trekken met Cache05 code

- opkuis wlip

  1. … 1 more file in changeset.
[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)

- hercompilatie van Magazijn enu triggerde (niet gerelateerde) test om te falen.

[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)

- Reviewopmerking, opzet en afbraak testen verduidelijken.

    • -27
    • +29
    ./TestUglyPickingService.cls.xml
Idem voor de cleanup van de UTDataVoorKvik --> aparte method Hier zelfs geen parameters nodig, denk.

Idem voor de cleanup van de UTDataVoorKvik --> aparte method
Hier zelfs geen parameters nodig, denk.

Ik heb lang gezocht naar het verschil in setup tussen deze method en de method erboven. Door de setup (de lijnen hierboven) in een aparte private method te zetten wordt het onderscheid meteen duide...

Ik heb lang gezocht naar het verschil in setup tussen deze method en de method erboven.
Door de setup (de lijnen hierboven) in een aparte private method te zetten wordt het onderscheid meteen duidelijk:

bvb. Do ..SetUpUTDataVoorKvik ( PalletIdVoorEWMS, PalletIdVoorEWPAL )

En hierdoor worden de 2 tests meteen een pak overzichtelijker

MethodNaam volgens vh-conventies : ZetOpEWPAL() en KuisOpEWPAL() en ZetOpEWMSData()

MethodNaam volgens vh-conventies : ZetOpEWPAL() en KuisOpEWPAL() en ZetOpEWMSData()

[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)

- fixen van de test data setup zodat de testen overal kunnen draaien

[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)

- tijdelijke fix test wegens inconsistentie van de verschillende sqlserver EWMSdatabases.

- een goede data setup moet geschreven worden. aanzet al in code.

[ICT-986] MP HaefeleDE opzetten

-initiele commit.

  1. … 107 more files in changeset.
[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)

- gelijkhouden van code met cache05 (trunk repository)

  1. … 3 more files in changeset.
Als er palletten meegeven zijn, en we vinden niet wat we zoeken, dan stoppen we. Als er geen palletten meegegeven zijn moeten we verder. (fallback) de test op Count is dus essentieel. en dient niet...

Als er palletten meegeven zijn, en we vinden niet wat we zoeken, dan stoppen we.
Als er geen palletten meegegeven zijn moeten we verder. (fallback)
de test op Count is dus essentieel. en dient niet om de 0 te detecteren, maar om niet 0 te detecteren.

OPGELET: Tracht zoveel mogelijk de code van deze klasse tussen caché v5 en cache2010 in sync te houden. Net voor uw wijzigingen waren deze methods redelijk goed in sync (behalve method FPalletsPerP...

OPGELET:
Tracht zoveel mogelijk de code van deze klasse tussen caché v5 en cache2010 in sync te houden.
Net voor uw wijzigingen waren deze methods redelijk goed in sync (behalve method FPalletsPerProdNr() die is ge-refactored).
Na uw wijzigingen dreigt de code uiteen te lopen en wordt het steeds lastiger voor iemand anders om nog toe te voegen, of erger nog: om te troubleshooten!

Maak er desnoods een apart kaartje van. Of overleg met Pieter en/of met mij

Early quit :-O Gelukkig gaat ie binnenkort wel weg.

Early quit :-O
Gelukkig gaat ie binnenkort wel weg.

Als reviewer ben ik verplicht om dit aan te geven : embedded sql aanpassen : beter : wegwerken en vervangen door TECH.Resultset reden: gedrag is beter onder controle bij errors, early quits, onvoo...

Als reviewer ben ik verplicht om dit aan te geven :

embedded sql aanpassen :
beter : wegwerken en vervangen door TECH.Resultset
reden: gedrag is beter onder controle bij errors, early quits, onvoorspelbaar gedrag indien 2 sql's gemixed worden (lees op andere stacklevel een andere &sql doorlopen), compile-issues.

In dit specifiek geval zou ik misschien toch ook de &sql behouden.

Zeer goed dat je deze lijn erbij zet! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zeer goed dat je deze lijn erbij zet!

Count() is niet nodig, want als Count=0 , dan zal Find() ook niks vinden.

Count() is niet nodig, want als Count=0 , dan zal Find() ook niks vinden.

[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)
[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)
no_comment
[ICT235] - OnBefores..OnAfters #super fixen