Externe

- validatie vervallenprijzen enkel op winkelkarren die maatwerklijnen hebben.

Na bespreking met PRA - Niet op deze manier geldigheid utischakelen. er moet een odnerscheid komen prod en maatwedrklijnen, maar eerst nog verder te bepalen. mogelijks gaat bv intracto ook herbereken uitvoeren als er enkel productlijen zijn, etc... ! revert !

Voor Vhisie4 prijscontrole uitschakelen bij validatie van de offerte geldigheid.

- AfterOneTest reset contaxten op testcase classe. Dus resetten is niet meer nodig afteralltests.

- WisOfferte gebruikt ondanks Fake.pubsubAPI toch productie instance. Hiervoor Fake naar before all test gezet.

- Test terug geactiveerd.

- delete event subscription vervangen door fake PubSub.

- Testen gefixed - abstract klassen ontbraken return values en dus ook mock klasse...

- nog verder te refactoren.

- tijdelijk uitschakelen totdat de test gefixed is ivm eventsubscription

- Restandje van het verwijderen van een TECH.Mockable implementatie van een interface klasse

- Refactoring winkelkar VoegToeMaatwerkLijn geeft object weer ipv ID.

- Raiseevent voor OfferteGemaakt is verhuisd naar VerkoopService.

Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv op repository) Indien er enkel geraiset wordt vanuit verkoopservice, dan heeft de event waarschijnlijk een andere bedoeling/betekenis.

- RaiseOfferteEvent hernoemt naar RaiseOfferteGemaaktEvent.

- Maakwinkelkar method op verkoopservice niet nodig. Maakwinkelkar roept maakofferte op op externe verkoop service.

- request.header.winkelkarId wordt in method meegegeven als ecterneId

- Maakofferte maakt ook nieuwe winkelkar, dus controle externeid ook voor winkelkar toepassen en meegeven bij maakwinkelkar

- Test externeId bewaard op offerte. (en verduidelijking dummy waarden.)

- Aanmaken winkelkar kan via VerkoopService verlopen. Den test

- Toelaten van lege lever/wens-datum

- CheckoutType inject voor falende test te fixen op validator.

- Aanpassing in res.vertaling die testen met daarin ListIterator,KeyValuePair als probleem oplossen

- Wisofferte naar VerkoopService (met alle gevolgen van dien)

- De testjes van ExterneVerkoopService verplaatsen

- Aangezien ik niet alle tests ken waarop dit invloed kan hebben (heb er wel een paar gedraaid), zal ik straks op het bord wel zien wat nog aangepast dient te worden. (of zoals in vervlogen tijden werd gezegd: "Commite eos. Novit enim Dashboardus qui sunt eius"