CacheAdminA_trunk2010/vhTest/Fake/APPS/Halux/PPS/Activiteit/impl

TAOR

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 8
Last Commit: 14 Sep 15
Commits this week: 0
Total Lines of Code (LoC): 308
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 16 Dec 2022
2:17 pm

Joannes Lemmens

Tuesday 13 Dec 2022
5:48 pm

Jelle Baele

5:38 pm

Jelle Baele

Ik vind dit persoonlijk verwarrend, aangezien 'MeenemenInfo' 4 lijnen hoger als leeg wordt geïnitialiseerd

Ik vind dit persoonlijk verwarrend, aangezien 'MeenemenInfo' 4 lijnen hoger als leeg wordt geïnitialiseerd

5:36 pm

Jelle Baele

Is een persoonlijke keuze, maar ik zet alle public methods steeds bovenaan

Is een persoonlijke keuze, maar ik zet alle public methods steeds bovenaan

Monday 12 Dec 2022
11:16 am

Joannes Lemmens

[ICT-3162] TAX - inhoud van tussenetiket wijzigen
[ICT-3162] TAX - inhoud van tussenetiket wijzigen
Friday 09 Dec 2022
8:09 am

Joannes Lemmens

[ICT-3162] TAX - inhoud van tussenetiket wijzigen

- ToeleveringAPI, ProductAPI en IngegevenKenmerkenAPI doorgegeven vanuit LijstVerwerker-klasse

- Unit test aangepast zodat deze API's worden opgezet

Options
    • -12
    • +6
    ./LijstVerwerkers/KlantEtiketten.cls.xml
  2. … 4 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) !

10:52 am

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 => ...

- Waar nodig wordt nu het ProductieSequentieItem mee doorgegeven

- Via ProductieSequentieItem en ProductTypeApi wordt het product en daarop de behavior HeeftFrontHoogteVerstelling opgehaald

- HeeftFrontHoogteVerstelling wordt nu gebruikt om juiste ZijkantLijmProgramma te bepalen

- Bijhorende TestCases aangepast

Options
    • -4
    • +4
    ./Zijkant/KamerFreesIsoProgrammaBepaler.cls.xml
  2. … 12 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...
Tuesday 18 Aug 2020
11:38 am

Laura Landuyt

Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequent...

Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequentienummer!?

11:34 am

Laura Landuyt

juist een bedenking. Moet deze test ook voor klantetiketten?

juist een bedenking. Moet deze test ook voor klantetiketten?

Thursday 06 Aug 2020
11:18 am

Tom Vermeulen

[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
Thursday 30 Jul 2020
8:25 am

Tom Vermeulen

[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken

- KlantEtiketten Verwerker uitmockbaar maken

Options
    • -0
    • +61
    ./LijstVerwerkers/KlantEtiketten.cls.xml
  2. … 2 more files in changeset.
Wednesday 29 Jul 2020
2:34 pm

Tom Vermeulen

[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken

- KlantEtiketWerklijstItemMaker uitmockbaar maken

Options
    • -0
    • +25
    ./Verpakking/KlantEtiketWerklijstItemMaker.cls.xml
  2. ./Verpakking
  3. … 3 more files in changeset.
Friday 22 Nov 2019
10:08 am

Martijn Vierstraete

Monday 04 Nov 2019
10:48 am

Tom Vermeulen

Thursday 17 Oct 2019
2:53 pm

Martijn Vierstraete

[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
Friday 11 Oct 2019
8:15 am

MartijnV

[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug): kantenbandkleur bepalen
Options
    • -9
    • +2
    ./common/KantenbandDataKleurBepaler.cls.xml
  2. … 3 more files in changeset.
Wednesday 07 Aug 2019
2:40 pm

MartijnV

[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug): KleurBepaler voor LadeInfo: rename methods
Options
    • -2
    • +2
    ./common/KantenbandDataKleurBepaler.cls.xml
  2. … 2 more files in changeset.
Tuesday 23 Jul 2019
5:16 pm

MartijnV

[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug): KleurBepaler voor LadeInfo
Options
    • -0
    • +26
    ./common/KantenbandDataKleurBepaler.cls.xml
  2. … 4 more files in changeset.
Friday 26 Apr 2019
2:09 pm

Tom Vermeulen

Reverse merged revision(s) 56420 from trunk2010:

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

revert van de revert omdat mijn wijzigingen wel goed waren, maar de rest van de ventjes niet :)

Options
    • -2
    • +1
    ./OpdeelZaagAansturingActieStatusUpdater.cls.xml
  2. … 65 more files in changeset.
1:53 pm

Tom Vermeulen

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

Options
    • -1
    • +2
    ./OpdeelZaagAansturingActieStatusUpdater.cls.xml
  2. … 65 more files in changeset.