Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() ) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builde...

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() )
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Deze method komt vele malen voor in de Builders (idem voor =Servodrive) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd ...

Deze method komt vele malen voor in de Builders (idem voor =Servodrive)
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt. Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en B...

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt.
Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en BuilderBasis vallen samen.
Maar wat mij betreft hoeft dit niet weggewerkt te worden in caché. Misschien wel bij het porten naar .NET

FYI: het was uiteindelijk de dubbele definitie van de method GeefGekoppeldIKType() die voor enige verwarring zorgde. Vandaar de review opm.

[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep
Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronde...

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronder verifiëren?

overbodige comment?

overbodige comment?

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten
[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
Is er een reden waarom je tests toevoegt en uitschakelt?

Is er een reden waarom je tests toevoegt en uitschakelt?

#dim zonder type is even erg als een Set

#dim zonder type is even erg als een Set

Method mag private (en bij voorkeur geen strings als methodnaam, maar UpperCamelCased) *Personality specifieke httpverzender meegeven i.p.v. extra logica in test (bepaling van httpverzender via o...
  • Method mag private (en bij voorkeur geen strings als methodnaam, maar UpperCamelCased)
  • Personality specifieke httpverzender meegeven i.p.v. extra logica in test (bepaling van httpverzender via objectapi)
Deze klasse test enkel of de dependencies opgeroepen worden: veel setup en weinig (duidelijke) asserts. Dit geeft aan dat hier logica opgesplitst zou moeten worden.

Deze klasse test enkel of de dependencies opgeroepen worden: veel setup en weinig (duidelijke) asserts. Dit geeft aan dat hier logica opgesplitst zou moeten worden.

Kan je ook omschrijven in de testnaam wat het standaard gedrag is?

Kan je ook omschrijven in de testnaam wat het standaard gedrag is?

Injecteren en testen

Injecteren en testen

Wat is de meerwaarde van deze getter-methode?

Wat is de meerwaarde van deze getter-methode?

Afzonderen en testen van TAX en LBX logica

Afzonderen en testen van TAX en LBX logica