CacheAdminA_trunk2010/vhTest/Mock/CHUI

VKP

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 29 Nov 17
Commits this week: 0
Total Lines of Code (LoC): 92
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 09 May 2018
11:21 am

Jo Willems

11:07 am

Jo Willems

[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
Monday 07 May 2018
5:23 pm

Wim Vermeulen

5:09 pm

Wim Vermeulen

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

5:06 pm

Wim Vermeulen

zie volgende comment, hieronder.

zie volgende comment, hieronder.

4:51 pm

Wim Vermeulen

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.

4:41 pm

Wim Vermeulen

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.

Friday 04 May 2018
3:55 pm

Jo Willems

[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- refactor naar juiste package

Options
    • -87
    • +0
    ./OrderIngave/TeVerwijderenTijdelijkeOrders.cls.xml
    • -58
    • +0
    ./OrderIngave/TeVerwijderenTijdelijkeOrderRepository.cls.xml
  3. … 19 more files in changeset.
Thursday 03 May 2018
11:17 am

Jo Willems

[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- Getters/Setters toegevoegd tbv testen

Options
    • -0
    • +87
    ./OrderIngave/TeVerwijderenTijdelijkeOrders.cls.xml
  2. … 2 more files in changeset.
Wednesday 02 May 2018
1:59 pm

Jo Willems

[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- persistent object en repo om Ordernummer van te verwijderen tijdelijko orders te kunnen beheren

Options
    • -0
    • +58
    ./OrderIngave/TeVerwijderenTijdelijkeOrderRepository.cls.xml
  2. … 3 more files in changeset.
Thursday 15 Feb 2018
10:44 am

MartijnV

[UST3556] OV: Doorsturen picking: rembours: enkel volledig order toelaten

- opvragen default verzendwijze afgezonderd

Options
    • -2
    • +9
    ./OrderIngave/OrderMarkeerder/Helper.cls.xml
  2. … 4 more files in changeset.
10:38 am

MartijnV

[UST3556] OV: Doorsturen picking: rembours: enkel volledig order toelaten

- MarkeerOrder naar Helper voor testbaarheid not procedureblock

Options
    • -0
    • +6
    ./OrderIngave/OrderMarkeerder/Helper.cls.xml
  2. … 4 more files in changeset.
Wednesday 14 Feb 2018
4:40 pm

MartijnV

[UST3556] OV: Doorsturen picking: rembours: enkel volledig order toelaten

- afzonderen controle of magazijn beschikbaar is

- popup tonen indien nodig

Options
    • -0
    • +13
    ./OrderIngave/OrderMarkeerder/Helper.cls.xml
  2. … 5 more files in changeset.
3:51 pm

MartijnV

[UST3556] OV: Doorsturen picking: rembours: enkel volledig order toelaten

- afzonderen controle of order reeds gemarkeerd is

Options
  1. ./OrderIngave/OrderMarkeerder
    • -0
    • +20
    ./OrderIngave/OrderMarkeerder/Helper.cls.xml
  3. … 7 more files in changeset.
Friday 02 Feb 2018
8:45 am

Kurt Callebaut

Thursday 01 Feb 2018
4:34 pm

Martijn Vierstraete

  • More
  • CR-966
  • finished reviewing
4:34 pm

Martijn Vierstraete

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.

Wednesday 31 Jan 2018
2:45 pm

Kurt Callebaut

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

2:14 pm

Jo Willems

  • More
  • CR-966
  • finished reviewing
2:13 pm

Jo Willems

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?

Tuesday 30 Jan 2018
3:00 pm

Martijn Vierstraete

Naamgeving: *VerkoopServiceStub ipv -Mock *AsynchroneOrderSluiterMock

Naamgeving:

  • VerkoopServiceStub ipv -Mock
  • AsynchroneOrderSluiterMock
12:07 pm

Kurt Callebaut

[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...
9:04 am

KurtC

[UST3611] Vhisie4 : Het bestellen van een winkelkar van bestel tot order, automatiseren om OV te ontlasten.

- Mockable voor testen ordersluiter

Options
    • -0
    • +20
    ./OrderIngave/OrderSluiterFactory.cls.xml
  2. … 1 more file in changeset.
Friday 08 Dec 2017
1:07 pm

Kurt Callebaut

Thursday 07 Dec 2017
4:13 pm

Tommy Hebb

  • More
  • CR-788
  • finished reviewing
1:46 pm

Kurt Callebaut

[UST3275] vhip901 test ordersluiter
[UST3275] vhip901 test ordersluiter
Wednesday 06 Dec 2017
1:55 pm

KurtC

[UST3275] vhip901 test ordersluiter
Options
    • -2
    • +2
    ./OrderIngave/OrderSluiter.cls.xml
  2. … 4 more files in changeset.
1:31 pm

Kurt Callebaut

Monday 04 Dec 2017
3:52 pm

Jo Willems

  • More
  • CR-768
  • finished reviewing
3:52 pm

Jo Willems

Nergens bewijs van het feit dat de sluiter opgeroepen wordt

Nergens bewijs van het feit dat de sluiter opgeroepen wordt