CreateAndNavigateToTemporaryOrder.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
    ./CreateAndNavigateToTemporaryOrder.cls.xml
  1. … 10 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
    ./CreateAndNavigateToTemporaryOrder.cls.xml
[UST2515]VHIP908: 02 - terminalnavigatie

- Review opm

    • -0
    • +2
    ./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
    ./CreateAndNavigateToTemporaryOrder.cls.xml
  1. … 1 more file in changeset.
[UST2602] VHIP908: 03 - AxOrderLevencyclus

- review opmerkingen

    • -1
    • +1
    ./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
Het feit dat je AxSalesId gebruikt zou ik hier ook AxCustomerId gebruiken ipv 01234 kwestie van het toch wat meer duidelijker gedocumenteerd te hebben. Als we het dan in latere fase meer gaan vers...

Het feit dat je AxSalesId gebruikt zou ik hier ook AxCustomerId gebruiken ipv 01234 kwestie van het toch wat meer duidelijker gedocumenteerd te hebben.

Als we het dan in latere fase meer gaan verstandaardiseren kunnen we nummerkes gebruiken voor AxCustomerId en AxSalesId

Privatiseren

Privatiseren

Casing : Request

Casing : Request

'k zou mijn variabele ook gewoon Service noemen want CANTO is wel wat raar

'k zou mijn variabele ook gewoon Service noemen want CANTO is wel wat raar

'k deze gewoon Request noemen ipv CANTORequest

'k deze gewoon Request noemen ipv CANTORequest

[UST2602] VHIP908: 03 - AxOrderLevencyclus
[UST2602] VHIP908: 03 - AxOrderLevencyclus
[UST2515]VHIP908: 02 - terminalnavigatie
[UST2515]VHIP908: 02 - terminalnavigatie