Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2059] SVG-040 Aanpassingen voor finale html

-ID toegevoegd aan tags + aanpassingen aan unitTests

    • -2
    • +2
    ./KaderToevoeger/VulTweeRects/Test.cls.xml
  1. … 8 more files in changeset.
[ICT-1656] SVG-036 HoekverbinderToevoeger + maker refactoren

-Aanpassing aan test van de kaderdeur

  1. … 1 more file in changeset.
[ICT-1656] SVG-036 HoekverbinderToevoeger + maker refactoren

-Aanpassing aan test van de kaderdeur

[ICT-1656] SVG-036 HoekverbinderToevoeger + maker refactoren

-HoekVerbinderToevoeger + maker aangepast + testen

    • -152
    • +56
    ./HoekverbinderToevoeger/Test.cls.xml
  1. … 3 more files in changeset.
[ICT-1125] Kaderdeurbeelden omzetten van VML naar SVG

-Code Cleanup

    • -0
    • +3
    ./KaderToevoeger/VulTweeRects/Test.cls.xml
  1. … 2 more files in changeset.
[ICT-1125] Kaderdeurbeelden omzetten van VML naar SVG

-Code Cleanup

  1. … 8 more files in changeset.
Standaard regel voor onze UT's : naamgeving = "vhUnitTest." + naam van de klasse " + "." + naam van de methode "." + "Test" Dus in dit geval was het bijna juist http://subversion02.vanhoecke.be/sta...

Standaard regel voor onze UT's : naamgeving = "vhUnitTest." + naam van de klasse " + "." + naam van de methode "." + "Test"
Dus in dit geval was het bijna juist
--> vhUnitTest.APPS.Halux.PPS.ALUFrees.Svg.Kaderdeur.GenereerSvg.Test

Er zijn uitzonderingen, maar hier is standaard regel zeker oké.
Graag ook je andere UT's eens nakijken. Aanpassen kan je best via :

Do ##class(Tools.CacheSourceUtils).MoveClass("","")
kommaneukerij, maar ik zou "uitvoeren van methodes" niet in de testnaam zetten, anders kan hij misschien te lang worden en 't is nogal triviaal

kommaneukerij, maar ik zou "uitvoeren van methodes" niet in de testnaam zetten, anders kan hij misschien te lang worden en 't is nogal triviaal

toch eens naar de rest gekeken en vraag me af wat de zin is van die write (buiten tijdens development) - > weg die handel http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/em...

toch eens naar de rest gekeken en vraag me af wat de zin is van die write (buiten tijdens development) - > weg die handel

niet gekeken wat hierboven nog gebeurt, maar je kan best w !,"standaard .... gebruiken. Dat ! geeft een CRLF

niet gekeken wat hierboven nog gebeurt, maar je kan best
w !,"standaard ....
gebruiken. Dat ! geeft een CRLF

[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods
[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods
[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods

-uniTesten aangepast

[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods

-unitTesten toegevoegd + standaardobjecten

  1. … 2 more files in changeset.
[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods

-kaderdeur opsplitsen + extra kaderdeur aangemaakt met los profiel

  1. … 2 more files in changeset.
[ICT-1285]

-GenereerSVG opgesplitst in private methodes + mock test geschreven

  1. … 1 more file in changeset.
Ik ben nogal verward hoe deze test is opgebouwd. Mogelijk is dit veroorzaakt door misleidende naamgevingen. Ook conceptueel lijkt er iets niet te kloppen. Laten we zeggen dat de test aan het licht ...

Ik ben nogal verward hoe deze test is opgebouwd. Mogelijk is dit veroorzaakt door misleidende naamgevingen.
Ook conceptueel lijkt er iets niet te kloppen. Laten we zeggen dat de test aan het licht brengt dat de interfaces niet helemaal goed zitten.
Aan deze UT's (en code) gaan we best wat knutselen. Contact me

Deze method gaan we eens samen bekijken: *afsplitsen in meerdere private methods : bevordert de leesbaarheid, maar bovenal vermijdt dependencies tussen stukken code die niks met mekaar te maken h...

Deze method gaan we eens samen bekijken:

  • afsplitsen in meerdere private methods : bevordert de leesbaarheid, maar bovenal vermijdt dependencies tussen stukken code die niks met mekaar te maken hebben.
  • oproep/interface geschikt maken om vanuit Prod.GADef.Kaderdeur op te roepen.
  • UT's rond GenereerSvg() zou ik een andere aanpak voorstellen. Contact me
Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets

Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets