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

[ICT-1136] TRANSP Transsmart : aanroepen webserviceclient vanuit de TransportService

- review opm

  1. … 6 more files in changeset.
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...

[ICT-1136] TRANSP Transsmart : aanroepen webserviceclient vanuit de TransportService

- Unittest AnnuleerLevering

  1. … 2 more files in changeset.
[ICT-1136] TRANSP Transsmart : aanroepen webserviceclient vanuit de TransportService

- opkuis wlips

[ICT-1136] TRANSP Transsmart : aanroepen webserviceclient vanuit de TransportService

- Diverse aanpassingen en refactors om code testbaar te maken.

  1. … 11 more files in changeset.
[ICT-1136] TRANSP Transsmart : aanroepen webserviceclient vanuit de TransportService

- response als transient gezet, maar wel request en response als stream opslaan

  1. … 2 more files in changeset.
[ICT-1136] TRANSP Transsmart : aanroepen webserviceclient vanuit de TransportService

- Bug bij verwijderen van verzenddata indien een zending wordt aangepast (verwijderen en terug toevoegen).

[ICT-1136] TRANSP Transsmart : aanroepen webserviceclient vanuit de TransportService

- Error meldingen verbeteren.

- Exceptions door de Webservice client -> door ict op te lossen fout

- De rest heeft omschrijving in de response van Transsmart -> zelf of bidi actie ondernemen.

  1. … 3 more files in changeset.
Perfection! of toch niet want casing niet ok

Perfection! of toch niet want casing niet ok

comment nog nodig?????????????

comment nog nodig?????????????

%IsA() gebruiken

%IsA() gebruiken

commentaar: commentaar

commentaar: commentaar

comment no nodig?

comment no nodig?

TODO comment

TODO comment