CacheAdminA_trunk2010/vhUnitTest/CHUI

VKP

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 16 Aug 17
Commits this week: 0
Total Lines of Code (LoC): 777
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 17 Oct 2018
5:30 pm

PieterVR

[UW554] DoorgestuurdNaarAX in BestelStatusHistoriek van Vhisie4-"winkelkarren".
Options
    • -1
    • +2
    ./OrderIngave/TijdelijkOrderSluiterv01/Test.cls.xml
Tuesday 05 Jun 2018
10:24 am

Jo Willems

Friday 01 Jun 2018
11:28 am

Kurt Callebaut

Thursday 31 May 2018
4:09 pm

Jo Willems

[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.
[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.
4:08 pm

Jo Willems

[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.

- Geen productorderlijnen aanbrengen als bezwaar.

- bijkomende test.

Options
    • -2
    • +34
    ./OrderIngave/TijdelijkOrderSluiterv01/Test.cls.xml
  2. … 1 more file in changeset.
Wednesday 09 May 2018
2:36 pm

Tommy Hebb

2:31 pm

Tommy Hebb

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

- Na review => refactor => naamgeving

Options
    • -28
    • +28
    ./OrderIngave/OrderControle/Test.cls.xml
2:04 pm

Martijn Vierstraete

1:52 pm

Martijn Vierstraete

naamgeving: OrderApiStub

naamgeving: OrderApiStub

1:46 pm

Tommy Hebb

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

Tommy Hebb

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

- UnitTest => variaties

- Kleine refactoring => afzonderen gezamelijke code naar private method => GeefOrderMockVolgensAantalOrderLijnen

Options
    • -15
    • +114
    ./OrderIngave/OrderControle/Test.cls.xml
1:23 pm

Tommy Hebb

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

- UnitTest => Default werking => IsOrderVolledigGemarkeerd -> False

Options
    • -0
    • +29
    ./OrderIngave/OrderControle/Test.cls.xml
12:32 pm

Tommy Hebb

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

- Refactor OrderControle => Injectie OrderApi

- Refactor OrderControle => OrderLijnIterator type TECH.Iterator ipv string

- Eerste UnitTest => Default werking

Options
  1. ./OrderIngave/OrderControle
    • -0
    • +125
    ./OrderIngave/OrderControle/Test.cls.xml
  3. … 1 more file in changeset.
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.

9:19 am

Jo Willems

Reverse merged revision(s) 51210 from trunk2010/vhUnitTest/APPS/VKP/OrderIngave:

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

- move van test opkuisen

........

misschien de juiste test opkuisen....

Options
  1. ./OrderIngave/TijdelijkOrderVerwijderaar
    • -146
    • +0
    ./OrderIngave/TijdelijkOrderVerwijderaar/Test.cls.xml
  3. … 2 more files in changeset.
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
    • -2
    • +2
    ./OrderIngave/TijdelijkOrderSluiterv01/Test.cls.xml
    • -6
    • +14
    ./OrderIngave/TijdelijkOrderVerwijderaar/Test.cls.xml
  3. … 19 more files in changeset.
Thursday 03 May 2018
11:47 am

Jo Willems

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

- inbouwen repo + registreren order indien verwijderen niet lukt.

- test gedrag indien orderverwijderen niet lukt.

Options
    • -3
    • +11
    ./OrderIngave/TijdelijkOrderSluiterv01/Test.cls.xml
  2. … 1 more file in changeset.
11:18 am

Jo Willems

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

- OrderVerwijderaar + testen

Options
    • -0
    • +138
    ./OrderIngave/TijdelijkOrderVerwijderaar/Test.cls.xml
  2. ./OrderIngave/TijdelijkOrderVerwijderaar
  3. … 1 more file in changeset.
Wednesday 25 Apr 2018
1:37 pm

Jo Willems

Tuesday 10 Apr 2018
10:54 am

Jo Willems

[ICT] VHIP504

- meer detail in logging indien verwijderen tijdelijk order mislukt.

Options
    • -2
    • +4
    ./OrderIngave/TijdelijkOrderSluiterv01/Test.cls.xml
  2. … 1 more file in changeset.
Thursday 08 Mar 2018
1:47 pm

Jo Willems

nee. Het probleem was dat er eerst doorgestuurd was voor picking. niet dat er het order gesloten was. Maakt niet uit hoe je het noemt. verkeerde handeling van gebruiker houdt daar geen rekening mee...

nee. Het probleem was dat er eerst doorgestuurd was voor picking. niet dat er het order gesloten was. Maakt niet uit hoe je het noemt. verkeerde handeling van gebruiker houdt daar geen rekening mee...
En als ze een order met klant halux naar AX willen sturen zijn ze al helemaal niet goe bezig.

12:11 pm

Kurt Callebaut

12:10 pm

Kurt Callebaut

Ik begrijp de oplossing, maar sluiten via F11 of S (Ax order) kan nu ook tegenstrijdig met wat menu aangeeft, geen AX order zijn. De volgende melding zal dan zijn ik heb een Ax order gemaakt en het...

Ik begrijp de oplossing, maar sluiten via F11 of S (Ax order) kan nu ook tegenstrijdig met wat menu aangeeft, geen AX order zijn.
De volgende melding zal dan zijn ik heb een Ax order gemaakt en het staat niet in AX ?

Waarom legacy die niet meer gebruikt werd terug in gebruik nemen ?

Ik stel voor dan een derde menupunt aan te maken
ofwel bevestigdAXorder te hernoemen naar BevestigdOrder en die te gebruiken ?

11:50 am

Kurt Callebaut