CancelContainerRequestConvertor.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-179
  • finished reviewing
Het is te hopen dat de andere code dat ook doet !? Gelieve hiervoor een betere omschrijving te bedenken. Een korte samenvatting van het wat gedrag is m.a.w. en niet hoe maar wat er juist gedaan wo...

Het is te hopen dat de andere code dat ook doet !?

Gelieve hiervoor een betere omschrijving te bedenken. Een korte samenvatting van het wat gedrag is m.a.w. en niet hoe maar wat er juist gedaan wordt.

Beter een $$$Not(BevatFout) zodoende 0 en "" hetzelfde terug geven.

Beter een $$$Not(BevatFout) zodoende 0 en "" hetzelfde terug geven.

  • More
  • CR-161
  • finished reviewing
macro $$$false gebruiken ipv 0

macro $$$false gebruiken ipv 0

Gelieve de bewuste id in de foutmelding op te nemen en indien mogelijk ook welke type bericht het dan wel was, kwestie om zo duidelijk mogelijke foutmelding te kunnen geven. ps : Enkel eigennamen o...

Gelieve de bewuste id in de foutmelding op te nemen en indien mogelijk ook welke type bericht het dan wel was, kwestie om zo duidelijk mogelijke foutmelding te kunnen geven.
ps : Enkel eigennamen of het begin van een zin begint met een hoofdletter

White space http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

White space

GeefVerzendData zegt weinig over de wat voor verzend data het is ? Als het echt niet uitmaakt gelieve dan een ##class(vhTest.Dummy) te gebruiken als het wel van belang is gelieve dan een iets duide...

GeefVerzendData zegt weinig over de wat voor verzend data het is ? Als het echt niet uitmaakt gelieve dan een ##class(vhTest.Dummy) te gebruiken als het wel van belang is gelieve dan een iets duidelijkere benaming te kiezen.

Utils mogen classmethods zijn

Utils mogen classmethods zijn

Gelieve het gedrag en het gewenste resultaat iets duidelijk in de test te vermelden zodoende er kan gecontroleerd worden dat de test setup wel is volgens het vooropgestelde gedrag. De volgende stap...

Gelieve het gedrag en het gewenste resultaat iets duidelijk in de test te vermelden zodoende er kan gecontroleerd worden dat de test setup wel is volgens het vooropgestelde gedrag. De volgende stap is dat dat de code voldoet aan het vooropgesteld gedrag van de test.

Lijkt mij beter om VerzendData uit te breiden met BevatFout en Foutmelding zodoende er maar één object moet doorgegeven worden. Idem dito voor de CancelContainer..

Lijkt mij beter om VerzendData uit te breiden met BevatFout en Foutmelding zodoende er maar één object moet doorgegeven worden. Idem dito voor de CancelContainer..

?? mag deze method niet weg daar de response controleur kan gebruikt worden ??

?? mag deze method niet weg daar de response controleur kan gebruikt worden ??

[UST2305] VHIP_GLS: CancelContainer implementeren
[UST2305] VHIP_GLS: CancelContainer implementeren
[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)

- VerzendData onder GLSv1 package

    • -1
    • +1
    ./CancelContainerRequestConvertor.cls.xml
  1. … 40 more files in changeset.
[UST2305] VHIP_GLS: CancelContainer implementeren

- initiele versie + test om onderliggende calls te verifieren.

    • -0
    • +21
    ./CancelContainerRequestConvertor.cls.xml
  1. … 13 more files in changeset.
Verzekeren dat er nergens data zit die niet verwijst naar niet GLSv1 als dat zo is deploy klasse maken om die data te corrigeren.

Verzekeren dat er nergens data zit die niet verwijst naar niet GLSv1 als dat zo is deploy klasse maken om die data te corrigeren.

Lijkt mij beter om deze klasse om te vormen naar een vhTest.Utils klasse .. kwestie van een beetje conformiteit in de code base te krijgen

Lijkt mij beter om deze klasse om te vormen naar een vhTest.Utils klasse .. kwestie van een beetje conformiteit in de code base te krijgen

AX testen zou ik niet samen steken met andere testen.. De zaken duidelijk van elkaar gescheiden houden lijkt mij wel handig in dit concept zodoende we makkelijk de zaken kunnen opkuisen eens de zak...

AX testen zou ik niet samen steken met andere testen.. De zaken duidelijk van elkaar gescheiden houden lijkt mij wel handig in dit concept zodoende we makkelijk de zaken kunnen opkuisen eens de zaken effectief in AX zitten.

auwch

auwch

Factory is niet testbaar daar er een classmethod gebruikt wordt . . -> depedency ook naar boven brengen er nu een andere klasse gebruikt wordt om dto klassen te maken.

Factory is niet testbaar daar er een classmethod gebruikt wordt . . -> depedency ook naar boven brengen er nu een andere klasse gebruikt wordt om dto klassen te maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

Foutmelding zal hoogstwaarschijnlijk is niet mogelijk moeten zijn. Foutmelding dan ook beetje uniformiseren en de waarde tussen ' plaatsen

Foutmelding zal hoogstwaarschijnlijk is niet mogelijk moeten zijn. Foutmelding dan ook beetje uniformiseren en de waarde tussen ' plaatsen

[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)
[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)