Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-XPos meegegeven ipv Breedte voor de rotation van 90 graden van een KadFreesRect

    • -24
    • +24
    ./MaakBoringSpecialeVorm/Test.cls.xml
  1. … 1 more file in changeset.
[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Aanpassingen aan testen door aanpassing voor rotation kadFreesRect

  1. … 1 more file in changeset.
[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Extra specialeBoringen toegevoegd na testen van kaderdeuren en daarvoor aanpassingen voor bestaande testen

    • -13
    • +13
    ./MaakBoringSpecialeVorm/Test.cls.xml
  1. … 4 more files in changeset.
[ICT-1656] SVG-036 HoekverbinderToevoeger + maker refactoren

-Aanpassing aan test van de kaderdeur

Indentatie (extreem rechts ? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif )

Indentatie (extreem rechts ? )

Deze lijn kan onder de if-structuur, moet niet telkens herhaald worden binnen de if.

Deze lijn kan onder de if-structuur, moet niet telkens herhaald worden binnen de if.

Ook doen of is het dan niet leesbaar genoeg?

Ook doen of is het dan niet leesbaar genoeg?

normaalgezien zou je heel deze if-elseif-else kunnen vervangen door Quit (Totaal && ##class(TECH.Math).IsEven(Totaal))

normaalgezien zou je heel deze if-elseif-else kunnen vervangen door Quit (Totaal && ##class(TECH.Math).IsEven(Totaal))

private

private

aaa, the good old wlip

aaa, the good old wlip

na opmerking van wim: testen toevoegen voor extra gedrag. Wat als er 0,1,2, veel coordinaten worden meegegeven?

na opmerking van wim: testen toevoegen voor extra gedrag.
Wat als er 0,1,2, veel coordinaten worden meegegeven?

Deze method is een leuke oefening (vanuit educatief standpunt), maar in de praktijk vind ik dit een nogal tricky interface. Als we deze willen blijven gebruiken/ondersteunen, dan is heb ik een rev...

Deze method is een leuke oefening (vanuit educatief standpunt), maar in de praktijk vind ik dit een nogal tricky interface.

Als we deze willen blijven gebruiken/ondersteunen, dan is heb ik een review-opmerking, namelijk : deze moet een InvalidInput-exception smijten indien het aantal "Punten" oneven is.
Let op : het tellen van Punten... is geen one-liner !

Ofwel overwegen om aan te passen naar $LB(X,Y) parameters, i.e.

MaakCoordinatenLijstVanXYPunten(lbXYPunten... As %List) 
[ICT-1340]SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-Uitbreiden van testen + aanpassing aan KadFreesRect

    • -12
    • +77
    ./MaakBoringSpecialeVorm/Test.cls.xml
  1. … 1 more file in changeset.
[ICT-1340]SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-Test dubbels gebruiken

  1. … 1 more file in changeset.
[ICT-1340]SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-extra parameters in constructor toegevoegd

  1. … 1 more file in changeset.
[ICT-1340]SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-Move class vhUnitTest.APPS.Halux.PPS.ALUFrees.Svg.KaderdeurElement.Beslag.BeslagBoringSpecialeVorm.Test naar vhUnitTest.APPS.Halux.PPS.ALUFrees.Svg.KaderdeurElement.Beslag.BeslagBoringSpecialeVorm.MaakBoringSpecialeVorm.Test

    • -0
    • +60
    ./MaakBoringSpecialeVorm/Test.cls.xml
coordinatesservice en factory injecteren (en gebruiken in testen)

coordinatesservice en factory injecteren (en gebruiken in testen)

[ICT-1340]

SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-aanmaken CoordinatesService.cls.xml en methodes + toepassen van methodes

  1. … 3 more files in changeset.
[ICT-1210]

SVG-015 Boringen toevoegen aan de kaderdeur

-toevoeging specialevormbeslagboring

  1. … 1 more file in changeset.