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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur/HoekverbinderToevoeger/Test.cls.xml
Revision Comment by Tom Vermeulen on 04 March 2020, 10:04 defect
http://subversion02:8060/cru/CR-1998#c3174

  Ge Set twee keer achter elkaar VerbindersGroup. De eerste is
  dus zinloos. Aangezien de tweede geen constructor is, vervang
  je de eerste best door een gewone #dim

  Reply by Wim Vermeulen on 05 March 2020, 14:51
  > Dit betekent wel dat de Toevoeger.VoegToeBlablabla() het
  > object VerbindersGroup gaat Newen.
  > dat mag, maar niet in "Clean coders world"  :-P
  > 
  > Ofwel met Tom aanpakken, ofwel nog wachten tot een volgende
  > (grote) ronde van refactoring.

Revision Comment by Tom Vermeulen on 04 March 2020, 10:06 
http://subversion02:8060/cru/CR-1998#c3175

  De test heet "... standaardkaderdeur", maar nergens wordt die
  standaarddeur gebruikt (logisch, want je zit een niveau lager).
  
  
  Is het voldoende om enkel het aantal te testen? Of moeten we
  van sommige items bepaalde properties aftoetsen (is misschien
  veiliger)
  Je kan zo'n constructie gebruiken:
  If $$$AssertEquals(blabla.Count(),2) {
    Do
  $$$AssertEquals(blabla.Getat(1).EenOfAndereProperty,"njanjanja")
    ...
  }
  
  Je gaat hier nog wat testen voor losprofiel, enkel links, enkel
  rechts, enkel boven, enz toevoegen klopt dat?

----------------------------------------
File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur/VulDrieRects/Test.cls.xml
Revision Comment by Tom Vermeulen on 04 March 2020, 10:30 
http://subversion02:8060/cru/CR-1998#c3176

  moet deze niet naast VulTweeRects staan?

----------------------------------------
File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Hoekverbinder/Test.cls.xml
Revision Comment by Wim Vermeulen on 05 March 2020, 15:18 
http://subversion02:8060/cru/CR-1998#c3189

  Hey Sam, ni erg hoor, maar deze test gaan we zeker nog eens
  helemaal herwerken, want is volgens de UT-principes behoorlijk
  naast de kwestie. 
  Het zijn dan ook uw eerste stappen in deze wereld, dus ge moet
  het u zeker niet aantrekken hé.

Revision Comment by Tom Vermeulen on 04 March 2020, 10:32 
http://subversion02:8060/cru/CR-1998#c3177

  testjes voor andere combinaties? RO, RB, LO en ALL


---

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

Title: [ICT-1287]
Statement of Objectives:


State: Closed

Summary: 

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