Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-UitgefaseerdeTesten]
    • -36
    • +51
    ./TBX/BouwsteenCreatie/Test.cls.xml
[ICT-2278] [rvWV] PM: Maatwerk: TAX: End2End-testen aanmaken, ter validatie van de builder(s):

- Deze testen zijn deprecated

    • -0
    • +2
    ./GebruikteHalffabItemsVoorPrijsberekening/TestBase.cls.xml
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:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Alles is nu MoventoKoppeling => Utils-kenmerken waren al aangepast, nu deze ook nog

- Verschillende UT stonden nog buiten dienst => terug geactiveerd => slagen allemaal na aanpassingen omtrent bovenstaande

[ICT-2836][rvPVR] LBX BomBol .Net klaarzetten en testen vanuit AX:

- Ladetype 'LEGRABOX pure' is in .Net 'LBX_Pure'

- "MAT Standaard_MatX" is in .Net aangepast door "MAT_Standaard_MatX"

- De rolnamen zelf worden vanuit .Net teruggeven in plaats van de leesbare waarde van de rolnaam voor V1 ladedoos en V1 losse onderdelendoos

[ICT-2359] TBX BOMBOL builders porten naar .NET

- Kosten uit bouwstenen halen want deze worden toch volledig herwerkt -> dit is dus niet relevant om mee te vergelijken

  1. … 12 more files in changeset.
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Toevoegen End2End test voor bouwstenen van een Tbx configuratie met stalen rug te monitoren

  1. … 2 more files in changeset.
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-2359] TBX BOMBOL builders porten naar .NET

- Reviewopmerking

    • -6
    • +10
    ./TBX/BouwsteenCreatie/Test.cls.xml
[ICT-2359] TBX BOMBOL builders porten naar .NET

- End2End testen opzetten zodat de bouwstenen kunnen gemonitord worden na ingebruikname nieuwe berekening bombol

    • -0
    • +151
    ./TBX/BouwsteenCreatie/Test.cls.xml
  1. … 25 more files in changeset.
[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Testen opgesplitst

  1. … 1 more file in changeset.
[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Toevoegen testen die bouwstenen controleert

    • -0
    • +34
    ./SST/BouwsteenCreatie/Test.cls.xml
  1. … 9 more files in changeset.
[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Wlip verwijderd

[ICT-1642] SVG-035 END2END test toevoegen voor verder wijzigingen

-end2end test uitgebreid + write verwijderd in vhTest/Utils/APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur.cls.xml

  1. … 5 more files in changeset.
[ICT-1642] SVG-035 END2END test toevoegen voor verder wijzigingen

-end2end test toegevoegd + kaderdeur opgesplitst in genereerSvg en WriteSvgToFile

    • -0
    • +40
    ./Svg/Kaderdeur/Test.cls.xml
    • -0
    • +68
    ./Svg/Kaderdeur/TestBase.cls.xml
  1. … 11 more files in changeset.
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