CacheAdminA_trunk2010/vhUnitTest/AXimpl/Dynamics/InventoryOut

SyncAll

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 14 Aug 17
Commits this week: 0
Total Lines of Code (LoC): 569
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 25 Sep 2019
2:21 pm

Tommy Hebb

1:29 pm

Tom Vermeulen

1:29 pm

Tom Vermeulen

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

1:25 pm

Tom Vermeulen

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)

11:29 am

Tommy Hebb

[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:...
[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:...
Monday 23 Sep 2019
3:27 pm

Tommy Hebb

3:26 pm

Tommy Hebb

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

3:26 pm

Tommy Hebb

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

Tom Vermeulen

11:35 am

Tom Vermeulen

idem

idem

11:33 am

Tom Vermeulen

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

11:30 am

Tom Vermeulen

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

Monday 22 Jan 2018
2:41 pm

Tom Vermeulen

Friday 19 Jan 2018
4:31 pm

Tommy Hebb

4:31 pm

Tommy Hebb

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).

4:23 pm

Tommy Hebb

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

4:21 pm

Tommy Hebb

  • More
  • CR-936
  • finished reviewing
3:22 pm

Tom Vermeulen

ongeveer ja http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

ongeveer ja

3:10 pm

Tom Vermeulen

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

2:51 pm

Tom Vermeulen

  • More
  • CR-932
  • finished reviewing
2:51 pm

Tom Vermeulen

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

2:49 pm

Tom Vermeulen

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?

10:13 am

Tommy Hebb

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"() ?

9:29 am

Tommy Hebb

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?

9:22 am

Tommy Hebb

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

Thursday 18 Jan 2018
2:16 pm

Tom Vermeulen

[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX
[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX
Wednesday 17 Jan 2018
5:00 pm

Tommy Hebb

[UST3572] VHIP maatwerksync:
[UST3572] VHIP maatwerksync:
Friday 12 Jan 2018
9:25 am

Tommy Hebb

9:18 am

Tom Vermeulen

  • More
  • CR-897
  • finished reviewing
Thursday 11 Jan 2018
3:34 pm

Tommy Hebb

Code aangepast => Verantwoordelijkheid verhuisd naar gepaste klasse (InventSync => MaatwerkStockVerschilAfhandelaar) en gedrag passender gemaakt (Indien geen stockverschillen => standaard mailtje d...

Code aangepast => Verantwoordelijkheid verhuisd naar gepaste klasse (InventSync => MaatwerkStockVerschilAfhandelaar) en gedrag passender gemaakt (Indien geen stockverschillen => standaard mailtje dat dit meldt, ipv een lege lijst te mailen met dezelfde tekst als mochten er wel stockverschillen zijn). UnitTests uitgebreid.