TerminalNavigationService

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

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
verkeerde log message nog in crucible; in svn al aangepast.

verkeerde log message nog in crucible; in svn al aangepast.