CacheAdminA_trunk2010/vhUnitTest/APPS/Halux/PPS/Activiteit/impl/TAOR

Zijkant

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 7
Last Commit: 13 Jul 15
Commits this week: 0
Total Lines of Code (LoC): 217
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
Thursday 01 Jul 2021
9:29 am

Tom Vermeulen

Wednesday 23 Jun 2021
5:19 pm

Tommy Hebb

Tuesday 22 Jun 2021
3:18 pm

Tom Vermeulen

yup stom probeersel om er onderuit te komen: 'k moest het rap in productie wijzigen voor die batch en zo moest ik minder typen. En dat is dan gebackport http://subversion02.vanhoecke.be/static/ogd...

yup

stom probeersel om er onderuit te komen: 'k moest het rap in productie wijzigen voor die batch en zo moest ik minder typen. En dat is dan gebackport

2:05 pm

Tommy Hebb

Mijnen toer om een beetje te muggenziften http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif Is het niet properder (en aangewezen volgens onze coding conven...

Mijnen toer om een beetje te muggenziften Is het niet properder (en aangewezen volgens onze coding conventions) dat het voluit geschreven wordt, dus: $Extract?

1:49 pm

Tom Vermeulen

[ICT-2409] tekst te lang op APPS.Halux.PPS.Activiteit.impl.common.dto.ZijkantKamerFreesData

- testjes vergeten committen

Options
    • -0
    • +31
    ./KamerFreesDataPrintTekstAanvuller/VulAan/Test.cls.xml
1:49 pm

Tom Vermeulen

[ICT-2409] tekst te lang op APPS.Halux.PPS.Activiteit.impl.common.dto.ZijkantKamerFreesData
[ICT-2409] tekst te lang op APPS.Halux.PPS.Activiteit.impl.common.dto.ZijkantKamerFreesData
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) !

3:03 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 => ...

- Bijhorende TestCase aangepast

- TestCase verwijderd wegens dubbele implementatie => waarschijnlijk overgebleven na een move class => ZijkantLijmProgrammaBepaler zit al een tijdje in een TAORProductie-package

Options
    • -28
    • +40
    ./KamerFreesIsoProgrammaBepaler/Test.cls.xml
  2. … 3 more files in changeset.
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
    • -1
    • +0
    ./KamerFreesDataAanvullerTestBase.cls.xml
    • -6
    • +4
    ./KamerFreesDataProgrammaAanvuller/VulAan/Test.cls.xml
    • -15
    • +2
    ./VerwerkingIDBepaler/BepaalVerwerkingID/Test.cls.xml
  4. … 10 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...
Wednesday 24 Feb 2021
4:54 pm

Tommy Hebb

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

- Rechtzetting in bestaande TestCases => 2e parameter van VulAan in KamerZijkantFreesDataAanvuller is een WerklijstItemMakerInput-object en niet een ProductieSequentieItem ==> is nog een tussenlaagje

- TestBase naargelang aangepast alsook de gebruikers ervan

- UnitTest gefixt waar KlantId van ProductieSequentieItem afgehaald wordt als 2e parameter van verschillende methods op ProgrammaBepaler

Options
    • -6
    • +1
    ./KamerFreesDataPickingAanvuller/VulAan/Test.cls.xml
    • -6
    • +8
    ./KamerFreesDataProgrammaAanvuller/VulAan/Test.cls.xml
    • -1
    • +1
    ./KamerFreesDataAanvuller/VulAan/Test.cls.xml
    • -0
    • +11
    ./KamerFreesDataAanvullerTestBase.cls.xml
    • -1
    • +1
    ./KamerFreesDataDimensiesAanvuller/VulAan/Test.cls.xml
3:40 pm

Tommy Hebb

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

- ZijkantLijmProgrammaBepaler => BepaalProgramma => gebruikt nu MagKlantTAORFrontHoogteVerstellingBestellen van de KlantInstellingenService om via KlantId juiste programma te geven => Indien True => X ipv L, Y ipv R, G ipv C, H ipv D

