Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

- falende testen omdat nu op het einde geen extra CRLF wordt toegevoegd, zoals vroeger (foutief) gebeurde : er zat ook geen CRLF in de originele stream

    • -1
    • +1
    ./AIF/SalesInvoice/Create/SalesInvoice-Create 586689 - VHBE - 20160501_091500_000000.xml
  1. … 4 more files in changeset.
[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
  • More
  • CR-437
  • finished reviewing
[UST2721] VHIP903: Period. voorraad: Aanpassing Generate() in AXimpl.Generator:

- Refactoring: Goede oplossing voor TimeStamp

- Correctie bestaande testen (filename testbestanden aangepast tot nieuwe implementatie)

    • -0
    • +83
    ./AIF/SalesInvoice/Create/SalesInvoice-Create 586689 - VHBE - 20160501_091500_000000.xml
  1. … 12 more files in changeset.
Zoals ge zelf gezien hebt, kan het zijn dat files elkaar overschrijven indien ze binnen dezelfde seconde gegenereerd worden. Subject misschien aanpassen naar <Nummer van de eerste> en <X> andere (...

Zoals ge zelf gezien hebt, kan het zijn dat files elkaar overschrijven indien ze binnen dezelfde seconde gegenereerd worden.

Subject misschien aanpassen naar <Nummer van de eerste> en <X> andere (X is het aantal in de subjectId -1)

[UST2721] VHIP903: Period. voorraad: Aanpassing Generate() in AXimpl.Generator:
[UST2721] VHIP903: Period. voorraad: Aanpassing Generate() in AXimpl.Generator:
  • More
  • CR-281
  • finished reviewing
?? Iets te ijverig

?? Iets te ijverig

?? Iets te ijverig

?? Iets te ijverig

[ICT260] Voorbereiding interfaces
[ICT260] Voorbereiding interfaces
[ICT260] Voorbereiding interfaces

- Verplaatsen testbestanden

    • -0
    • +83
    ./AIF/SalesInvoice/Create/SalesInvoice-Create 586689 - VHBE - 20160501_091500.xml
  1. … 68 more files in changeset.