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
[UST3558] VHIP904 receptie goederen: alle betrokken producten locken

- bijkomende test converter op gedrag indien exceptie bij productbepaling

    • -0
    • +15
    ./Testafhandelingexceptietijdensbepalentelockenproducten/Journaal1lijn.xml
  1. ./Testafhandelingexceptietijdensbepalentelockenproducten
  2. … 1 more file in changeset.
[UST3558] VHIP904 receptie goederen: alle betrokken producten locken

- Locker inbouwen

- tests aanpassen + bijkomende tests

  1. ./Testafhandelingindiennietgelockedkanworden
    • -0
    • +15
    ./Testafhandelingindiennietgelockedkanworden/Journaal1lijn.xml
  2. … 5 more files in changeset.
[UST3529] VHIP904 Interface artikelontvangstjournaal - vervoersreferentie

- review opmerkingen : bijkomende testen.

  1. … 3 more files in changeset.
  • 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: ontvangst: error tekst indien geen klant of leverancier

- bericht niet verwerken indien bezwaren, maar bezwaren als tekst terug geven

- geefbezwaren in aparte klasse toegevoegd

- bijbehorende tests voor elke validatie

    • -0
    • +14
    ./Testafhandelingindienfout/Journaal1lijn.xml
  1. … 3 more files in changeset.
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..

[UW411]VHIP904 : Receptie goederen : Receptienummer bepalen als JournalId + volgnummer

- de receptienummer moet bepaald worden adhv Onvangstjournaal nummer. postfix in functie van hoeveel recepties er zijn, max 10 (0 tem 9)

    • -1
    • +1
    ./TestindienLegalentityontbreektinberichtwordteenexceptiegesmeten/Journaal1lijn.xml
    • -1
    • +1
    ./Test2lijnenverschillendeleverancierMaakt2toeleveringen2recepties/Journaal2lijnenverschillendeleverancier.xml
  1. ./Test11verschillendeleveranciersmetreceptiegeeftexceptie
    • -1
    • +1
    ./Test2lijnenzelfdeleverancierzelfdeproductmaakt1Toeleveringen1receptie/Journaal2lijnenZelfdeProduct.xml
    • -1
    • +1
    ./TestmaakttoeleveringenGEENreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct/Journaal1lijn.xml
    • -1
    • +1
    ./Test2lijnenverschillendproductzelfdeleveranciermaakt1Toeleveringen1receptie/Journaal2lijnen.xml
    • -1
    • +1
    ./Teststandaardgedragmaakttoeleveringenreceptieaanengeefteeninventsyncberichtterug/Journaal1lijn.xml
    • -0
    • +83
    ./Test11verschillendeleveranciersmetreceptiegeeftexceptie/Journaal11leveranciers.xml
  2. … 14 more files in changeset.
[UST2945] VHIP904: Ontvangst: Levering van producten met OpslagZone werkvloer
[UST2945] VHIP904: Ontvangst: Levering van producten met OpslagZone werkvloer
[UST2945] VHIP904: Ontvangst: Levering van producten met OpslagZone werkvloer

- testnaam aangepast

  1. ./TestmaakttoeleveringenGEENreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct
  2. ./TestmaakttoeleveringenreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct
    • -0
    • +14
    ./TestmaakttoeleveringenGEENreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct/Journaal1lijn.xml
    • -14
    • +0
    ./TestmaakttoeleveringenreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct/Journaal1lijn.xml
  3. … 1 more file in changeset.
  • More
  • CR-578
  • finished reviewing
[UST2945] VHIP904: Ontvangst: Levering van producten met OpslagZone werkvloer

- ook bepalen of een receptie moet aangemaakt worden adhv geleverd product

  1. ./TestmaakttoeleveringenreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct
    • -0
    • +14
    ./TestmaakttoeleveringenreceptieaanengeefteeninventsyncberichtterugindienWerkvloerProduct/Journaal1lijn.xml
  2. … 5 more files in changeset.