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

ArrayMatcher.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 04 Jun 18
Commits this week: 0
Total Lines of Code (LoC): 39
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
Monday 04 Jun 2018
5:18 pm

Tommy Hebb

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
    ./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...
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
    ./ArrayMatcher.cls.xml
  2. … 2 more files in changeset.