PlaatsBestelling

is niet volledig duplicate, setup is beetje anders. aangezien het de bedoeling was dat dat alpnachverhaal vrij snel van situatie 1 naar 2 geswitched ging worden, heb ik het niet liggen afzonderen

duplicate code. kan deze via assert met personality en klantid ?

Kan het zijn dat hier vroeger al een stukje code ontbrak ? de controle op sfs personality ?
(zoals in de controle IsAlpnach() . mogelijks geeft dat hier geen probleem. 't is maar een vaststelling.

alpnach Vanhoecke klant ipv rechtstreeks ?

gemeenschappelijke code. code op de APPS laag gebruikt toch al de DOM laag.

hier ook : is vanhoecke klant. ipv rechtstreeks. heeft iets meer betekenis.

rechtstreeks betekent vanhoecke klant.
misschien benamingen : sfs klant of vanhoecke klant gebruiken dat geeft iets meer betekenis.eventueel via personality enum.

els if om instructie te sparen in geval van sfs- alpnach ?

nope, code vliegt er over twee weken toch weer uit

Eén instantie maken van legacypartij lijkt me voldoende.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvragen van klantsessiedata.

Duplicate testcode op klantid na ?

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevestigingschroevenVoorKlantID().

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?

equals() ?

stringutils.equals ?

2 ifs samenvoegen om duplicate code te vermijden.

ik ben niet helemaal mee in het verhaal. Hoe noemen we een klantid met prefix hier? In deze klasse gebruiken we || voor partij id, terwijl deze methods met underscore werken. Horen deze methods hier thuis ? en zoja. Als dit verschil is tussen partijid en legacypartijId, dan zou ik de method hernoemen naar bv GeefKlanIdVanLegacyPartijID() en GeefLegacyPartijIDVanKlantID()

injecteren zodat er kan uitgemocked worden ?