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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/BereidVoorBestelling/Test.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 13:56 
http://subversion02:8060/cru/CR-7#c23

  Als je data gebruikt gelieve representatieve data te gebruiken.
  Dus ik zou kijken om de standaard adressen te gebruiken naar
  analogie met vhTest.Utils.APPS.common.dto.Adres

  Reply by Kurt Callebaut on 07 January 2017, 12:50
  > Belgisch adres toegevoegd.

----------------------------------------
File: WSimpl/Vhisie4/Winkelkar/WinkelkarService/BereidVoorBestelling.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 13:53 
http://subversion02:8060/cru/CR-7#c19

  Gebruik maken van gemeenschappelijke klasse om ExterneId te
  controleren

  Reply by Kurt Callebaut on 07 January 2017, 12:51
  > Zie uitleg CR-3. misschien later te refactoren als ook
  > verkoopservice en dergelijke aangepakt wordt.

Revision Comment by Jo Claes on 05 January 2017, 13:54 
http://subversion02:8060/cru/CR-7#c20

  Zeker dat de landcodes die gaan gebruikt worden door intracto
  mappen op onze landcodes ?

  Reply by Kurt Callebaut on 07 January 2017, 12:52
  > Mocht dat niet zo zijn, dan stel ik voor dat we net zoals bij
  > den unishop zelf de landen en codes aanleveren aan intracto.
  > Wat ik wel eventueel bedenk is validaties toe te voegen op
  > postcodes, landcodes, etc.. maar dat was niet voor deze fase
  > dacht ik. dat zal ik es navragen.

    Reply by Kurt Callebaut on 12 January 2017, 08:05
    > Uitzoeken landcodes ik maak hiervoor een kaartje aan. ->
    > besproken met PVR. Idee om landen aan te bieden net zoals
    > bij unishop is ok. Wordt later nog uitgewerkt.

----------------------------------------
File: vhUnitTest/DOM/EC/Winkelkar/impl/WinkelkarRepository/Test.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 13:56 
http://subversion02:8060/cru/CR-7#c22

  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

----------------------------------------
File: vhUnitTest/APPS/VKP/Offerte/impl/OfferteRepository/Test.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 13:55 
http://subversion02:8060/cru/CR-7#c21

  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

  Reply by Kurt Callebaut on 07 January 2017, 12:34
  > Dit was je voorstel van testen bij het refactoren van de
  > repository in de eerste fase vorig jaar rond de zomer.
  > Voorlopig goed genoeg en beter dan geen. 
  > Als we data testen gaan opzetten,dan kunnen we daar best ook
  > een aparte package en aparte testserver voor voorzien.

    Reply by Jo Claes on 09 January 2017, 09:58
    > Waarom moet dat aparte package en aparte testserver zijn .
    > het zullen allemaal heel kleine , zeer vlugge testen zijn. 
    > Als blijkt dat er dan toch trage(re) testen zijn .. zal dat
    > eerder te wijten zijn het ontbreken van indexen op de data
    > dus terug een trigger om iets te moeten doen


---

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

Title: [UST1960] Vh4_SVC : BereidVoorBestelling
Statement of Objectives:


State: Closed

Summary: 

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