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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/DOM/EC/Winkelkar/impl/LijnRepository/Test.cls.xml
Revision Comment by Jo Claes on 13 January 2017, 10:47 
http://subversion02:8060/cru/CR-21#c100

  Lijkt mij  overbodig

  Reply by Kurt Callebaut on 16 January 2017, 14:08
  > Dan compileert de klasse niet.

Revision Comment by Jo Claes on 13 January 2017, 10:47 
http://subversion02:8060/cru/CR-21#c101

  Lijkt mij  overbodig

  Reply by Kurt Callebaut on 16 January 2017, 14:08
  > Dan compileert de klasse niet.

Revision Comment by Jo Claes on 13 January 2017, 10:47 
http://subversion02:8060/cru/CR-21#c102

  Lijkt mij  overbodig

  Reply by Kurt Callebaut on 16 January 2017, 14:09
  > Dan compileert de klasse niet.

----------------------------------------
File: vhUnitTest/DOM/EC/Winkelkar/impl/WinkelkarRepository/Test.cls.xml
Revision Comment by Jo Claes on 13 January 2017, 10:50 defect
http://subversion02:8060/cru/CR-21#c103

  Dit soort type testen schrijf je ofwel met echte data - data
  opzetten en kijken of de repo zijn ding doet .
  ofwel met een spy die op zijn minst de Query prepared - zodat
  we zeker zijn dat het query statement kan uitgevoerd worden.
  
  Het gevaar van dit soort testen is dat je uw Query statement in
  uw code gaat overkopieëren naar uw testen en als er een fout in
  het statement zit tabel naam dat bijvoorbeeld niet meer bestaat
  dat merk je het niet op.

  Reply by Kurt Callebaut on 16 January 2017, 14:14
  > thx for the feedback.

----------------------------------------
File: WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn.cls.xml
Revision Comment by Jo Claes on 13 January 2017, 10:42 
http://subversion02:8060/cru/CR-21#c97

  Controles zijn niet nodig daar de GeefViaId op de winkelkar
  repo en de lijn repo de juiste foutmeldingen zullen geven

  Reply by Kurt Callebaut on 16 January 2017, 13:22
  > Hieronder staat GeefViaID ?

Revision Comment by Jo Claes on 13 January 2017, 10:37 
http://subversion02:8060/cru/CR-21#c94

  GeefViaID method gebruiken daar de
  WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen notie hebben van
  externeid in hun interface enkel in de implementatie

  Reply by Kurt Callebaut on 16 January 2017, 14:00
  > Hierboven staat GeefViaId ?
  > Methods ....ViaExterneId refactored naar ViaID voor
  > Vhisie4.WInkelkar.... Repositories

Revision Comment by Jo Claes on 13 January 2017, 10:39 
http://subversion02:8060/cru/CR-21#c95

  BestaatViaID WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen
  notie hebben van externeid in hun interface enkel in de
  implementatie

  Reply by Kurt Callebaut on 16 January 2017, 14:01
  > Methods ....ViaExterneId refactored naar ViaID voor
  > Vhisie4.WInkelkar.... Repositories

----------------------------------------
File: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn/Test.cls.xml
Revision Comment by Jo Claes on 13 January 2017, 10:56 
http://subversion02:8060/cru/CR-21#c105

  Hier zou ik gaan voor de WinkelkarLijnRepositoryMock ipv de
  LijnRepositoryMock .. als we dan toch voluit gaan...

  Reply by Kurt Callebaut on 16 January 2017, 12:48
  > OKay.

Revision Comment by Jo Claes on 13 January 2017, 10:53 
http://subversion02:8060/cru/CR-21#c104

  Ik zou de naam van variabele renamen naar Service ofwel moeten
  we de methods renamen naar Invoke.. 
  Want het is nu een beetje onduidelijk
  VerwijderLijn.VerwijderLijn .. klikt een beetje raar
  
  ofwel VerwijderLijnService.VerwijderLijn(
  ofwel VerwijderLijn.Invoke(

  Reply by Kurt Callebaut on 16 January 2017, 12:44
  > Er is gekozen voor deze benaming.
  > Hadden ze op voorhand maar moeten bedenken. :-)

Revision Comment by Jo Claes on 13 January 2017, 10:59 defect
http://subversion02:8060/cru/CR-21#c107

  Overbodig dus

  Reply by Kurt Callebaut on 16 January 2017, 12:54
  > Laat maar staan, dan is het duidelijk dat er op 2 waarden
  > gecontroleerd wordt.

Revision Comment by Jo Claes on 13 January 2017, 10:57 
http://subversion02:8060/cru/CR-21#c106

  Foutmelding is beter Winkelkar met id 'WinkelkarId' bestaat
  niet.

  Reply by Kurt Callebaut on 16 January 2017, 12:46
  > Een goeie week geleden heb je gezegd dat we niet offerte id
  > en winkelkar id mogen gebruiken in foutmeldingen omdat het
  > een en dezelfde functionaliteit is. We hebben dus voor
  > externeid gekozen. Dus laat externeid maar staan. WinkelkarId
  > is gewoon de testwaarde. Niet blokkerend.

----------------------------------------
File: WSimpl/Vhisie4/Winkelkar/WinkelkarLijnRepository.cls.xml
Revision Comment by Jo Claes on 13 January 2017, 10:44 
http://subversion02:8060/cru/CR-21#c98

  BestaatViaID

Revision Comment by Jo Claes on 13 January 2017, 10:44 
http://subversion02:8060/cru/CR-21#c99

  GeefViaID


---

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

Title: [UST2065] Vh4_SVC:VerwijderLijn
Statement of Objectives:


State: Closed

Summary: 

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