Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-701
  • resumed reviewing
Hier ook een paar tabkes...

Hier ook een paar tabkes...

Tabkes my dear Watson, tabkes http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Hier en op meerdere plaatsen in de klasse

Tabkes my dear Watson, tabkes Hier en op meerdere plaatsen in de klasse

[UST3053] VHIP901: ORDER2ADMIN: SOPRs
[UST3053] VHIP901: ORDER2ADMIN: SOPRs
[UST3053] VHIP901: ORDER2ADMIN: SOPRs

- ProductBepaler verplaatst. Deze zijn overbodig geworden.

  1. … 2 more files in changeset.
[UST2855]VHIP904: Ontvangst: SOPR's op de juiste manier omzetten bij verwerken van request

- Method verwijderd

  1. … 2 more files in changeset.
[UST2855]VHIP904: Ontvangst: SOPR's op de juiste manier omzetten bij verwerken van request

- Review : Logica Sopcontractid of niet in productbepaler steken ipv de oproepende code.

  1. … 2 more files in changeset.
Had het nu pas gezien, mee gecommit onder ust2918

Had het nu pas gezien, mee gecommit onder ust2918

  • More
  • CR-575
  • finished reviewing
opkuisen

opkuisen

GeefProductIdVoorLeverancier(IdentNr , LeverancierId, SoprContractId) om deze beslissingen te verplaatsen

GeefProductIdVoorLeverancier(IdentNr , LeverancierId, SoprContractId) om deze beslissingen te verplaatsen

[UST2855]VHIP904: Ontvangst: SOPR's op de juiste manier omzetten bij verwerken van request
[UST2855]VHIP904: Ontvangst: SOPR's op de juiste manier omzetten bij verwerken van request
  • More
  • CR-497
  • finished reviewing
[UST2855]VHIP904: Ontvangst: SOPR's op de juiste manier omzetten bij verwerken van request

- method om productId te bepalen adhv identnummer en soprcontract

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

[ust2726] VHIP904: ontvangst: mapping velden uit XSD bij maken van toelevering

- review opmerkingen : naamgeving

  1. … 3 more files in changeset.
ValideerProductId

ValideerProductId

API overbodig

API overbodig