ProductieGroepBepaler.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[PRJ051-360] [rvJBA] Zendesk ticket: smalle lade op lijn?

- Productiegroepbepaler moet ook aangepast worden

  1. … 3 more files in changeset.
Verwachte productiegroep is probox maar in testnaam staat er pallet? En ook test voorzien waarbij die in productiegroep specials komt ?

Verwachte productiegroep is probox maar in testnaam staat er pallet?

En ook test voorzien waarbij die in productiegroep specials komt ?

Ook test voorzien waarbij die in productiegroep specials komt ?

Ook test voorzien waarbij die in productiegroep specials komt ?

Mag deze test weg?

Mag deze test weg?

[ICT-1324] Klant Interfurn TBX in specialsKlant
[ICT-1324] Klant Interfurn TBX in specialsKlant
[ICT-1324] Klant Interfurn TBX in specialsKlant

- IsSpecialsKlant Variant-aware gemaakt

  1. … 6 more files in changeset.
Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

[ICT-968][+WV] HX: Planning: Probox-lades voor klant De Vliegher (2856) in -Specials- "Andere productiegroep" laten terechtkomen
[ICT-968][+WV] HX: Planning: Probox-lades voor klant De Vliegher (2856) in -Specials- "Andere productiegroep" laten terechtkomen
[ICT-968][+WV] HX: Planning: Probox-lades voor klant De Vliegher (2856) in -Specials- "Andere productiegroep" laten terechtkomen

SpecialKLT hernoemen naar SpecialsKlant

  1. … 9 more files in changeset.
[ICT-968][+WV] HX: Planning: Probox-lades voor klant De Vliegher (2856) in -Specials- "Andere productiegroep" laten terechtkomen

Toevoegen methode IsSpecialKLT aan ProductieGroepBepaler

De stat juist, Heb ik bewust gewijzigd. Anders krijgen we standaard het gedrag dat spoelbak en smalle lade altijd in spoelbakgroep terecht komen en default gedrag is dat dit niet gebeurd. daarom he...

De stat juist, Heb ik bewust gewijzigd. Anders krijgen we standaard het gedrag dat spoelbak en smalle lade altijd in spoelbakgroep terecht komen en default gedrag is dat dit niet gebeurd. daarom heb ik ook de altijdsplitser moeten toevoegen.
maw indien true, faalde de bestaande testen op gewone aantallen als er geen splitser meegegeven wordt..

(Ik vind het nog steeds een gevaarlijk iets om te beslissen dat de groepering per kenmerk vervangen wordt door groepering op productiegroep, terwijl het in dit geval toevallig hetzelfde is.)

Kritische vraag (misschien om in de groep te gooien) : Is dit een project-setting? --> volgens mij enkel bedoeld om iets nieuw te kunnen "infaseren". Nadien wordt een project-setting opgekuist (in ...

Kritische vraag (misschien om in de groep te gooien) :
Is dit een project-setting? --> volgens mij enkel bedoeld om iets nieuw te kunnen "infaseren". Nadien wordt een project-setting opgekuist (in theorie althans )
Is dit dan eerder een gewone ConfigItem? --> is eigenlijk bedoeld om onderscheid te kunnen maken tussen de verschillende omgevingen (bvb. andere waarde op PROD dan op DEV)
Is het voldoende om deze gewoon als Class Parameter te definieren? --> volgens mij wel. Dit is een single-responsibility klasse, dus een wijziging van de waarde committen en mergen, kan supersnel.

Willen we dit toch wegtrekken uit onze code, dan misschien toch een aparte data-structuur voor maken. Functioneel is config-item hiervoor ook wel geschikt, maar ik vraag me af of we den boel dan niet te veel gaan vervuilen?

Ook testmethod maken met VerwachtMethodCall("IsNodigTeSplitsen" --> dan Return False

Ook testmethod maken met
VerwachtMethodCall("IsNodigTeSplitsen" --> dan Return False

ToeleveringSplitserMock.Verifieer()

ToeleveringSplitserMock.Verifieer()

AltijdSplitser kan best in de constructor, want nu krijg je een nieuwe instantie bij iedere "VoegToe"-call.

AltijdSplitser kan best in de constructor, want nu krijg je een nieuwe instantie bij iedere "VoegToe"-call.

Quit $$$True (want ik zie dat de UT-resultaten gewijzigd zijn, en dat was niet de bedoeling http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

Quit $$$True
(want ik zie dat de UT-resultaten gewijzigd zijn, en dat was niet de bedoeling )

Deze klasse mag weg, indien geen Config-item gebruikt wordt. (zie opmerking bij ToeleveringSplitser)

Deze klasse mag weg, indien geen Config-item gebruikt wordt. (zie opmerking bij ToeleveringSplitser)

Deze klasse is niet meer nodig, denk.

Deze klasse is niet meer nodig, denk.

Deze klasse is niet meer nodig, denk.

Deze klasse is niet meer nodig, denk.

Klein detail : deze lijn onder de ..IngegevenKenmerkenTypeApi plaatsen (volgorde zelfde als de args erboven)

Klein detail : deze lijn onder de ..IngegevenKenmerkenTypeApi plaatsen (volgorde zelfde als de args erboven)

[ICT-817] Productiegroep Specials PM voor meer dan 10 LBX smalle lades

- werken met splitter in productiegroepbepaler.

  1. … 11 more files in changeset.
ICT-817: Productiegroep Specials PM voor meer dan 10 LBX smalle lades
ICT-817: Productiegroep Specials PM voor meer dan 10 LBX smalle lades
[ICT-817] Productiegroep Specials PM voor meer dan 10 LBX smalle lades

- Voor alle productiegroepbepalers

  1. … 5 more files in changeset.