Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Misschien kan je toch stukken code, die 3x voorkomen, verplaatsen naar private methods. Bijvoorbeeld .GeefProductOrderlijnMock() of .GeefProductOrderlijnMockVoorStandaardOrderMetProductLijn() ofzo....

Misschien kan je toch stukken code, die 3x voorkomen, verplaatsen naar private methods. Bijvoorbeeld .GeefProductOrderlijnMock() of .GeefProductOrderlijnMockVoorStandaardOrderMetProductLijn() ofzo.
idem OrderMock

Dat zou de 3 testmethods mogelijk overzichtelijker maken (lees: duidelijkere focus op de verschillen tussen de 3)

Als het toch veel te lastig blijkt te zijn, dan wellicht toch beter zo laten staan. Kies zelf maar.

[VRB622] BIDI: Orderingave: SPT-orders mogen niet geüpdated worden vanuit AX:
[VRB622] BIDI: Orderingave: SPT-orders mogen niet geüpdated worden vanuit AX:
[VRB622] BIDI: Orderingave: SPT-orders mogen niet geüpdated worden vanuit AX:

- Indien meerdere orderlijnen met dezelfde AxInventTransId => allemaal verwijderen ipv slechts 1 => worden daarna allemaal herzet

- Order MOET daarna terug bewaard worden om persistering te syncen met object in memory => anders worden opnieuw gezette ordelijnen nadien terug gewist

- VerwijderOrderLijnViaAxInventTransId ===> VerwijderOrderLijnenViaAxInventTransId => Alle lijnen met die AxInventTransId worden nu verwijderd

  1. … 8 more files in changeset.
Vraag me af of de verwacht hier wel nodig is aangezien we enkel moeten check op verwijder order en inderdaad dan is den indien van de base voldoende.

Vraag me af of de verwacht hier wel nodig is aangezien we enkel moeten check op verwijder order en inderdaad dan is den indien van de base voldoende.

Yep das verkeerd.

Yep das verkeerd.

base voorziet indienmethodcall

base voorziet indienmethodcall

Base voorziet indienmethod call en wordt hier overschreven met verwachtmethodcall

Base voorziet indienmethod call en wordt hier overschreven met verwachtmethodcall

VerwachtMethodCall is hier ook niet nodig => Base-class voorziet dit.

VerwachtMethodCall is hier ook niet nodig => Base-class voorziet dit.

Misschien, naar analogie van de andere TestCase ("Test") de OrderMock ook ophalen van de Base-class (..GeefOrder()), dan hoef je hier niet overal deze VerwachtMethodCalls te doen.

Misschien, naar analogie van de andere TestCase ("Test") de OrderMock ook ophalen van de Base-class (..GeefOrder()), dan hoef je hier niet overal deze VerwachtMethodCalls te doen.

Zijn die hier in alle unit tests wel nodig? De Base-class voorziet dit toch al?

Zijn die hier in alle unit tests wel nodig? De Base-class voorziet dit toch al?

OrderID bestaat niet als variabele. Moet InkomendSalesId zijn?

OrderID bestaat niet als variabele. Moet InkomendSalesId zijn?

[UW567] voorkomen dat oude of hergebruikte orderid's verkeerde updates uitvoeren. maw controleren op...
[UW567] voorkomen dat oude of hergebruikte orderid's verkeerde updates uitvoeren. maw controleren op...
[UW567] voorkomen dat oude of hergebruikte orderid's verkeerde updates uitvoeren. maw controleren op salesid.

  1. … 4 more files in changeset.
Blijkbaar ben ik hier een type vergeten.. is intussen toegevoegd.

Blijkbaar ben ik hier een type vergeten.. is intussen toegevoegd.

inderdaad moet "zet" zijn.

inderdaad moet "zet" zijn.

Ik heb geleerd dat ge niet altijd op cache moogt vertrouwen, dus liever een controle meer dan minder. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ik heb geleerd dat ge niet altijd op cache moogt vertrouwen, dus liever een controle meer dan minder.

  • More
  • CR-817
  • finished reviewing
Geef of Zet?

Geef of Zet?

Kan dit anders zijn zonder dat hierboven een exception gesmeten werd? Indien ja, gebeurd er niets zonder dat het gelogd werd.

Kan dit anders zijn zonder dat hierboven een exception gesmeten werd?
Indien ja, gebeurd er niets zonder dat het gelogd werd.

  • More
  • CR-781
  • finished reviewing
[UST3177] [rvJWI] VHIP901: ORDER2ADMIN: updates op gedeeltelijk afgewerkte orders. maw geen lijnen...
[UST3177] [rvJWI] VHIP901: ORDER2ADMIN: updates op gedeeltelijk afgewerkte orders. maw geen lijnen...
[UST3177] [rvJWI] VHIP901: ORDER2ADMIN: updates op gedeeltelijk afgewerkte orders. maw geen lijnen updaten die status "in productie" hebben.

- Stap 1 van de ...

- negeren van updates voor orderlijnen die reeds in verwerking zijn.

- todo toeleveringen verwijderen

- todo toeleveringen aanmaken.

- ...

  1. … 4 more files in changeset.
[UST3272] [EPIC74] Vhip901 - scenario Toeleveringen verwijderen en opnieuw toevoegen.
[UST3272] [EPIC74] Vhip901 - scenario Toeleveringen verwijderen en opnieuw toevoegen.
[UST3272] [EPIC74] Vhip901 - scenario Toeleveringen verwijderen en opnieuw toevoegen.

- nieuwe toeleveringen aanmaken, nog te testen met meerdere types lades.

LBX alleen : ok

TAX alleen : ok

Beide lades in combinatie met beslag productlijn. (dus geen toelevering.) : ok

Meerdere TAX lijnen : ok

  1. … 5 more files in changeset.