Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-753
  • finished reviewing
Implementatie is volgens mij wel correct en sowieso nodig voor AX. Alleen stel ik me alleen de vraag of het gooien van de foutmelding correct is ? We kunnen hiervoor een nieuwe exceptie introduceren.

Implementatie is volgens mij wel correct en sowieso nodig voor AX. Alleen stel ik me alleen de vraag of het gooien van de foutmelding correct is ? We kunnen hiervoor een nieuwe exceptie introduceren.

[UST3197] VHIP908: warning bij navigatie naar nog niet bevestigd order, ipv errormail
[UST3197] VHIP908: warning bij navigatie naar nog niet bevestigd order, ipv errormail
[UST3197] VHIP908: warning bij navigatie naar nog niet bevestigd order, ipv errormail

- alleen indien order nog niet bestaat een warning loggen db out of sync exception zonder extrainfo

- in de andere gevallen toch error loggen

- in de bijzondere gevallen een aangepaste melding voor AX voorzien.

    • -3
    • +14
    ./TerminalNavigationService/NavigateToConfirmedOrder.cls.xml
  1. … 1 more file in changeset.
testen zou zijn op ordernummer. en die lijst bouwt de test zelf op. ik denk niet dat het zin heeft de query engine van de database test als deze resultaten terug geeft die niet overeenstemmen met d...

testen zou zijn op ordernummer. en die lijst bouwt de test zelf op. ik denk niet dat het zin heeft de query engine van de database test als deze resultaten terug geeft die niet overeenstemmen met de query.
Dan gaan er meer dingen fout gaan.

  • More
  • CR-449
  • finished reviewing
Deze testen valideren Query en testen op resultset en niet op grenswaarden in de query. Bv 101000 wijzigen in VKP.OrderLookup query, geeft nog steeds een geslaagde unittest. Is het nodig om dit ook...

Deze testen valideren Query en testen op resultset en niet op grenswaarden in de query.
Bv 101000 wijzigen in VKP.OrderLookup query, geeft nog steeds een geslaagde unittest.
Is het nodig om dit ook te controleren ?

Vorostel om te werken met ExactAantalKeer(1).DanDoeNiks() zodat we zeker zijn dat deze method maar 1 keer opgeroepen wordt.

Vorostel om te werken met ExactAantalKeer(1).DanDoeNiks() zodat we zeker zijn dat deze method maar 1 keer opgeroepen wordt.

Overerven van repository en naam ?

Overerven van repository en naam ?

[UST2679] VHIP908: NavigateToConfirmedOrder
[UST2679] VHIP908: NavigateToConfirmedOrder
[UST2679] VHIP908: NavigateToConfirmedOrder

- 2de deel volgt : specs over tijdelijk order zijn veranderd : indien reeds een tijdelijk order bestaat voor gegeven axsalesid, dat openen ipv nieuw maken

- betere differentiatie methodnames

- maar te veel in één commit :-(

    • -3
    • +11
    ./TerminalNavigationService/CreateAndNavigateToTemporaryOrder.cls.xml
  1. … 10 more files in changeset.
[UST2679] VHIP908: NavigateToConfirmedOrder

- eerste deel Navigate to AX confirmed Order

- 2de deel volgt : specs over tijdelijk order zijn veranderd

    • -0
    • +42
    ./TerminalNavigationService/NavigateToConfirmedOrder.cls.xml
  1. … 9 more files in changeset.
[UST2515]VHIP908: 02 - terminalnavigatie

- filteren nullcharacter in response message in geval AX lege parameters stuurt (wat normaal niet kan, maar wegens onvolledige AX code toch gebeurde)

    • -0
    • +1
    ./TerminalNavigationService/CreateAndNavigateToTemporaryOrder.cls.xml
[UST2515]VHIP908: 02 - terminalnavigatie

- Review opm

    • -0
    • +2
    ./TerminalNavigationService/CreateAndNavigateToTemporaryOrder.cls.xml
  1. … 1 more file in changeset.
Dis is wat er misliep. Het is de omschrijving van de exceptie die gesmeten werd.

Dis is wat er misliep. Het is de omschrijving van de exceptie die gesmeten werd.

  • More
  • CR-332
  • finished reviewing
  • More
  • CR-331
  • finished reviewing
?? En wat liep er mis ??

?? En wat liep er mis ??

Misschien moeten we wel de error loggen zodoende we weten dat er iets is mis is, dit is van toepassing op verdere fouten ook

Misschien moeten we wel de error loggen zodoende we weten dat er iets is mis is, dit is van toepassing op verdere fouten ook

[UST2515]VHIP908: 02 - terminalnavigatie

- eventuele execpties opvangen en response message aanpassen

    • -3
    • +14
    ./TerminalNavigationService/CreateAndNavigateToTemporaryOrder.cls.xml
  1. … 1 more file in changeset.
[UST2602] VHIP908: 03 - AxOrderLevencyclus

- review opmerkingen

    • -1
    • +1
    ./TerminalNavigationService/CreateAndNavigateToTemporaryOrder.cls.xml
  1. … 5 more files in changeset.
Ofwel asserten dat het teruggegeven order wel degelijk hetzelfde order is dan de OrderMock ( moet niet persé dezelfde instantie zijn ) Ofwel het order niet teruggeven , hetgeen eigenlijk nog beter...

Ofwel asserten dat het teruggegeven order wel degelijk hetzelfde order is dan de OrderMock ( moet niet persé dezelfde instantie zijn )

Ofwel het order niet teruggeven , hetgeen eigenlijk nog beter is - je voldoet aan CQRS en je hebt minder test werk

Repo gebruiken ipv Save op het order

Repo gebruiken ipv Save op het order

  • More
  • CR-330
  • finished reviewing