Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

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 .

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

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)

[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
[ICT-1365] SVG-029 Mock Testen uitbreiden
[ICT-1365] SVG-029 Mock Testen uitbreiden
[ICT-1365] SVG-029 Mock Testen uitbreiden

-mock testen schrijven en klassen aanpassen voor losprofiel, schuinehoeken en kadertoevoeger

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

testjes voor andere combinaties? RO, RB, LO en ALL

testjes voor andere combinaties? RO, RB, LO en ALL

moet deze niet naast VulTweeRects staan?

moet deze niet naast VulTweeRects staan?

De test heet "... standaardkaderdeur", maar nergens wordt die standaarddeur gebruikt (logisch, want je zit een niveau lager). Is het voldoende om enkel het aantal te testen? Of moeten we van sommi...

De test heet "... standaardkaderdeur", maar nergens wordt die standaarddeur gebruikt (logisch, want je zit een niveau lager).

Is het voldoende om enkel het aantal te testen? Of moeten we van sommige items bepaalde properties aftoetsen (is misschien veiliger)
Je kan zo'n constructie gebruiken:
If $$$AssertEquals(blabla.Count(),2)

Unknown macro: { Do $$$AssertEquals(blabla.Getat(1).EenOfAndereProperty,"njanjanja") ... }


Je gaat hier nog wat testen voor losprofiel, enkel links, enkel rechts, enkel boven, enz toevoegen klopt dat?

Ge Set twee keer achter elkaar VerbindersGroup. De eerste is dus zinloos. Aangezien de tweede geen constructor is, vervang je de eerste best door een gewone #dim

Ge Set twee keer achter elkaar VerbindersGroup. De eerste is dus zinloos. Aangezien de tweede geen constructor is, vervang je de eerste best door een gewone #dim

[ICT-1287]

SVG-020 Testen van wegschrijven kaderdeur

-toevoegen van testen voor kaderdeur

  1. … 13 more files in changeset.