This is a list of all comments for CR-126.

Review Summary: No summary

General Comment by Jo Claes on 13 March 2017, 17:15 defect
http://subversion02:8060/cru/CR-126#c289

  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.


----------------------------------------
File: vhUnitTest/APPS/EC/Winkelkar/impl/ExterneWinkelkarService/KenmerkenAanpasser/LBX/TestPasAan.cls.xml
Revision Comment by Jo Claes on 13 March 2017, 17:08 
http://subversion02:8060/cru/CR-126#c286

  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.

  Reply by Kurt Callebaut on 15 March 2017, 11:50
  > En als ik ze er expliciet bij zet, dan zijn ze waarschijnlijk
  > overbodig. :-)

Revision Comment by Jo Claes on 13 March 2017, 17:13 
http://subversion02:8060/cru/CR-126#c288

  Test is voldoende ipv TestPasAan cfr TAX equivalent

Revision Comment by Jo Claes on 13 March 2017, 17:01 defect
http://subversion02:8060/cru/CR-126#c284

  ??????

Revision Comment by Jo Claes on 13 March 2017, 17:10 
http://subversion02:8060/cru/CR-126#c287

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

Revision Comment by Jo Claes on 13 March 2017, 17:03 defect
http://subversion02:8060/cru/CR-126#c285

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


---

ID: CR-126 http://subversion02:8060/cru/CR-126

Title: [DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop- afdekkaprugwand
Statement of Objectives:
Kan je eens nazien of dit functioneel correct is. :-) 

State: Closed

Summary: 

Author: Kurt Callebaut
Moderator: Kurt Callebaut
Reviewers: (0 active, 1 completed*)
     Jo Claes   (*)