Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
mag deze dan niet volledig weg?

mag deze dan niet volledig weg?

Ook properties asserten, niet enkel de SVG output (idem voor andere testjes hieronder)

Ook properties asserten, niet enkel de SVG output (idem voor andere testjes hieronder)

niet noodzakelijk voor een goeie werking http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (hieronder ook nog eentje)

niet noodzakelijk voor een goeie werking (hieronder ook nog eentje)

dees gaat altijd true zijn, ge hebt hierboven twee items toegevoegd.

dees gaat altijd true zijn, ge hebt hierboven twee items toegevoegd.

[ICT-1196] SVG-013 Factory toepassen op de svg basiselementen
[ICT-1196] SVG-013 Factory toepassen op de svg basiselementen
[ICT-1126] SVG-001 DOM element aanmaken waar de basic type van kunnen afleiden

-Aanpassing resultaat van svg test door aanpassing van fill naar "none"

-Test scenario uitgebreid naar alle set methoden

  1. … 1 more file in changeset.
[ICT-1192]

SVG-011 Detail aanpassing aan basisobjecten tijdens het project

-Toevoegen van opacity door aanpassing DomElement

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. U...

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

Akkoord met Tom. Maar laten we dit eens tesamen bekijken, hoe we dit best kunnen beter maken, volgens de regels van de (UT)kunst http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/i...

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.

Wim Vermeulen schrijft uw gedacht ook eens neer ivm het meegeven van values aan de constructor. Sam en ik hebben het er gisteren even over gehad. In theorie is het properder om de constructor enkel...

Wim Vermeulen schrijft uw gedacht ook eens neer ivm het meegeven van values aan de constructor. Sam en ik hebben het er gisteren even over gehad. In theorie is het properder om de constructor enkel objecten mee te geven waarop dependencies zijn. Dan moet elk basisobject natuurlijk wel een "setparams" of "initialize" method krijgen.
Voordeel: testen duidelijker en conform de rest van de nieuwere code
Nadeel: Sam gaat wat door zijn code moeten crossen om het over te zetten (dacht dat het bij het DomElement wel al zo was)

commentaarlijnen kan je beter wegsmijten

commentaarlijnen kan je beter wegsmijten

deze review werd aangemaakt zonder directe link naar het issue in kwestie

deze review werd aangemaakt zonder directe link naar het issue in kwestie

leuk, maar deze test test niets http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Als de constructor faalt, zal er wel één of andere exception geraised w...

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

[ICT-1126] SVG-001 DOM element aanmaken waar de basic type van kunnen afleiden
[ICT-1126] SVG-001 DOM element aanmaken waar de basic type van kunnen afleiden
[ICT-1128]

SVG-003 aanmaken klasse voor een rechthoek

[ICT-1128]

SVG-003 aanmaken klasse voor een rechthoek

  1. … 1 more file in changeset.