CacheAdminA_trunk2010/vhUnitTest/APPS/EC/Winkelkar/impl/ExterneWinkelkarService/KenmerkenAanpasser

LBX

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 13 Mar 17
Commits this week: 0
Total Lines of Code (LoC): 34
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
Wednesday 15 Mar 2017
1:55 pm

Kurt Callebaut

1:55 pm

KurtC

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- Test voor WinkelkarAanpasser.

Options
    • -0
    • +1
    ./Test.cls.xml
  2. … 5 more files in changeset.
11:52 am

KurtC

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- extra validaties.

Options
    • -0
    • +4
    ./Test.cls.xml
11:51 am

Kurt Callebaut

En als ik ze er expliciet bij zet, dan zijn ze waarschijnlijk overbodig. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

En als ik ze er expliciet bij zet, dan zijn ze waarschijnlijk overbodig.

11:47 am

KurtC

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- move class

Options
    • -29
    • +0
    ./TestPasAan.cls.xml
    • -0
    • +29
    ./Test.cls.xml
11:43 am

KurtC

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- overbodige lijn code

Options
    • -2
    • +0
    ./TestPasAan.cls.xml
Monday 13 Mar 2017
5:17 pm

Jo Claes

  • More
  • CR-126
  • finished reviewing
5:16 pm

Jo Claes

Ik zou echter alles testen door de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser ipv expliciet de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser.LBX aan te ro...

Ik zou echter alles testen door de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser ipv expliciet de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser.LBX aan te roepen.
Nu ontbreken we de stap dat de KenmerkenAanpasser de juiste aanpassing doet indien het om LBX gaat.

5:13 pm

Jo Claes

Test is voldoende ipv TestPasAan cfr TAX equivalent

Test is voldoende ipv TestPasAan cfr TAX equivalent

5:12 pm

Jo Claes

Klasse kan best hetzelfde overervings traject volgen als de TAX equivalent dan is de LBXEnum niet nodig

Klasse kan best hetzelfde overervings traject volgen als de TAX equivalent dan is de LBXEnum niet nodig

5:11 pm

Jo Claes

privatiseren ( maar zal uiteindelijk weg mogen ) daar de eerste versie van deze test wel duidelijker was

privatiseren ( maar zal uiteindelijk weg mogen ) daar de eerste versie van deze test wel duidelijker was

5:10 pm

Jo Claes

Eerste versie van de test leek mij wel iets duidelijker .. ik zou er wel een extra arrange-assert in zetten do $$$AssertTrue(LBXKenmerken.NietMeeleveren.Find(Rol)) ; hiermee verzeker je dat de rol...

Eerste versie van de test leek mij wel iets duidelijker .. ik zou er wel een extra arrange-assert in zetten

do $$$AssertTrue(LBXKenmerken.NietMeeleveren.Find(Rol)) ; hiermee verzeker je dat de rol er initieel in zat.

Do Actie

do $$$AssertFalse(LBXKenmerken.NietMeeleveren.Find(Rol)) ; hiermee verzeker je dat de rol er uit is.

5:01 pm

Jo Claes

??????

??????

3:11 pm

Kurt Callebaut

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop- afdekkaprugwand
[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop- afdekkaprugwand
2:47 pm

KurtC

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- Test met APPS ipv DOM LBXKenmerken

Options
    • -3
    • +14
    ./TestPasAan.cls.xml
2:23 pm

KurtC

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- afdekkaprugwandhouder niet tonen op webshop test.

Options
  1. ./LBX
    • -0
    • +20
    ./TestPasAan.cls.xml