Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT465] EDI TAX tag EdgeBandColour voorzien
[ICT465] EDI TAX tag EdgeBandColour voorzien
Is uw probleem natuurlijk, ge moet lezen wat er staat...

Is uw probleem natuurlijk, ge moet lezen wat er staat...

Bij AssertEquals() is het derde argument enkel zinvol als je daar een uitleg wil schrijven. Met Asserts zoals hierboven lijkt het al gauw op $$$AssertEquals(StreamSize, "") Dat staat er natuur...

Bij AssertEquals() is het derde argument enkel zinvol als je daar een uitleg wil schrijven.

Met Asserts zoals hierboven lijkt het al gauw op

 	$$$AssertEquals(StreamSize, "") 

Dat staat er natuurlijk niet, maar mijn dislectisch brein interpretteert dit wel zo :-\
Ook met de andere Asserts in deze klasse heb ik moeite om snel te begrijpen wat er getest wordt.

Wil je de lege Comments in de Asserts (alle in de hele klasse) verwijderen? Uw voorganger heeft niet proper gedaan http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticon...

Wil je de lege Comments in de Asserts (alle in de hele klasse) verwijderen?

Uw voorganger heeft niet proper gedaan
Vandaar mijn vraag of je hier het Boyscout principe wil toepassen? Merci Jo.

Deze assert (van uw voorganger) kan VEEL specifieker en veiliger : Do $$$AssertEquals(Stream.Size, StreamSize + (9*25 )) ; i.e. "*" komt 9x per lijn voor, er zijn 25 lijnen. Wil je deze aanpa...

Deze assert (van uw voorganger) kan VEEL specifieker en veiliger :

	Do $$$AssertEquals(Stream.Size, StreamSize + (9*25 ))   ; i.e. "*" komt 9x per lijn voor, er zijn 25 lijnen.  

Wil je deze aanpassen? Thx.

Beter : Replace naar "Line ... AA" of naar "Line ... 00" , want "Line ... 99" bestaat reeds vooraleer iets ge-replaced wordt --> werkt wel, maar is een beetje misleidend. (wellicht ook in de volgen...

Beter :
Replace naar "Line ... AA" of naar "Line ... 00" , want "Line ... 99" bestaat reeds vooraleer iets ge-replaced wordt --> werkt wel, maar is een beetje misleidend.
(wellicht ook in de volgende testmethods).

[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
[UST4108] LBX: MeenemerSet voor binnenlade: Conversie voor EDI:
[UST4108] LBX: MeenemerSet voor binnenlade: Conversie voor EDI:
[UST4058] LBX: MeenemerSet voor binnenlade: Econ-kenmerken (in Caché) toevoegen + Converters aanpassen:...
[UST4058] LBX: MeenemerSet voor binnenlade: Econ-kenmerken (in Caché) toevoegen + Converters aanpassen:...
Ik snap de redenering maar ik vind dit niet correct. Mss eens in de groep gooien en beslissen wat we hiermee doen?

Ik snap de redenering maar ik vind dit niet correct. Mss eens in de groep gooien en beslissen wat we hiermee doen?

De testmethod voor de standaardlade staat nu inderdaad dubbel. Dus ik heb ze in deze klasse verwijderd. Wat de ZijkantLogo-testen van Kurt betreft, daar ga ik wel afblijven, hé http://subversion02...

De testmethod voor de standaardlade staat nu inderdaad dubbel. Dus ik heb ze in deze klasse verwijderd.

Wat de ZijkantLogo-testen van Kurt betreft, daar ga ik wel afblijven, hé

Vond het eerst zelf ook een beetje vreemd, waarom da aangepast is :-P Verklaring vrij eenvoudig : Naam moet consistent zijn met de andere vhTest.Utils ... Kenmerken() klassen, omdat de UT's voor de...

Vond het eerst zelf ook een beetje vreemd, waarom da aangepast is :-P
Verklaring vrij eenvoudig :
Naam moet consistent zijn met de andere vhTest.Utils ... Kenmerken() klassen, omdat de UT's voor de convertoren dit oproepen:

Method "Test: TAX Standaardlade"()
{
Do ..AssertConverter("StandaardLade")
}

Method "Test: TAX BinnenLade"()
{
Do ..AssertConverter("BinnenLade")
}

Method "Test: TAX LadeMetVeelExtras"()
{
Do ..AssertConverter("LadeMetVeelExtras")
}

Anders falen de UT's ... spijtig genoeg. (normaal gezien moet dan de UT-impl aangepast worden en niet de code, maar deze wijziging heeft geen invloed op de werking van de nietmeeleverens, denk.) De...

Anders falen de UT's ... spijtig genoeg.
(normaal gezien moet dan de UT-impl aangepast worden en niet de code, maar deze wijziging heeft geen invloed op de werking van de nietmeeleverens, denk.)
Deze tactiek was ook al toegepast bij LBX door één van m'n voorgangers.

De "ByRef" is hier om aan te geven dat het EconKenmerken-object zal aangepast worden. Technisch is het niet fout om da als .local door te geven. Objecten worden trouwens altijd als "pointer naar ge...

De "ByRef" is hier om aan te geven dat het EconKenmerken-object zal aangepast worden.
Technisch is het niet fout om da als .local door te geven. Objecten worden trouwens altijd als "pointer naar geheugenplaats" doorgegeven.

Principe is bovendien overgenomen van de 2 lijnen eronder, en dat leek mij niet verkeerd om het zo aan te duiden.

Laat zeker weten als je hier toch aan twijfelt.

Zal bij volgende opkuis verwijderd worden.

Zal bij volgende opkuis verwijderd worden.

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-? Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 . De review heb ik wel hier toegevoegd, w...

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-?
Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 .
De review heb ik wel hier toegevoegd, want gij hebt er achter gevraagd :-D :-D

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven! *vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: ...

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven!

  • vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: TAX LadeMetVeelExtras)
    Maar ik maak een nieuwe testmethod bij, die controleert of verpakking "" wordt geconverteerd naar V1.
in volgende fase wordt dit opgekuist http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

in volgende fase wordt dit opgekuist