Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merged revision(s) 72255-72256 from trunk2010:

[XS-184] Send shipment notification emails from Schenker to customer

- in dit geval is het de wens van Blum dat de transporteur de mail stuurt, wij geven dus het emailadres door aan de transporteur.

- inkomend bestand van AX Salesorder uitbreiden met MailAddressTrackTrace

- Opslaan op Order

- Ook voorzien op hoofding Bon (KUL)

- Maken dat TransportService dit mee doorgeeft ad RequestDataBouwer in de LeveringInfo

- Email adres op Ontvangeradres van Transsmart doBooking bericht invullen, zodat het mee verstuurd word, en uiteindelijk bij Transporteur terecht komt.

- bijbehorende testen

........

[XS-184] Send shipment notification emails from Schenker to customer

- falende integratie test voor niet meer gebruikte GLS code gefixed.

........

  1. … 19 more files in changeset.
[ICT-2758] nieuw etiket magazijn/vrd

- Verzenddata van een shipment terug kunnen ophalen.

- veldje ID in de global is null, maar de echte sleutel zit in ID1, daarom override van GeefViaFilter, die juiste id veld gaat ophalen (ID1)

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

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

- Repository over de TRanssmart verzenddata zodat deze in testen kan uitgemockt worden

    • -0
    • +63
    ./VerzendDataRepository.cls.xml
  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

goed zo kan je direct gebruiken in je testen

goed zo kan je direct gebruiken in je testen

injecteren.. mocking mogelijk maken voor de testen die JIJ later nog gaat schrijven (of ik http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif) zie ook inj...

injecteren.. mocking mogelijk maken voor de testen die JIJ later nog gaat schrijven (of ik )
zie ook inject volgende methode

Zinloze methode die gewoon wat extra bits kost?

Zinloze methode die gewoon wat extra bits kost?

TransportAPI injecteren, niet van context halen

TransportAPI injecteren, niet van context halen