RequestConverter.cls.xml

Testje bijgezet als AttrVsAdminProdNr niet SalesPriceBOM is => dan moet er betreffende de ProductIdRecycleerder niets gebeuren.

Ik vond dat nogal overkill, omdat die list dan amper een codelijn of 6 later alweer omgezet zou moeten worden naar een delimited string. Moest de omzetting naar delimited string in een andere method gebeuren, dan zou ik het zeker gedaan hebben

Ook zoals aan telefoon besproken... het gaat over de LadeVariant zoals gekend in AX, waarvoor overal in de BOMBOL de AXimpl gebruikt wordt.

Zoals besproken aan de telefoon: Dat Request komt in de BOMBOL-interface ook al binnen als parameter, dus de afhankelijkheid is sowieso al een feit. Bovendien wordt er 1 laagje hoger ook een andere property van uitgelezen

ikzelf probeer constructies als deze meestal te vermijden omdat het om de zoveel tijd wel eens kan leiden naar een rariteit met die delimiter. Misschien beter om een list te gebruiken? Kunt ge gewoon op adden indien nodig. Een list kan daarna 'gewoon' in een delimited string omgezet worden indien nodig.

minor nitpicking: ik las dit als een Event "OnJadajada" en het duurde toch een fractie van een seconde voor ik doorhad dat dat niet echt logisch zou zijn indeze context. Een kleine "B" was misschien logische geweest. Nogmaals: kommaneuken hoor

een beetje in de trand van de vorige opmerking: zou het niet properder zijn om de DOM....LadeVariant te gebruiken (qua hiërarchie hé). Je gaat dan wel een 'converter' moeten opzetten die de AX-variant omzet naar een DOM-variant, maar daar kan de LadeVariantBepaler zich mee bezighouden. Op die manier verdwijnt de dependency op AXimpl.* package

Van deze request wordt enkel Request.Attribute gebruikt. Het zou beter zijn om die attribute als parameter mee te geven, anders introduceer je een (extra?) afhankelijkheid tss WSimpl en AXif packages.

#dim ProductServiceMock As vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService = ##class(vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService).%New()
Do ProductServiceMock.VerwachtMethodCall("GeefProduct", ..#RandomProductId, Request.ItemId).ExactAantalKeer(0)
Set ..ProductService = ProductServiceMock

Dit toevoegen, dan kan je de Do ..Verifieer() oproepen

Deze property wordt niet meer gebruikt in deze klasse en mag dus weg (is verplaatst naar productUpdater)

Apart kaartje: UST4276 (ICT-729)

Goed gedaan!
goede uitbreiding :
uwe CheckDelControleur is nen "interface" en alles wat er achter die interface gebeurt, is hier niet relevant (single responsibility).
dus door die uit te mocken heb je perfect op eenvoudige wijze deze UT uitgebreid, en zo getest wat er moest getest worden.
Toppie!