Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3558] VHIP904 receptie goederen: alle betrokken producten locken
[UST3558] VHIP904 receptie goederen: alle betrokken producten locken
  • 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
Resultaat hangt af van leverancieer, is dus essentieel.

Resultaat hangt af van leverancieer, is dus essentieel.

  • More
  • CR-604
  • finished reviewing
Het feit dat er één aanpassing is en dat er 5 testen moeten aangepast worden dit moet toch één of andere bellentje doen rinkelen dat de structuur toch niet zo 'bien' is -> niet voor nu op te lossen...

Het feit dat er één aanpassing is en dat er 5 testen moeten aangepast worden dit moet toch één of andere bellentje doen rinkelen dat de structuur toch niet zo 'bien' is -> niet voor nu op te lossen , maar te aanschouwen als een verbeterpuntje

Geen goed idee om dit iedere keer te herhalen -> beter één maal in basis test , die het standaard gedrag test , maar voor de rest een stub gebruiken , met eventueel een dummy product repo -> aandac...

Geen goed idee om dit iedere keer te herhalen -> beter één maal in basis test , die het standaard gedrag test , maar voor de rest een stub gebruiken , met eventueel een dummy product repo -> aandacht gaat teveel naar zaken die er niet toe doen in de testen.

!!! Testen moeten duidelijk zijn -> werken is maar een bijzaak , indien ze niet werken moeten ze makkelijk aan te passen zijn -> dus hoe duidelijker hoe beter;

Van een MaakReceptieVoorToelevering zou ik eerder een toelevering object / toeleveringid verwachten en geen differse parameters -> misschien is MaakReceptie voldoende

Van een MaakReceptieVoorToelevering zou ik eerder een toelevering object / toeleveringid verwachten en geen differse parameters -> misschien is MaakReceptie voldoende

GeefProductIdVoorLeverancierId lijkt mij toch wel een rare naamgeving ; misschien is een GeefProductId al voldoende

GeefProductIdVoorLeverancierId lijkt mij toch wel een rare naamgeving ; misschien is een GeefProductId al voldoende

Lijkt mij beter om de productlijn in een variabele te steken en deze hier te gebruiken ipv ProductLijnen.GetAt(1) -> verhoogt de leesbaar -> begrijpbaarheid

Lijkt mij beter om de productlijn in een variabele te steken en deze hier te gebruiken ipv ProductLijnen.GetAt(1) -> verhoogt de leesbaar -> begrijpbaarheid

StandaardToelevering hoofding met flexibele in te stellen data lijkt mij niet echt standaard te zijn -> lijkt mij eerder iets in om in .Fake. te zetten

StandaardToelevering hoofding met flexibele in te stellen data lijkt mij niet echt standaard te zijn -> lijkt mij eerder iets in om in .Fake. te zetten

Opletten met negatieve resultaten in functies ; beter IsReceptieNodig -> dubbele ontkenningen etc..

Opletten met negatieve resultaten in functies ; beter IsReceptieNodig -> dubbele ontkenningen etc..

[UST2945] VHIP904: Ontvangst: Levering van producten met OpslagZone werkvloer
[UST2945] VHIP904: Ontvangst: Levering van producten met OpslagZone werkvloer
  • More
  • CR-578
  • finished reviewing
[UST2918] VHIP904: Ontvangst: goederen voor Halux
[UST2918] VHIP904: Ontvangst: goederen voor Halux
  • 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.