Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1656] SVG-036 HoekverbinderToevoeger + maker refactoren

-Aanpassing aan test van de kaderdeur

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

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is (kleine kaartjes, rap committen, enz enz http://subversion02.vanhoecke.b...

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is

(kleine kaartjes, rap committen, enz enz )

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen. Idem bij constructors andere boringen

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen.

Idem bij constructors andere boringen

klasse newen in constructor

klasse newen in constructor

indents ongedaan maken

indents ongedaan maken

[ICT-1365] SVG-029 Mock Testen uitbreiden

-Test toegevoegd en klassen geinjecteerd

  1. … 1 more file in changeset.
[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger

-Toevoegen van ElementToevoeger en bewerken van het toekennen van variabelen

  1. … 17 more files in changeset.
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 g...

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

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

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.