Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-252
  • finished reviewing
[ICT258] - Fix falende Test

- Review opmerkingen

  1. … 1 more file in changeset.
Is deze nodig ?

Is deze nodig ?

In de test is niet duidelijk dat het hier gaat om prijs affichering. Is deze test compleet ?

In de test is niet duidelijk dat het hier gaat om prijs affichering. Is deze test compleet ?

beter zou zijn OrderIterator ipv itOrders

beter zou zijn OrderIterator ipv itOrders

idem Set ipv dim

idem Set ipv dim

Set ipv dim

Set ipv dim

namen test methods verduidelijken.

namen test methods verduidelijken.

##class(DOM.DomeinContext).Instance().GeefOrderAPI() Zou naar constructor mogen en property voorzien. Meegeven als constructorparameter heeft zijn gevolgen op afgeleide klassen die reeds met meerde...

##class(DOM.DomeinContext).Instance().GeefOrderAPI() Zou naar constructor mogen en property voorzien. Meegeven als constructorparameter heeft zijn gevolgen op afgeleide klassen die reeds met meerdere parameters werken, dus dat zou ik voorlopig niet doen.

##class(DOM.DomeinContext).Instance().GeefOrderAPI() Zou naar constructor mogen en property voorzien. Meegeven als constructorparameter heeft zijn gevolgen op afgeleide klassen die reeds met meerde...

##class(DOM.DomeinContext).Instance().GeefOrderAPI() Zou naar constructor mogen en property voorzien. Meegeven als constructorparameter heeft zijn gevolgen op afgeleide klassen die reeds met meerdere parameters werken, dus dat zou ik voorlopig niet doen.

Ook hier zou ik meer informatie meegeven in de omschrijving bv lijnnr alsook bonnummer waarvoor deze uitgevoerd is. zodat er voldoende info beschikbaar is als er een fout optreed.

Ook hier zou ik meer informatie meegeven in de omschrijving bv lijnnr alsook bonnummer waarvoor deze uitgevoerd is. zodat er voldoende info beschikbaar is als er een fout optreed.

dim mag op 1 lijn met methodcall GeefBonLijn

dim mag op 1 lijn met methodcall GeefBonLijn

OrderAPI MockInstance ?

OrderAPI MockInstance ?

Bij deze test zijn de omschrijvingen steeds dezelfde. Misschien een idee om lijnnummers bij te vermelden. Mocht er een fout optreden, dan weet je op zijn minst het aantal dat goed gaat.

Bij deze test zijn de omschrijvingen steeds dezelfde. Misschien een idee om lijnnummers bij te vermelden. Mocht er een fout optreden, dan weet je op zijn minst het aantal dat goed gaat.

Casing

Casing

[ICT258] - Fix falende Test
[ICT258] - Fix falende Test
[ICT258] - Fix falende Test

- Fix + Test Cleanup ( Early Quits iterator checks )

  1. … 2 more files in changeset.
[UW150] Fix Unittesten (producten verwijderd door PM)
  1. … 9 more files in changeset.