ProductToeleveringLijn.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST4217] PA-UB-025 Reageren op ToeleveringLijnGewijzigd event

- Reviewopmerking verwerkt (duh)

... DanReturn(2) want 3-1 = 2 !!! en aldus ExactAantalKeer(1) ... of was dit om te zien of ik wel wakker was? N.B.: tis nu 14.53u :-P

... DanReturn(2)
want 3-1 = 2 !!!

en aldus ExactAantalKeer(1)

...
of was dit om te zien of ik wel wakker was?
N.B.: tis nu 14.53u :-P

#dim .. As DOM.AKP.ProductToeleveringLijn , want is het resultaat van vhTest.Utils... in de andere testmethods ook.

#dim .. As DOM.AKP.ProductToeleveringLijn , want is het resultaat van vhTest.Utils...
in de andere testmethods ook.

waarde 222222 lijkt me eerder de ID voor de Toelevering. Je kiest natuurlijk wat je maar wil, alleen wordt het dan nogal verwarrend achteraf. Misschien beter gewoon "100" ofzo pakken? Oftewel is d...

waarde 222222 lijkt me eerder de ID voor de Toelevering.
Je kiest natuurlijk wat je maar wil, alleen wordt het dan nogal verwarrend achteraf. Misschien beter gewoon "100" ofzo pakken?

Oftewel is de MethodNaam "GeefID" niet juist gekozen?

[UST4217] PA-UB-025 Reageren op ToeleveringLijnGewijzigd event
[UST4217] PA-UB-025 Reageren op ToeleveringLijnGewijzigd event
[UST4217] PA-UB-025 Reageren op ToeleveringLijnGewijzigd event

- WijzigAantalDnaCodesVoorToeleveringLijn toegevoegd die al rekening houdt met een teveel aan lijnen

  1. … 3 more files in changeset.
Done!

Done!

try-catch toevoegen en lokaal afhandelen (warning ofzo, maar geen error verder throwen)

try-catch toevoegen en lokaal afhandelen (warning ofzo, maar geen error verder throwen)

misschien nog wat testjes voor het geval de bon zélf niet bestaat

misschien nog wat testjes voor het geval de bon zélf niet bestaat

Best niet in de constructor zetten, maar in een aparte method. In dit concreet geval maakt dat natuurlijk niet echt uit, maar 't is "good practice".

Best niet in de constructor zetten, maar in een aparte method. In dit concreet geval maakt dat natuurlijk niet echt uit, maar 't is "good practice".

[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:
[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:
[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:

- GeefAxInventTransId en GeefAxConfigId en GeefAxMaatwerkGenerischItemId toegevoegd met mapping op respectievelijk piece 55 en 56 en 57 van een lijn

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

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

- review : testdata toelevering -> standaardToelevering

    • -0
    • +29
    ./ProductToeleveringLijn.cls.xml
  1. … 2 more files in changeset.
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