Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:

- Na review => Verduidelijking dat het allemaal gerelateerd is aan handelingen met ProductNr 1000

Eerst de comment hieronder lezen. Gelezen? Ok: een ExactAantalKeer(0) gecombineerd met een Verwacht heeft dan net weer eerder nood aan een $$$ElkeWaarde omdat ge in de meeste gevallen wilt dat de...

Eerst de comment hieronder lezen.

Gelezen?

Ok: een ExactAantalKeer(0) gecombineerd met een Verwacht heeft dan net weer eerder nood aan een $$$ElkeWaarde omdat ge in de meeste gevallen wilt dat de Verwachte method nooit of te nimmer, ongeacht de parameters wordt uitgevoerd.

Deze test blijft slagen als ge 1000 door 1111 vervangt (of gelijk welke parameter door gelijk welke waarde), wat ook niet volledig koosjer is

Eigenlijk geen $$$ElkeWaarde bij een VerwachtMethodCall. Ge 'verwacht' iets, dus ge weet wat er gaat gebeuren, dan kan je best mbv een ..Matches iets meegeven (en ja, ook bij de andere reeds bestaa...

Eigenlijk geen $$$ElkeWaarde bij een VerwachtMethodCall. Ge 'verwacht' iets, dus ge weet wat er gaat gebeuren, dan kan je best mbv een ..Matches iets meegeven (en ja, ook bij de andere reeds bestaande methods)

[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:...
[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:...
[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:

- Geen CheckStockTussenOrderLijnenEnStockAdmin i.g.v. IsSpaceTowerProduct

- Aanpassing en uitbreiding TestCase

  1. … 1 more file in changeset.
Na uitgebreid intern overleg... toch maar niet http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Na uitgebreid intern overleg... toch maar niet

[ICT-348] [rvTVE] VHIP903: Voorraadsync: Uitz spacetower in AxProductValidator NIET igv SyncAll:
[ICT-348] [rvTVE] VHIP903: Voorraadsync: Uitz spacetower in AxProductValidator NIET igv SyncAll:
idem

idem

zie opmerking in de test van IsGeldig. zelfde bij de IsOngewenstProductInMigratieFilter ook in een paar testjes hieronder

zie opmerking in de test van IsGeldig.
zelfde bij de IsOngewenstProductInMigratieFilter
ook in een paar testjes hieronder

Direct de ..ProductTypeApiStub gebruiken. IndienMethodCalls kunnen overschreven worden (Verwacht... niet). De #dim kan dan weg en de Set ..ProductTypeApi hieronder ook. Deze constructie staat op no...

Direct de ..ProductTypeApiStub gebruiken. IndienMethodCalls kunnen overschreven worden (Verwacht... niet). De #dim kan dan weg en de Set ..ProductTypeApi hieronder ook.
Deze constructie staat op nog een paar andere testjes ook

[ICT-348] [rvTVE] VHIP903: Voorraadsync: Uitz spacetower in AxProductValidator NIET igv SyncAll:

- Fix van TestCase => Gebruikte producten moeten nu bestaan wegens verandering in de IsGeldig op de AXimpl.common.AxProductValidator => gebruikt nu altijd IsLadeVanSpaceTowerProduct op ProductTypeApi => Die gebruikt GeefProduct op ProductApi => Exceptie indien Not BestaatProduct

    • -5
    • +11
    ./GenerateDelayedInventSyncs/Test.cls.xml
[UST3571R] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX

- Test kuist de data helemaal op en de eventverwerker kan daar niet zo goed tegen (probleem op dev workstation waar de code moet werken en ook de testen lopen).

    • -0
    • +1
    ./GenerateDelayedInventSyncs/Test.cls.xml
Nope. Dan geeft de HasNext() netjes 0, zoals verwacht => Global is leeg => Geen stockverschillen gevonden => Default mailtje zal verstuurd worden met melding hiervan (zonder een onbestaand lijstje ...

Nope. Dan geeft de HasNext() netjes 0, zoals verwacht => Global is leeg => Geen stockverschillen gevonden => Default mailtje zal verstuurd worden met melding hiervan (zonder een onbestaand lijstje mee te sturen).

I'll remember for next time http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

I'll remember for next time

  • More
  • CR-936
  • finished reviewing
[UST3571R] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX

- Opmerkingen verwerkt

    • -15
    • +0
    ./GenerateDelayedInventSyncs/Test.cls.xml
  1. … 1 more file in changeset.
ongeveer ja http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

ongeveer ja

Best duidelijk zijn. Neemt niet weg dat het gewoon Set DynIterator = ##class().... moeten zijn

Best duidelijk zijn. Neemt niet weg dat het gewoon Set DynIterator = ##class().... moeten zijn

  • More
  • CR-932
  • finished reviewing
nitpicking qua naamgeving: ik zelf zou een DataBouwer klasse een BouwHuppeldepup method geven. Als de method StelSamenDingesEnDinges is, kan je dat in een *SamenSteller steken. Moet dat niet verand...

nitpicking qua naamgeving: ik zelf zou een DataBouwer klasse een BouwHuppeldepup method geven. Als de method StelSamenDingesEnDinges is, kan je dat in een *SamenSteller steken.
Moet dat niet veranderen hoor, maar op veel andere plaatsen wordt dat ook gedaan: een *Bepaler heeft een Bepaal() enzo

juist een vraagske: moet er iets extra voorzien worden voor het geval de resultset leeg is?

juist een vraagske: moet er iets extra voorzien worden voor het geval de resultset leeg is?

Is dat niet dezelfde test als: "Test: Generate wordt NIET aangeroepen indien enkel items met verkeerde hoofdnode aanwezig"() ?

Is dat niet dezelfde test als: "Test: Generate wordt NIET aangeroepen indien enkel items met verkeerde hoofdnode aanwezig"() ?

Mag deze niet gedimd worden als TECH.Iterator? Maakt niet uit zeker, wegens toch maar een dimmeke?

Mag deze niet gedimd worden als TECH.Iterator? Maakt niet uit zeker, wegens toch maar een dimmeke?

Waarom hier IsGeldig en niet IsValid, zoals in alle filters? Dit dan nog de "Valid"-ator http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Waarom hier IsGeldig en niet IsValid, zoals in alle filters? Dit dan nog de "Valid"-ator