Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
again niet default behaviour..

again niet default behaviour..

hier ook enkel blue sky testen, niet als webservice fout terug geeft..

hier ook enkel blue sky testen, niet als webservice fout terug geeft..

ahaaaaa hier staat ie http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif well done!

ahaaaaa hier staat ie well done!

Test toevoegen waarbij de projectsetting eens een true terug geeft?

Test toevoegen waarbij de projectsetting eens een true terug geeft?

Is de X ertussen nodig als de projectsetting false is ??? Unit test kan dit miss verduidelijken http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Is de X ertussen nodig als de projectsetting false is ???
Unit test kan dit miss verduidelijken

Ook een test toevoegen met een collli waarvan gewicht 0 is? zoals je op demo hebt gedaan.

Ook een test toevoegen met een collli waarvan gewicht 0 is? zoals je op demo hebt gedaan.

Verduidelijking in naam wat je test ipv gwn Test http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif + ook hier terug enkel het positieve/ blue sky scenario...

Verduidelijking in naam wat je test ipv gwn Test

+ ook hier terug enkel het positieve/ blue sky scenario en niet de andere cases

Volgende methodes moeten eigenlijk bovenaan staan in volgende volgorde: OnBeforeAll, OnBeforeOne,OnAfterOne, OnAfterAll

Volgende methodes moeten eigenlijk bovenaan staan in volgende volgorde: OnBeforeAll, OnBeforeOne,OnAfterOne, OnAfterAll

Opmerking vergelijkbaar met AnnuleerLevering

Opmerking vergelijkbaar met AnnuleerLevering

Naamgeving .. moeten mocks/ stub zijn VerwachtMethodCall --> Mock IndienMethodCall --> Stub Aangezien alles mooi is afgezonderd in aparte private methods (FYI niet allemaal als private gezet, graa...

Naamgeving .. moeten mocks/ stub zijn
VerwachtMethodCall --> Mock
IndienMethodCall --> Stub

Aangezien alles mooi is afgezonderd in aparte private methods (FYI niet allemaal als private gezet, graag aanpassen )
Maar dus ook een test voorzien voor het mogelijkse niet standaard gedrag dat zich zou kunnen voordoen..

Wat als verzenddata leeg is of als de doDelete request een errror geeft!?
Modeleren in testen

Quit setten op een "result" property en deze dan na de if else quitten - aka Wim wisdom

Quit setten op een "result" property en deze dan na de if else quitten - aka Wim wisdom

Beter nog een initialize methode voorzien om alle logica die niet in de constructor mag in te steken

Beter nog een initialize methode voorzien om alle logica die niet in de constructor mag in te steken

spaties voor en na = voor leesbaarheid http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

spaties voor en na =
voor leesbaarheid

Why geen logger injecteren?

Why geen logger injecteren?

nope. controle op transport. als er een fout in zit wordt er een exceptie gesmeten die het afsluiten tegenhoudt zodat de gebruiker weet dat er nog iets mankeert aan het transport...

nope. controle op transport. als er een fout in zit wordt er een exceptie gesmeten die het afsluiten tegenhoudt zodat de gebruiker weet dat er nog iets mankeert aan het transport...

me so happy.

me so happy.

mja. validatiefout met verschillende oorzaken, maar zelfde afhandeling. veel werk om hetzelfde te bereiken.

mja. validatiefout met verschillende oorzaken, maar zelfde afhandeling.
veel werk om hetzelfde te bereiken.

oef, nu ben ik gerustgesteld

oef, nu ben ik gerustgesteld

was idd nodig om de boel te kunnen testen.

was idd nodig om de boel te kunnen testen.

ja. eigenlijk een todo, vraag is of het de moeite van de refactoring waard is, gezien de verwachte levensduur van de code...

ja. eigenlijk een todo, vraag is of het de moeite van de refactoring waard is, gezien de verwachte levensduur van de code...

response controleur is idd in andere story gecommit geweest. etiket service later toegevoegd

response controleur is idd in andere story gecommit geweest.
etiket service later toegevoegd

commentaar verduidelijkt, en ja relevant.

commentaar verduidelijkt, en ja relevant.