CacheAdminA_trunk2010/vhUnitTest/APPS/Halux/common/impl/ProductieGroepBepaler/ProductieOpstartfaseBepalerTAX

IsInOpstartfase

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 20 Apr 18
Commits this week: 0
Total Lines of Code (LoC): 46
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 30 Sep 2022
11:40 am

Tommy Hebb

[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

Options
    • -12
    • +0
    ./Test.cls.xml
  2. … 130 more files in changeset.
Tuesday 16 Mar 2021
4:47 pm

Tommy Hebb

4:46 pm

Tom Vermeulen

Thursday 11 Mar 2021
4:43 pm

Tommy Hebb

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.

3:13 pm

Tommy Hebb

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?

8:57 am

Wim Vermeulen

Wednesday 10 Mar 2021
7:51 pm

Wim Vermeulen

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

7:29 pm

Wim Vermeulen

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

7:17 pm

Wim Vermeulen

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) !

Tuesday 09 Mar 2021
3:24 pm

Tommy Hebb

[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

Options
    • -40
    • +15
    ./Test.cls.xml
  2. … 6 more files in changeset.
Monday 01 Mar 2021
9:55 am

Tom Vermeulen

ineens een Set van maken, lijn wordt korter en leesbaarder

ineens een Set van maken, lijn wordt korter en leesbaarder

Friday 26 Feb 2021
3:00 pm

Tommy Hebb

[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
Monday 22 Feb 2021
5:29 pm

Tommy Hebb

[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

Options
    • -11
    • +46
    ./Test.cls.xml
  2. … 5 more files in changeset.
12:49 pm

Tommy Hebb

[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

Options
    • -2
    • +2
    ./Test.cls.xml
  2. … 1 more file in changeset.
Wednesday 30 Sep 2020
8:34 am

Peter Vercoutter

[ICT-1838] TAX - productiegroep - nieuwe kleuren niet meer in 'opstartfase'
Options
    • -66
    • +0
    ./Test.cls.xml
  2. … 1 more file in changeset.
Tuesday 18 Aug 2020
4:51 pm

Tommy Hebb

[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

Options
    • -53
    • +54
    ./Test.cls.xml
  2. … 1 more file in changeset.
Thursday 26 Sep 2019
10:35 am

Wim Vermeulen

10:35 am

Wim Vermeulen

Ja, idd. Nadat de nieuwe matten (ago-Flock) volledig ingefaseerd zijn.

Ja, idd.
Nadat de nieuwe matten (ago-Flock) volledig ingefaseerd zijn.

Wednesday 25 Sep 2019
10:56 am

Martijn Vierstraete

10:56 am

Martijn Vierstraete

Wordt de code ivm wolvilt mettertijd weggewerkt?

Wordt de code ivm wolvilt mettertijd weggewerkt?

10:51 am

Martijn Vierstraete

[ICT-684] Uitfaseren wolvilt matjes
[ICT-684] Uitfaseren wolvilt matjes
Friday 13 Sep 2019
1:46 pm

Wim Vermeulen

[ICT-684] Uitfaseren wolvilt matjes

- Sifonlade is niet meer in Opstartfase (methodnaam aangepast)

Options
    • -1
    • +1
    ./Test.cls.xml
Wednesday 11 Sep 2019
12:20 pm

Martijn Vierstraete

10:30 am

Wim Vermeulen

[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)

Options
    • -1
    • +1
    ./Test.cls.xml
  2. … 1 more file in changeset.
Wednesday 28 Aug 2019
9:08 am

Tom Vermeulen

Monday 26 Aug 2019
3:41 pm

Martijn Vierstraete

[ICT-803] Kantenbandprogramma's worden niet ingevuld bij BrutEik: fineeronbehandeld wegwerken
[ICT-803] Kantenbandprogramma's worden niet ingevuld bij BrutEik: fineeronbehandeld wegwerken
12:07 pm

MartijnV

[ICT-803] Kantenbandprogramma's worden niet ingevuld bij BrutEik: fineeronbehandeld wegwerken
Options
    • -2
    • +2
    ./Test.cls.xml
  2. … 18 more files in changeset.
Monday 21 Jan 2019
11:15 am

MartijnV

[VRB600] FineerOnbehandeld in productiegroep specialsPM
Options
    • -0
    • +12
    ./Test.cls.xml
  2. … 10 more files in changeset.
Thursday 29 Nov 2018
1:07 pm

Kurt Callebaut

Wednesday 28 Nov 2018
4:07 pm

Wim Vermeulen