Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
waarom worden deze twee velden niet meer ingevuld?

waarom worden deze twee velden niet meer ingevuld?

TECH.Mockable erft al over van TECH.RegisteredObject, dus die TECH.RegisteredObject mag weg

TECH.Mockable erft al over van TECH.RegisteredObject, dus die TECH.RegisteredObject mag weg

[ICT-1248] PA-UB: Productie afmelden: etiketten printen
[ICT-1248] PA-UB: Productie afmelden: etiketten printen
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
Nu komt die "1" precies uit de lucht gevallen. Men moet al moeite doen en verder zoeken, om te vinden dat die vanuit de method GeefTvwTijdelijkOrder() komt. Voorstel: ..GeefTeVerwijderenTijdelijkO...

Nu komt die "1" precies uit de lucht gevallen. Men moet al moeite doen en verder zoeken, om te vinden dat die vanuit de method GeefTvwTijdelijkOrder() komt.

Voorstel:
..GeefTeVerwijderenTijdelijkOrder(TvwOrderID)
Do TeVerwijderenOrderRepositoryMock.VerwachtMethodCall("VerwijderViaID",TvwOrderID).DanTralala

idem in de volgende methods

zie volgende comment, hieronder.

zie volgende comment, hieronder.

Deze review-opmerking mocht ik niet maken (van JoW) : klassenaam in t enkelvoud.

Deze review-opmerking mocht ik niet maken (van JoW) :
klassenaam in t enkelvoud.

Tja, ik moet het melden, hé ;-P $Inject() kan best de OrderAPI newen i.p.v. Context.Instance() ... maar als de discussie nog gesloten is, dan laat je het maar zo.

Tja, ik moet het melden, hé ;-P
$Inject() kan best de OrderAPI newen i.p.v. Context.Instance()
... maar als de discussie nog gesloten is, dan laat je het maar zo.

  • More
  • CR-966
  • finished reviewing
De ordersluiter is nog altijd zonder mock/stub (in naamgeving) http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif Ter info: ingeval van de verkoopservice ...

De ordersluiter is nog altijd zonder mock/stub (in naamgeving)
Ter info: ingeval van de verkoopservice zou je waar je "verwachte calls" hebt, een lokale mock kunnen gebruiken. Voor de rest kan je dan de globale stub (uit de onbefore) gebruiken.

ok voor ordersluiterMock. De verkoopservice laat ik als mock staan aangezien dit een algemene property is voor alle testen in deze testunit.(zie onbefore.)

ok voor ordersluiterMock. De verkoopservice laat ik als mock staan aangezien dit een algemene property is voor alle testen in deze testunit.(zie onbefore.)

  • More
  • CR-966
  • finished reviewing
Zou je in de catch dan niet alsnog de mail versturen die vroeger door de BesteldeOfferteMelder gestuurd werd? De order is niet in AX geraakt, verschijnt daar dus niet op een lijst, en ze worden nie...

Zou je in de catch dan niet alsnog de mail versturen die vroeger door de BesteldeOfferteMelder gestuurd werd? De order is niet in AX geraakt, verschijnt daar dus niet op een lijst, en ze worden niet verwittigd dat er een te sluiten order in cache zit te wachten. nu is alleen ict verwittigd, maar is het niet eerst ov die moet proberen of de verwerking manueel kan?

Naamgeving: *VerkoopServiceStub ipv -Mock *AsynchroneOrderSluiterMock

Naamgeving:

  • VerkoopServiceStub ipv -Mock
  • AsynchroneOrderSluiterMock
[UST3611] Vhisie4 : Het bestellen van een winkelkar van bestel tot order, automatiseren om OV te...
[UST3611] Vhisie4 : Het bestellen van een winkelkar van bestel tot order, automatiseren om OV te...
  • More
  • CR-788
  • finished reviewing
[UST3275] vhip901 test ordersluiter
[UST3275] vhip901 test ordersluiter
  • More
  • CR-768
  • finished reviewing
Nergens bewijs van het feit dat de sluiter opgeroepen wordt

Nergens bewijs van het feit dat de sluiter opgeroepen wordt

[UST3227] [EPIC74] vhip901 Verkooporders toelevering aanmaken voor standaard taor lade.
[UST3227] [EPIC74] vhip901 Verkooporders toelevering aanmaken voor standaard taor lade.