CacheAdminA_trunk2010/vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg

Kaderdeur

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 03 Mar 20
Commits this week: 0
Total Lines of Code (LoC): 538
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 23 Nov 2020
8:57 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:55 am

Sam Van Hoey

8:51 am

Sam Van Hoey

Friday 15 May 2020
12:24 pm

Wim Vermeulen

11:55 am

Wim Vermeulen

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("","")
Friday 08 May 2020
10:53 am

Tom Vermeulen

10:22 am

Tom Vermeulen

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

10:20 am

Tom Vermeulen

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

10:19 am

Tom Vermeulen

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

Wednesday 06 May 2020
1:13 pm

Sam Van Hoey

[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods
[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods
Tuesday 07 Apr 2020
8:40 am

Tom Vermeulen

Thursday 02 Apr 2020
6:21 pm

Wim Vermeulen

6:10 pm

Wim Vermeulen

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

5:56 pm

Wim Vermeulen

4:54 pm

Wim Vermeulen

4:43 pm

Wim Vermeulen

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
Wednesday 25 Mar 2020
12:21 pm

Tom Vermeulen

12:17 pm

Tom Vermeulen

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

12:13 pm

Tom Vermeulen

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

12:12 pm

Tom Vermeulen

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 )

12:10 pm

Tom Vermeulen

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

12:07 pm

Tom Vermeulen

klasse newen in constructor

klasse newen in constructor

12:05 pm

Tom Vermeulen

indents ongedaan maken

indents ongedaan maken

12:00 pm

Tom Vermeulen

11:41 am

Tom Vermeulen

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif.

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen .

11:39 am

Tom Vermeulen

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er i...

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er in deze klasse gebruikt worden

11:38 am

Tom Vermeulen

ge kunt het niet laten hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (hieronder nog een paar)

ge kunt het niet laten hé (hieronder nog een paar)

Monday 23 Mar 2020
4:36 pm

Sam Van Hoey

[ICT-1285]
[ICT-1285]