CacheAdminA_trunk2010

Dacht dat over een paar dagen/weken de converter ook LBX en TAX ging moeten verwerken, daarom geen TBXKenmerken in naam. De tests hebben wel TBX in hun naam. Andere convertoren behandelen nu ook alle types schuifkes.

Overbodige lijn

Overbodige lijn

Overbodige lijn

waarom zijn dit geen kenmerken van een standaardlade kwestie van toch iets volledig terug te geven naar intracto

In de naamgeving van de klasse ontbreekt het feit dat het over TbxKenmerken gaat en ECONimpl is een root package
cfr AXimpl en WSimpl

Dus eigenlijk zou ECONimpl.PM.Maatwerk.TbxKenmerkenEdiConverter een correctere naam zijn --> natuurlijk is het dan noodzakelijk om de UT's hun naam te wijzigen

Is eigenlijk een beetje muggenziften maar het zou nog dat tikkeltje beter zijn mocht de code er ongeveer zo uitzien

do ..AssertEdiNaarEcon(##class(vhTest.Utils.APPS.EDI.common.dto.LadeTBX).StandaardLade(),##class(vhTest.Utils.ECON.PM.Maatwerk.dto.TbxKenmerken).StandaardLade())

Als je data gebruikt gelieve representatieve data te gebruiken. Dus ik zou kijken om de standaard adressen te gebruiken naar analogie met vhTest.Utils.APPS.common.dto.Adres

Deze testen zijn zo basic dat ik zelf de moeite niet zou doen om deze manueel te maken.
Het lijkt mij beter om ( later ) aan een systeem te werken :
dat op zijn minst ofwel de queries gebruikt in een repository gaat controleren of die überhaupt uitvoerbaar is.
ofwel effectief wat data gaat opzetten en uitlezen daar het gaat om een repository

Deze testen zijn zo basic dat ik zelf de moeite niet zou doen om deze manueel te maken.
Het lijkt mij beter om ( later ) aan een systeem te werken :
dat op zijn minst ofwel de queries gebruikt in een repository gaat controleren of die überhaupt uitvoerbaar is.
ofwel effectief wat data gaat opzetten en uitlezen daar het gaat om een repository

Zeker dat de landcodes die gaan gebruikt worden door intracto mappen op onze landcodes ?

Gebruik maken van gemeenschappelijke klasse om ExterneId te controleren

Het feit dat er op de DOM.EC.. winkelkar en APPS.VKP.Offerte een ExterneId staat is een intern gegeven dus de controle op winkelkar en offerte horen samen te zitten.
Het zou in principe niet kunnen dat de ExterneId reeds op de winkelkar staat en niet op de offerte dus is eigenlijk een controle op de offerte voldoende.
Dus de foutmelding die we naar de buitenwereld sturen is eerder één van alla "ID reeds gebruik" want intracto heeft geen weet van onze offertes en Winkelkarren.

Mock & Indien = Stub

Mock met $$$ElkeWaarde not done

Wat is de nood van deze klasse ?

OfferteBewaarder mag vervangen worden door OfferteRepository - OfferteBewaarder mag eventueel integraal verwijderd worden als het niet teveel inpakt heeft