Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3839] VHIP903: PickingCompleet: veld toegevoegd ShippingDate:

- Code uit comment gehaald, mag in productie, hoewel AX er nog niets mee doet, maar interface kent het veld al

    • -8
    • +10
    ./MessageParts/AxdEntityPick.cls.xml
  1. … 1 more file in changeset.
[UST3839] VHIP903: PickingCompleet: veld toegevoegd ShippingDate:

- In comment gezet zodat code in productie kan zonder dat AX er klaar voor is en ondertussen op accept kan getest worden door het daar uit comment te halen

    • -10
    • +8
    ./MessageParts/AxdEntityPick.cls.xml
  1. … 2 more files in changeset.
[UST3839] VHIP903: PickingCompleet: veld toegevoegd ShippingDate:
[UST3839] VHIP903: PickingCompleet: veld toegevoegd ShippingDate:
[UST3839] VHIP903: PickingCompleet: veld toegevoegd ShippingDate:

- Upload nieuwe xsd en SharedTypes met nieuw veld ADUShippingDate

    • -0
    • +19
    ./MessageParts/AxdExtTypeADUShippingDate.cls.xml
    • -1
    • +34
    ./MessageParts/AxdEntityPick.cls.xml
  • More
  • CR-774
  • finished reviewing
zal ne restant zijn. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (overige testen hebben ook wlip.)

zal ne restant zijn.
(overige testen hebben ook wlip.)

Algemeen zou ik lange methods opsplitsen in kleinere. zodat de naam van de method verduidelijkt wat er gebeurd. Dan hoeft commentaar niet. bv ControleerTerugnameMaatwerk, etc ...

Algemeen zou ik lange methods opsplitsen in kleinere. zodat de naam van de method verduidelijkt wat er gebeurd. Dan hoeft commentaar niet. bv ControleerTerugnameMaatwerk, etc ...

Geen idee of het belangrijk is, maar bij Journaal staat er 01234

Geen idee of het belangrijk is, maar bij Journaal staat er 01234

Storages op dto's nog opnieuw toe te voegen.

Storages op dto's nog opnieuw toe te voegen.

[UST3198] VHIP904/VHIP903 : ontvangst: error tekst indien geen klant of leverancier
[UST3198] VHIP904/VHIP903 : ontvangst: error tekst indien geen klant of leverancier
[UST3198] VHIP904/VHIP903 : ontvangst: error tekst indien geen klant of leverancier

- nieuwe xsd voor inventsync

    • -48
    • +5
    ./MessageParts/AxdEntityInventSync.cls.xml
    • -21
    • +5
    ./MessageParts/AxdADUMsgOutboundInQuery.cls.xml
    • -23
    • +4
    ./MessageParts/AxdEntityProdRAF.cls.xml
    • -25
    • +6
    ./MessageParts/AxdEntityPick.cls.xml
    • -0
    • +27
    ./MessageParts/AxdExtTypeFreeText.cls.xml
    • -0
    • +22
    ./MessageParts/AxdUnionAxdExtTypeADUCachePickIdRefAxdTypeExtCodeValue.cls.xml
    • -30
    • +10
    ./MessageParts/AxdEntityHeader.cls.xml
    • -22
    • +0
    ./MessageParts/AxdUnionAxdTypeStrAxdTypeExtCodeValue.cls.xml
  • More
  • CR-497
  • finished reviewing
Moeilijk te doen zonder complete integratie test op te zetten. commentaar in InventoryOut convertor gezet met alle converters in het zelfde geval.

Moeilijk te doen zonder complete integratie test op te zetten. commentaar in InventoryOut convertor gezet met alle converters in het zelfde geval.

In standaard gedrag word getest of de converter van de terug te geven response opgeroepen wordt. De mock geeft een voorgedefinieerd antwoord terug. Asserten van dit lijkt mij niet direct zinvol hie...

In standaard gedrag word getest of de converter van de terug te geven response opgeroepen wordt. De mock geeft een voorgedefinieerd antwoord terug.
Asserten van dit lijkt mij niet direct zinvol hier, maar wel in de test van die converter.

is al gebeurd, wsl outdated

is al gebeurd, wsl outdated

anders opgelost.

anders opgelost.

Gelieve ook de response te asserten

Gelieve ook de response te asserten

'k zou hier opteren om de LijnId en ProductId mee te geven zodat je duidelijk aan toont dat het over het zelfde product gaat met verschillende LijnId's

'k zou hier opteren om de LijnId en ProductId mee te geven zodat je duidelijk aan toont dat het over het zelfde product gaat met verschillende LijnId's

Gelieve het resultaat duidelijk te maken in de response

Gelieve het resultaat duidelijk te maken in de response

Gelieve in uw test benamingen ook uw gedrag te beschrijven maw het resultaat bvb Test: 2 Lijnen voor zelfde leverancier worden toegevoegd aan de receptie

Gelieve in uw test benamingen ook uw gedrag te beschrijven maw het resultaat

bvb Test: 2 Lijnen voor zelfde leverancier worden toegevoegd aan de receptie

Op een stub wordt er geen Verifieer() uitgevoerd, die maakt het zeer onduidelijk waar je eigenlijk de nadruk wil op leggen. Het principe is zo : voor het basis gedrag werk je met verwachts , vanaf...

Op een stub wordt er geen Verifieer() uitgevoerd, die maakt het zeer onduidelijk waar je eigenlijk de nadruk wil op leggen.

Het principe is zo : voor het basis gedrag werk je met verwachts , vanaf dat je de nadruk wil leggen op iets dat afwijkt van het standaard gedrag dan is enkel hetgeen je wil testen verwacht en de rest worden dan allemaal stubs of dummy of fakes afhankelijk van welke situatie.

Het belangrijkste is de intentie duidelijk maken en door alles te behandelen op dezelfde manier vergaat hetgeen eigenlijk aandacht nodig heeft in de massa.

ToeleveringId ipv ToeleveringNummer

ToeleveringId ipv ToeleveringNummer

ValideerProductId

ValideerProductId

API overbodig

API overbodig

Array mag niet opgezet worden in de constructor -> moet per call opnieuw geset worden -> meerdere malen callen kan voor raar gedrag zorgen

Array mag niet opgezet worden in de constructor -> moet per call opnieuw geset worden -> meerdere malen callen kan voor raar gedrag zorgen

Als je LeverancierId doorgeeft tot het laagste niveau -> maw verlaten de encapsulatie en dan gaan we over naar de specifieke types die de ge-encapsuleerde code nodig heeft dan moet je tussendoor ni...

Als je LeverancierId doorgeeft tot het laagste niveau -> maw verlaten de encapsulatie en dan gaan we over naar de specifieke types die de ge-encapsuleerde code nodig heeft dan moet je tussendoor niet al teveel datatypes en id wisselen van hot naar her