CacheAdminA_trunk2010/vhUnitTest/DOM/PM/Maatwerk/Calc/HF/impl/LBX

Halffab

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 24 May 18
Commits this week: 0
Total Lines of Code (LoC): 303
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
Tuesday 17 Sep 2019
11:58 am

KurtC

[ICT-853] fix - sgeleider (falende test na wijzigingen in kenmerkeneditor.)

- terug activeren.

Options
    • -0
    • +19
    ./HalffabGeleider/Test.cls.xml
Thursday 12 Sep 2019
9:48 am

Kurt Callebaut

9:44 am

Wim Vermeulen

9:30 am

Kurt Callebaut

ICT-853: Opkuis sgeleider wijzigingen, na falende test
ICT-853: Opkuis sgeleider wijzigingen, na falende test
7:53 am

KurtC

[ICT] fix - sgeleider (falende test na wijzigingen in kenmerkeneditor.)
Options
    • -18
    • +0
    ./HalffabGeleider/Test.cls.xml
  2. … 1 more file in changeset.
Monday 04 Jun 2018
5:18 pm

Tommy Hebb

5:11 pm

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- Na review => refactor UnitTests

Options
    • -84
    • +56
    ./HalffabGeleider/Test.cls.xml
4:07 pm

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- Na review => liquideren dubbele negatie => DezeHalffabDientEnkelVoorDezeGeleiderTechnologie => IsHalffabEenSGeleiderVoorBlumotionEnTOB

- Na review => DeAndereGeleiderTechnologie => enum als type => DOM.PM.enu.LBXGeleiderTechnologie

- Na review => in UnitTest => Array.Count ipv Object.Count en juiste datatypes

Options
    • -3
    • +3
    ./HalffabGeleider/ArrayMatcher.cls.xml
  2. … 1 more file in changeset.
Friday 01 Jun 2018
12:14 am

Wim Vermeulen

12:13 am

Wim Vermeulen

Is exact dezeflde implementatie als de vorige method, behalve ..#ProductId1. Ik heb al een zeer geslaagde poging gedaan om de volledige UT te herschrijven. Misschien een goede gelegenheid om er in...

Is exact dezeflde implementatie als de vorige method, behalve ..#ProductId1.

Ik heb al een zeer geslaagde poging gedaan om de volledige UT te herschrijven.
Misschien een goede gelegenheid om er in pair-programming door te lopen.
Ik zal iets inboeken in uw agenda.

Thursday 31 May 2018
5:06 pm

Wim Vermeulen

#dim ... As DOM.PM.enu...

#dim ... As DOM.PM.enu...

4:08 pm

Wim Vermeulen

beter: Array.Count() i.p.v. Object.Count() idem Verwacht...

beter: Array.Count() i.p.v. Object.Count()

idem Verwacht...

4:03 pm

Wim Vermeulen

Gewoon ter info: Deze test is een mooi voorbeeld, want hier zie beide gevallen naast (onder) elkaar : TOB 270mm is nog steeds de "oude" geleider TOB 300mm is de nieuwe S-geleider, dus de technolog...

Gewoon ter info:

Deze test is een mooi voorbeeld, want hier zie beide gevallen naast (onder) elkaar :
TOB 270mm is nog steeds de "oude" geleider
TOB 300mm is de nieuwe S-geleider, dus de technologie is nu weggelaten.

Daarom is het nice als een test in één oogopslag beide scenario's verifieert

3:55 pm

Wim Vermeulen

Bij nader inzien staat er nu een dubbele negatie, want de laatste lijn van de functie is ook Quit $$$Not( ... ) Dat is vermoeiend om te begrijpen http://subversion02.vanhoecke.be/static/ogdo0b/2sta...

Bij nader inzien staat er nu een dubbele negatie, want de laatste lijn van de functie is ook Quit $$$Not( ... )
Dat is vermoeiend om te begrijpen

Dus als we dit omdraaien naar ..IsHalffabEenSGeleiderVoorBlumotionEnTOB()
dan kunnen we 2x "$$$Not" weglaten.
en bovendien geeft de methodnaam dan veel specifieker aan wat er getest wordt.

3:53 pm

Wim Vermeulen

Dus als je akkoord bent, deze naam wijzigen in ..IsHalffabEenSGeleiderVoorBlumotionEnTOB() en onderaan de $$$Not() wegdoen.

Dus als je akkoord bent, deze naam wijzigen in ..IsHalffabEenSGeleiderVoorBlumotionEnTOB()
en onderaan de $$$Not() wegdoen.

3:51 pm

Wim Vermeulen

hier dus http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

hier dus

Friday 25 May 2018
2:35 pm

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen...
[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen...
2:10 pm

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- Naamgeving => IsNieuweGeleider => MagGeleiderTechnologieWeglaten

- Voorziening igv niet-TIPONBLUMOTION of niet-BLUMOTION

Options
    • -0
    • +13
    ./HalffabGeleider/Test.cls.xml
  2. … 1 more file in changeset.
10:35 am

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- Aanpassing enkel igv GeefOmschrijvingVoorPicking en kleine refactor ter verduidelijking => gebruik van boolean IsNieuweGeleider

- GeefOmschrijvingVoorDocs gerevert naar oorspronkelijke

Options
    • -4
    • +4
    ./HalffabGeleider/Test.cls.xml
  2. … 1 more file in changeset.
Thursday 24 May 2018
3:14 pm

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- HalffabGeleider => kleine refactoring en gebruik variabele DeAndereGeleiderTechnologie ipv hardcoded en GeefProductID ipv property te gebruiken want private

- Testen 4 variaties => BLUMOTION oude en nieuwe geleider en TIPONBLUMOTION oude en nieuwe geleider => met behulp van custom ArrayMatcher voor Kenmerken

Options
    • -0
    • +39
    ./HalffabGeleider/ArrayMatcher.cls.xml
    • -8
    • +110
    ./HalffabGeleider/Test.cls.xml
  3. … 1 more file in changeset.
11:19 am

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- Custom HalffabContextMock

- ZetBerekeningWijze op Dummy BerekeningsSpecificatie

- Falende test in comment om trunk terug groen te zetten

Options
    • -0
    • +121
    ./HalffabGeleider/HalffabContextMock.cls.xml
    • -4
    • +12
    ./HalffabGeleider/Test.cls.xml
  3. … 1 more file in changeset.
10:17 am

Tommy Hebb

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- GeefOmschrijvingVoorDocs uitgebreid => geeft nu TIP-ON BLUMOTION of BLUMOTION enerzijds, of de nieuwe S-geleider anderzijds zijnde geen speciefieke omschrijving

- InjectDependencies => ProductService injectie voor testdoeleinden

- ProductServiceGet getter om te bepalen of geïnjecteerde of echte ProductService moet gebruikt worden

- Eerste test => Opgelet: werkt nog niet wegens gebrek in mock van HalffabContext => GeefBerekenDatum in constructor van echte HalffabContext

Options
  1. ./HalffabGeleider
    • -0
    • +47
    ./HalffabGeleider/Test.cls.xml
  3. ./Halffab
  4. … 1 more file in changeset.