WS.Vhisie4.KlantDocument.KlantDocumentService.txt

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1082] [rvTVE] EC: VH4: MP: Hahle multipersonality (Finland)
[ICT-1082] [rvTVE] EC: VH4: MP: Hahle multipersonality (Finland)
Moet blijkbaar ook niet wegens deprecated.

Moet blijkbaar ook niet wegens deprecated.

op de zelfde plaats als al de andere mp's http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (nergens dus alleen VH wordt getest.)

op de zelfde plaats als al de andere mp's (nergens dus alleen VH wordt getest.)

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

Opkuis van deze klasse is wenselijk.

Opkuis van deze klasse is wenselijk.

Probleempje met de TAB toets? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

Probleempje met de TAB toets?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Is deze test overbodig?

Is deze test overbodig?

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Waar wordt dit getest?

Waar wordt dit getest?

Unishop is uitgefaseerd --> renamen

Unishop is uitgefaseerd --> renamen

BeslagTechnikk ontbreekt

BeslagTechnikk ontbreekt

Veel duplicate code, ik maak een aparte issue aan

Veel duplicate code, ik maak een aparte issue aan

[ICT-774] EC: MP: BeslagTeknikk multipersonality opzetten
[ICT-774] EC: MP: BeslagTeknikk multipersonality opzetten
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