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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/TECH/Svg/Basis/DomElement/Test.cls.xml
Revision Comment by Tom Vermeulen on 29 January 2020, 09:48 defect
http://subversion02:8060/cru/CR-1938#c2953

  leuk, maar deze test test niets :)
  
  Als de constructor faalt, zal er wel één of andere exception
  geraised worden, maar dat is niet het gedrag dat je wil testen
  in dit geval.
  
  Een test moet na een Assign ook een Act en een Assert hebben,
  het is dat laatste deel dat gaat vergewissen of de test slaag
  of niet.
  
  In dit geval zou je bv als assert de volgende lijn kunnen
  gebruiken:
  Do $$$AssertTrue($IsObject(DomElement))
  
  Je moet je natuurlijk de vraag stellen of het zinvol is om
  enkel de constructor van een object te gaan testen. Als dat een
  standaardconstructor (dus zonder eigen logica), is dat overkill
  omdat we er van uitgaan dat Caché er wel in zal slagen om
  objecten te creëren :)
  
  Je zou de test zelf bv kunnen vervangen door eentje die een
  object maakt en dan de nodige setmethods uitvoert en dan
  tenslotte verifieert dat de gesette waarden ook aanwezig zijn
  op het gecreëerde object. Dit klopt dan niet 100% met de
  defaultnaamgeving van onze testen, maar dat maakt nu niet zo
  veel uit

  Reply by Wim Vermeulen on 30 January 2020, 14:10
  > Akkoord met Tom.
  > Maar laten we dit eens tesamen bekijken, hoe we dit best
  > kunnen beter maken, volgens de regels van de (UT)kunst :-)
  > Mijn idee : Svg.Basis.DomElement moet een abstracte klasse
  > zijn, dus zal niet kunnen ge-newed worden.  Daarom een
  > concrete UT-klasse maken die afleidt van DomElement.  Daarvan
  > kan je dan wel een UT-SvgObject maken om uw methods te
  > testen.

Revision Comment by Tom Vermeulen on 06 February 2020, 11:09 
http://subversion02:8060/cru/CR-1938#c2979

  Als je deze apart voorziet, moet je in principe ook eentje voor
  Stroke, Overflow, ... maken (wat de meest pure manier is als je
  het TDD-gewijs opbouwt)
  Alternatief: breidt de andere test wat uit. Uiteindelijk doe je
  daar hetzelfde: properties zetten zonder parameters mee te
  geven.
  Je kan in dat geval meer dan 1 ding asserten in een test


---

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

Title: [ICT-1126] SVG-001 DOM element aanmaken waar de basic type van kunnen afleiden
Statement of Objectives:


State: Closed

Summary: 

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