Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
zie andere review van dit kaartje

zie andere review van dit kaartje

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot Do $$$AssertEquals("O...

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot
Do $$$AssertEquals("Omschrijving in het Nederlands"$$$CRLF"Kleur:VertalingKleur",DescriptionBepaler.GeefDescription(ProductMock,RandomTaal))
Hoe dan ook is dat iets waarover je kan blijven discussiëren: langere lijnen en minder variabelen, of kortere lijnen en meer one-time-use variabelen...
Indien je toch gaat voor meer variabelen: de gewoonte is om de #dim's niet allemaal bovenaan te smijten, maar vlak voordat je ze nodig hebt

Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven. Idem voor andere #dim/Set combinaties in de...

Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven.
Idem voor andere #dim/Set combinaties in deze en de andere test

om goed te zijn zou je ook een test moeten voorzien die laat zien dat er geen Nederlands gebruikt wordt indien de omschrijving in andere taal niet ontbreekt

om goed te zijn zou je ook een test moeten voorzien die laat zien dat er geen Nederlands gebruikt wordt indien de omschrijving in andere taal niet ontbreekt

[ICT-883] UAT 1422: verkeerde tekst in engels document
[ICT-883] UAT 1422: verkeerde tekst in engels document
Variabele moet zijn : VerwachtPercentage

Variabele moet zijn : VerwachtPercentage

[VRB627] VHIP481 - UAT1305- uitval% en nettohoeveelheid doorgeven aan AX:
[VRB627] VHIP481 - UAT1305- uitval% en nettohoeveelheid doorgeven aan AX:
In gedachten voor volgende keer

In gedachten voor volgende keer

Dit is sneller http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Dit is sneller

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

Mondeling besproken

Is contains en bijgevolg de LijstMetUniekeIdentnummers nodig? Je zou kunnen voor de if-else kijken of de LijstMetUniekeBouwstenen de bouwsteen al bevat (via FindPropertyWithValue)

Is contains en bijgevolg de LijstMetUniekeIdentnummers nodig? Je zou kunnen voor de if-else kijken of de LijstMetUniekeBouwstenen de bouwsteen al bevat (via FindPropertyWithValue)

Naamgeving: BouwsteenLijst of Bouwstenen (idem voor iterator)

Naamgeving: BouwsteenLijst of Bouwstenen (idem voor iterator)

Je was goe bezig met 'dimmen', dat mag ook voor lists http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Je was goe bezig met 'dimmen', dat mag ook voor lists

[DEF732] PM: VHIP481: BOMBOL: Space Tower: aantal van "middenniveau" niet in rekening gebracht:
[DEF732] PM: VHIP481: BOMBOL: Space Tower: aantal van "middenniveau" niet in rekening gebracht:
  • More
  • CR-338
  • finished reviewing
[UST3614] VHIP481: BOMBOL: Gewicht meegeven
[UST3614] VHIP481: BOMBOL: Gewicht meegeven
  • More
  • CR-891
  • finished reviewing
[UST3515] VHIP481: BOMBOL: extra tests (regressietests)
[UST3515] VHIP481: BOMBOL: extra tests (regressietests)