CacheAdminACommon_trunk2010/vhUnitTest/TECH/StreamUtils/Replace

Test.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 10 Dec 18
Commits this week: 0
Total Lines of Code (LoC): 187
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 10 Dec 2018
11:59 am

Jo Willems

[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000

- review opm

- opkuis assert

- test specifieker

Options
    • -56
    • +49
    ./Test.cls.xml
11:53 am

Jo Willems

11:49 am

Jo Willems

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

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

Thursday 06 Dec 2018
3:33 pm

Wim Vermeulen

3:32 pm

Wim Vermeulen

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.

3:22 pm

Wim Vermeulen

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.

2:54 pm

Wim Vermeulen

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.

2:30 pm

Wim Vermeulen

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).

Wednesday 05 Dec 2018
12:33 pm

Jo Willems

[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
12:31 pm

Jo Willems

[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000

- testen voor

- Line terminator doet er niet meer toe,

- Maxstring zal nog steeds blijven optreden indien replace een blok van 32000 vergroot naar meer dan 32768

Options
    • -13
    • +169
    ./Test.cls.xml
Tuesday 08 Jul 2014
4:47 pm

JoC

TECH.StreamUtils
Options
    • -0
    • +38
    ./Test.cls.xml
  2. … 3 more files in changeset.