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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/HerberekenMaatwerk/Test.cls.xml
Revision Comment by Jo Claes on 17 March 2017, 14:13 defect
http://subversion02:8060/cru/CR-136#c309

  Volgens mij gaat deze test identiek hetzelfde testen dan één
  van de LijnCacheBuilder testen .. DRY

Revision Comment by Jo Claes on 17 March 2017, 14:09 defect
http://subversion02:8060/cru/CR-136#c308

  Geen logica gebruiken in testen .. beter hiervoor een hardcoded
  waarde voor gebruiken . zal wel te vinden zijn in de
  vhTest.Utils 
  Als de ToArray faalt dan zal dit negatieve inpakt hebben op de
  testen terwijl het er los van staat.

  Reply by Kurt Callebaut on 20 March 2017, 08:29
  > Ik zou dan voorstellen dat je zelf ook geen .ToArray gebruikt
  > in je testen. Commit 39782 Joc en commit 39550 TomV ->
  > vhUnitTest.WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Test


---

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

Title: [UW352] Herberekenmaatwerk Visie 4
Statement of Objectives:
Jo, kan je dit al eens nazien en laten weten wat er nog moet gebeuren. (Los van de synchroniseer, die komt later aan de beurt.) -> zie cachebuilder. Ik zou voor synchroniseer lijncache ook niet public willen maken, maar visitor of dergelijke willen toepassen. suggesties ?

State: Closed

Summary: 

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