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

Review Summary: No summary


----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/BasisElement/CrossDiag.cls.xml
Revision Comment by Tom Vermeulen on 23 March 2020, 08:25 
http://subversion02:8060/cru/CR-2013#c3239

  nu die constructor leeg is, kan je je de vraag stellen of hij
  hier überhaupt nog moet blijven staan.
  
  Een constructie als deze kan een potentieel risico inhouden:
  aangezien je enkel een Quit $$$OK doet, maskeer je logica die
  in de constructor van de Superklasses zit. In dit geval is dat
  niet echt relevant, omdat het toch klasses zijn zonder enige
  inhoud (bijna). Wou het gewoon even melden.
  
  (logica van de superklasse oproepen:   do ##super()  )

----------------------------------------
File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/BasisElement/CrossFactory/Test.cls.xml
Revision Comment by Tom Vermeulen on 23 March 2020, 08:32 defect
http://subversion02:8060/cru/CR-2013#c3240

  AssertClassName bestaat ook

Revision Comment by Wim Vermeulen on 02 April 2020, 10:40 
http://subversion02:8060/cru/CR-2013#c3306

  Zoals het er nu staat, kan je enigszins verwarring scheppen
  tussen de echte MaakLijn() en de VerwachtMethodCall, aangezien
  deze (eerste) dezelfde parameters hebben.
  Bij voorkeur 2 verschillende (dummy) Lijn-objecten gebruiken
  bij de DanReturn  :
  
  DummyLine1 = MaakLijn(0,0,0,0)
  DummyLine2 = MaakLijn(0,0,0,0)

Revision Comment by Wim Vermeulen on 02 April 2020, 10:10 
http://subversion02:8060/cru/CR-2013#c3305

  Bij iedere UT moet je denken   "Kan deze setup fouten
  verbergen?" of anders gezegd 
  "Kan ik een programmeerfout aan het licht brengen, puur door
  andere input te kiezen?"
  
  Dus de (30, 30) zou je beter aanpassen naar verschillende
  waarden (zoals bvb. erboven 25,30)


---

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

Title: [ICT-1374] SVG-031 Aanmaken CrossFactory
Statement of Objectives:


State: Closed

Summary: 

Author: Sam Van Hoey
Moderator: Sam Van Hoey
Reviewers: (0 active, 2 completed*)
     Tom Vermeulen   (*)
     Wim Vermeulen   (*)