ProductieGroepBepaler

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- HeeftFrontHoogteVerstelling uit opstartfase gehaald => alles is nu HeeftFrontHoogteVerstelling = true

- Utils.DOM-kenmerken => nu MoventoKoppeling

- Al wat testjes gefixt

- Nieuwe testbestanden

    • -2
    • +0
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 130 more files in changeset.
Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt. Ik dacht dat een OnBeforeOne...

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt.
Ik dacht dat een OnBeforeOneTest bedoeld is voor als er een gevaar is dat hij door eerdere UTs zou kunnen bevuild worden, wat hier niet het geval is.

Bwa, is een beetje zo gegroeid door refactoren... Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests? Daarna de afweging: Waarom dan nog de call on...

Bwa, is een beetje zo gegroeid door refactoren...
Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests?
Daarna de afweging: Waarom dan nog de call onder test speciaal gaan afzonderen? En bovendien: Waarom beschikken we anders over een $$$AssertTrue en $$$AssertFalse?
Het was bovendien daardoor ook niet meer nodig om een betekenisvolle, extra lokale variabele te voorzien in de UnitTesten (VerwachtInOpstartfase), dus ook al een regel minder per UT, zonder aan leesbaarheid in te boeten (eerder het tegendeel).
Doe de nieuwe versie eens open in studio en zie eens hoe leesbaar die is
Misschien moeten we dan eerder (mettertijd, als die klassen eens onder change komen) die van LBX en TBX aanpassen naar dit model?

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval. Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (pro...

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval.
Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (productieSequentie) toe te voegen.
Na al je inspanningen, keur ik deze oplossing goed

Ik begrijp niet echt waarom je deze "Assert..." method hebt weggewerkt. Dit is immers een algemeen gebruikt principe, o.a. om aan te geven dat de test-methods in deze klasse dezelfde logica oproepe...

Ik begrijp niet echt waarom je deze "Assert..." method hebt weggewerkt.
Dit is immers een algemeen gebruikt principe, o.a. om aan te geven dat de test-methods in deze klasse dezelfde logica oproepen. Hoe eenvoudig deze ook is.

En voor zover consistentie een argument i : de implementatie bij TAX is nu minder consistent met die van LBX en TBX.

Ik vind dit geen drama, hoor. Als je wilt kunnen we hierover van gedachten wisselen

Kleine suggestie : OpstartfaseBepaler voor iedere test(method) newen, dat is ietsje properder --> dus in de OnBeforeOneTest() is beter. Let op: parameter (aTestCase) !

Kleine suggestie :
OpstartfaseBepaler voor iedere test(method) newen, dat is ietsje properder --> dus in de OnBeforeOneTest() is beter.
Let op: parameter (aTestCase) !

[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing (frontverstelmogelijkheid):

- VERVOLG GRONDIGE REFACTOR => Architecturaal is het niet ok dat in productie-aansturing met KlantId gewerkt wordt => ...

- Er werd gebruik gemaakt van MaatwerkProduct-object in TAOR-package => mag/moet TAORLade zijn => is andere en specifiekere interface => MaatwerkProduct kent HeeftFrontHoogteVerstelling niet => is enkel voor TAOR

- KlantId is niet meer nodig voor ProductieOpstartfaseBepalerTAX

- Behavior HeeftFrontHoogteVerstelling op de TaxLade gebruiken.

- Bijhorende TestCases aangepast

    • -21
    • +3
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 6 more files in changeset.
ineens een Set van maken, lijn wordt korter en leesbaarder

ineens een Set van maken, lijn wordt korter en leesbaarder

[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing (frontverstelmogelijkheid):

- KlantId wordt nu meegegeven aan IsInOpstartfase van ProductieOpstartfaseBepaler

- MagKlantTAORFrontHoogteVerstellingBestellen van KlantInstellingenService bepaalt nu via KlantId of TAORLade in opstarfase zit

- Fix en uitbreiding TestCases

    • -12
    • +21
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 5 more files in changeset.
[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing (frontverstelmogelijkheid):

- PalletVerpakking mag uit OpstartFase => In comment gezet, want 1 van volgende commits zal code vervangen door nieuwe opstartfase FrontHoogteVerstelling

    • -1
    • +1
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 1 more file in changeset.
hier nog een (in eerste instantie belachelijk simpele) test voorzien die gewoon checkt of mvx op lijn 4 komt. Het zou kunnen zijn dat er later meer logica aan te pas komt, maar dan is de test al vo...

hier nog een (in eerste instantie belachelijk simpele) test voorzien die gewoon checkt of mvx op lijn 4 komt. Het zou kunnen zijn dat er later meer logica aan te pas komt, maar dan is de test al voorzien (om direct uit te breiden)

aangezien implementatie er vrij dummyachtig uitziet, is deze lijn (momenteel) overbodig

aangezien implementatie er vrij dummyachtig uitziet, is deze lijn (momenteel) overbodig

[ICT-1858] MVX: Productiegroepbepaler en Productielijnbepaler implementeren
[ICT-1858] MVX: Productiegroepbepaler en Productielijnbepaler implementeren
[ICT-1858] MVX: Productiegroepbepaler en Productielijnbepaler implementeren

- Toevoegen productiegroepenbepaler + lijnbepaler implementeren voor MVX

- Toevoegen MVX in de categoriebepaler

- Opstartfasebepaler preventief al toegevoegd voor MVX

- Later testen uitbreiden voor Spoelbaklades en sifonlades

- Dummyklasse voor MVX uitbreiden voor smallelades, en productiewijze Front

    • -0
    • +23
    ./ProductieOpstartfaseBepalerMVX.cls.xml
  1. … 16 more files in changeset.
[ICT-1838] TAX - productiegroep - nieuwe kleuren niet meer in 'opstartfase'
    • -22
    • +0
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 1 more file in changeset.
[ICT-1682] TAX - 3 nieuwe kleuren toevoegen - fase 3 en 4:

- Laden met nieuwe kleuren moeten in opstartfase

- Ineens wat opkuis gedaan en vergeten return types voorzien

- Obsolete testen verwijderd en nieuwe toegevoegd

    • -1
    • +24
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 1 more file in changeset.
[ICT-1682] TAX - 3 nieuwe kleuren toevoegen - fase 3 en 4:

- Voorafgaande opkuis van obsolete implementatie

    • -32
    • +1
    ./ProductieOpstartfaseBepalerTAX.cls.xml
[ICT-1628] TAX: Palletverpakking Productieaansturing

-PalletVerpakking in opstartfase gezet en Flatpack uit opstartfase gehaald

    • -3
    • +3
    ./ProductieOpstartfaseBepalerTAX.cls.xml
[ICT-1628] TAX: Palletverpakking Productieaansturing

-Verpakkingstype pallet toegevoegd voor etiket probox onderdelendoos

-vhTestFiles aangepast

    • -0
    • +8
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 13 more files in changeset.
[UST3902] TAX-SY: ProductieGroep instellen

- Sifonlade is niet meer in Opstartfase : groep is nu SpecialsPM --> APPS.Halux.Machine.OpdeelZaag.enu.Afhandeling is nu manueel (zoals initieel gevraagd was)

    • -1
    • +1
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 1 more file in changeset.
[ICT-803] Kantenbandprogramma's worden niet ingevuld bij BrutEik: fineeronbehandeld wegwerken
[ICT-803] Kantenbandprogramma's worden niet ingevuld bij BrutEik: fineeronbehandeld wegwerken
[ICT-803] Kantenbandprogramma's worden niet ingevuld bij BrutEik: fineeronbehandeld wegwerken
    • -9
    • +0
    ./ProductieOpstartfaseBepalerTAX.cls.xml
  1. … 18 more files in changeset.