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

Review Summary: No summary


----------------------------------------
File: WSimpl/Vhisie4/Winkelkar/WinkelkarService/BewaarProductLijn.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 10:29 
http://subversion02:8060/cru/CR-4#c6

  Het lijkt mij voorlopig beter om de
  APPS.EC.Verkoop.impl.Externe.VerkoopService te gebruiken -
  kwestie om alles lang de verkoopservice te laten lopen

Revision Comment by Jo Claes on 05 January 2017, 10:05 
http://subversion02:8060/cru/CR-4#c4

  Ik zou een algemene klasse introduceren
  WSimpl...WinkelkarService.ExterneIdControleur die zowel de
  controle doet op offerte en winkelkar id

----------------------------------------
File: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/BewaarProductLijn/Test.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 10:54 
http://subversion02:8060/cru/CR-4#c11

  Algemenere opermerking : 
  
  De method VoegToeProduct geeft een resultaat terug dus de mock
  dient ook een resultaat terug te geven en dan is noodzakelijk
  om uw test aan te passen zodat het resultaat wordt verwerkt.

  Reply by Kurt Callebaut on 07 January 2017, 13:06
  > Method BewaarProductLijn(BewaarProductLijnRequest As
  > WS.Vhisie4.Winkelkar.BewaarProductLijnRequest) geeft geen
  > returnvalue, maw met het resultaat VoegToeproduct wordt niks
  > gedaan. Dus ook niet nodig om dit te testen.

Revision Comment by Jo Claes on 05 January 2017, 11:04 
http://subversion02:8060/cru/CR-4#c14

  Het lijkt mij beter om deze 2 aparte testen samen te nemen en
  daar één test van maken rekening houdend met de opmerking
  hierboven, dan hebben we één die de standaard use-case
  beschrijft.
  Andere testen kunnen dan toegevoegd worden om alternatief
  gedrag te gaan testen

Revision Comment by Jo Claes on 05 January 2017, 11:01 
http://subversion02:8060/cru/CR-4#c13

  Ongebruikte lijn

----------------------------------------
File: vhTest/Utils/WS/Vhisie4/Winkelkar/BewaarProductLijnRequest.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 10:39 
http://subversion02:8060/cru/CR-4#c7

  Wat is het nut van deze klasse ??

----------------------------------------
File: vhUnitTest/DOM/EC/Winkelkar/impl/WinkelkarRepository/Test.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 10:43 
http://subversion02:8060/cru/CR-4#c9

  Deze testen zijn zo basic dat ik zelf de moeite niet zou doen
  om deze manueel te maken.
  Het lijkt mij beter om ( later ) aan een systeem te werken :
  - dat op zijn minst ofwel de queries gebruikt in een repository
  gaat controleren of die überhaupt uitvoerbaar is.
  - ofwel effectief wat data gaat opzetten en uitlezen daar het
  gaat om een repository


---

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

Title: [UST2037] vh4_SVC : BewaarProductLijn
Statement of Objectives:


State: Closed

Summary: 

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