- Klassen die BepaalProgramma van ZijkantLijmProgrammaBepaler gebruiken => geven nu KlantId mee door => van ProductieSequentieItem afgehaald

- Aanpassing en uitbreiding TestCases

- Mock met verkeerde package-naam (wellicht is klasse zelf ooit gemoved) verwijderd en het gebruik ervan vervangen door de juiste, die ook al bestond (automatisch gegenereerd)

Options
    • -2
    • +15
    ./VerwerkingIDBepaler/BepaalVerwerkingID/Test.cls.xml
  2. … 11 more files in changeset.
1:17 pm

Tommy Hebb

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

- Louter voorafgaande refactoring => Naamgeving ===> LijmProgrammaBepaler => ZijkantLijmProgrammaBepaler

Options
    • -7
    • +7
    ./VerwerkingIDBepaler/BepaalVerwerkingID/Test.cls.xml
1:09 pm

Tommy Hebb

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

- Louter voorafgaande refactoring => Casing, spaties, naamgeving

Options
    • -38
    • +39
    ./VerwerkingIDBepaler/BepaalVerwerkingID/Test.cls.xml
Tuesday 10 May 2016
6:17 pm

MartijnV

[UST1629] TAX: Correcte dimensies op zaagetiket

- eindafmetingen vermelden op kamerfrees etiketten

Options
    • -2
    • +2
    ./KamerFreesDataPrintTekstAanvuller/VulAan/Test.cls.xml
  2. … 19 more files in changeset.
Friday 06 May 2016
4:18 pm

MartijnV

[UST1619] TAX VR PAS: Kamerfrees progr. bepaling

- test verplaatst

- ongebruikte (test)Lade klasse verwijderd

Options
  1. ./KamerFreesIsoProgrammaBepaler/General
    • -0
    • +163
    ./KamerFreesIsoProgrammaBepaler/Test.cls.xml
    • -35
    • +0
    ./KamerFreesIsoProgrammaBepaler/General/Test.cls.xml
    • -45
    • +0
    ./KamerFreesIsoProgrammaBepaler/General/Lade.cls.xml
4:01 pm

MartijnV

[UST1619] TAX VR PAS: Kamerfrees progr. bepaling

- ZijkantLijmProgrammaBepaler logica apart getest

- aangepaste tests voor KamerFreesIsoProgrammaBepaler: bepalers worden opgeroepen + isoprogrammanaam wordt correct opgebouwd

Options
    • -54
    • +112
    ./KamerFreesIsoProgrammaBepaler/General/Test.cls.xml
  2. … 3 more files in changeset.
10:56 am

MartijnV

[UST1619] TAX VR PAS: Kamerfrees progr. bepaling (na review)
Options
    • -5
    • +3
    ./KamerFreesIsoProgrammaBepaler/General/Test.cls.xml
  2. … 5 more files in changeset.
Wednesday 04 May 2016
8:43 am

MartijnV

[UST1619] TAX VR PAS: Kamerfrees progr. bepaling
Options
    • -1
    • +24
    ./KamerFreesIsoProgrammaBepaler/General/Test.cls.xml
  2. … 9 more files in changeset.
Friday 13 Nov 2015
5:29 pm

JoC

[UST1288] TAX-PAS-duocolor: opdeelzaag

- vanaf heden ook de juiste plaatafwerking

Options
    • -0
    • +20
    ./OpdeelzaagUitvoeringInfoAanvuller/VulAan/Test.cls.xml
Wednesday 14 Oct 2015
1:30 pm

PirminVDB

[UST1258] TAX-PAS-LC: opdeelzaag en kantenband en kamerfrees

- rename testmethod

Options
    • -1
    • +1
    ./KamerFreesDataProgrammaAanvuller/VulAan/Test.cls.xml
Monday 12 Oct 2015
3:46 pm

JoC

[UW178] - Kantenband kleur in kamerfrees data
Options
    • -1
    • +1
    ./KamerFreesDataAanvuller/VulAan/Test.cls.xml
  2. … 17 more files in changeset.