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

Boringen

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 28 Jan 20
Commits this week: 0
Total Lines of Code (LoC): 244
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:56 am

Sam Van Hoey

8:55 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:51 am

Sam Van Hoey

Monday 10 Aug 2020
3:20 pm

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Aanpassingen aan testen door aanpassing voor rotation kadFreesRect

Options
    • -1
    • +1
    ./VoegToeBeslagBoringen/Test.cls.xml
  2. … 1 more file in changeset.
10:30 am

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

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

Options
    • -1
    • +1
    ./VoegToeBeslagBoringen/Test.cls.xml
  2. … 4 more files in changeset.
Monday 20 Jul 2020
12:16 pm

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Aanpassing aan test door wijziging van beslagboring

Options
    • -1
    • +1
    ./VoegToeBeslagBoringen/Test.cls.xml
11:57 am

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Kleur aangepast van Tcross + coordinates + puntboring ipv rugboring

Options
    • -2
    • +2
    ./VoegToeBoringen/Test.cls.xml
  2. … 7 more files in changeset.
Tuesday 07 Apr 2020
8:40 am

Tom Vermeulen

Thursday 02 Apr 2020
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:58 pm

Wim Vermeulen

5:58 pm

Wim Vermeulen

Deze method hoort niet thuis in deze klasse, want heeft niets met BeslagBoringen te maken. We zullen samen bekijken hoe we dit best kunnen afsplitsen.

Deze method hoort niet thuis in deze klasse, want heeft niets met BeslagBoringen te maken.
We zullen samen bekijken hoe we dit best kunnen afsplitsen.

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
10:53 am

Wim Vermeulen

10:51 am

Wim Vermeulen

Bij iedere UT moet je denken "Kan deze setup fouten verbergen?" of anders gezegd "Kan ik een programmeerfout aan het licht brengen, puur door andere input te kiezen?" Dus de (30, 30) zou je beter ...

Bij iedere UT moet je denken "Kan deze setup fouten verbergen?" of anders gezegd
"Kan ik een programmeerfout aan het licht brengen, puur door andere input te kiezen?"

Dus de (30, 30) zou je beter aanpassen naar verschillende waarden (zoals bvb. erboven 25,30)

10:49 am

Wim Vermeulen

Zoals het er nu staat, kan je enigszins verwarring scheppen tussen de echte MaakLijn() en de VerwachtMethodCall, aangezien deze (eerste) dezelfde parameters hebben. Bij voorkeur 2 verschillende (du...

Zoals het er nu staat, kan je enigszins verwarring scheppen tussen de echte MaakLijn() en de VerwachtMethodCall, aangezien deze (eerste) dezelfde parameters hebben.
Bij voorkeur 2 verschillende (dummy) Lijn-objecten gebruiken bij de DanReturn :

DummyLine1 = MaakLijn(0,0,0,0)
DummyLine2 = MaakLijn(0,0,0,0)

Wednesday 25 Mar 2020
11:31 am

Tom Vermeulen

Monday 23 Mar 2020
4:36 pm

Sam Van Hoey

[ICT-1285]
[ICT-1285]
2:52 pm

Sam Van Hoey

[ICT-1326] SVG-023 Uitbreiden testen svg-project
[ICT-1326] SVG-023 Uitbreiden testen svg-project
2:51 pm

SamV

[ICT-1326] SVG-023 Uitbreiden testen svg-project

-Toepassen van Testen op de klasse Boringen hier voor Standaard kaderdeur met de verschillende boringen aangemaakt en beslagen met de verschillende boringen aangemaakt, BeslagBoringSpecialeVorm mockable gemaakt om te kunnen testen in Boringen + toegevoegd als property

Options
    • -0
    • +107
    ./VoegToeBeslagBoringen/Test.cls.xml
    • -0
    • +60
    ./VoegToeBoringen/Test.cls.xml
  3. … 8 more files in changeset.
8:34 am

Tom Vermeulen

8:32 am

Tom Vermeulen

AssertClassName bestaat ook

AssertClassName bestaat ook

8:28 am

Tom Vermeulen

nu die constructor leeg is, kan je je de vraag stellen of hij hier überhaupt nog moet blijven staan. Een constructie als deze kan een potentieel risico inhouden: aangezien je enkel een Quit $$$OK ...

nu die constructor leeg is, kan je je de vraag stellen of hij hier überhaupt nog moet blijven staan.

Een constructie als deze kan een potentieel risico inhouden: aangezien je enkel een Quit $$$OK doet, maskeer je logica die in de constructor van de Superklasses zit. In dit geval is dat niet echt relevant, omdat het toch klasses zijn zonder enige inhoud (bijna). Wou het gewoon even melden.

(logica van de superklasse oproepen: do ##super() )

Thursday 19 Mar 2020
2:34 pm

Tom Vermeulen

11:42 am

Sam Van Hoey

[ICT-1374] SVG-031 Aanmaken CrossFactory
[ICT-1374] SVG-031 Aanmaken CrossFactory
11:22 am

SamV

[ICT-1374] SVG-031 Aanmaken CrossFactory

-Toevoegen van CrossFactory en test + verwijderen test cross objecten + aanpassen cross objecten + Aanpassing BoringELementFactory/Test.cls.xml

Options
    • -0
    • +1
    ./VoegToeBoringen/Test.cls.xml
  2. … 19 more files in changeset.
Thursday 12 Mar 2020
6:26 pm

Wim Vermeulen

6:21 pm

Wim Vermeulen

Sam, deze implementatie gaan we in "pair reviewen" (wss rechtstreeks vanop de code). We kunnen hier immers vele kanten mee uit http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ico...

Sam, deze implementatie gaan we in "pair reviewen" (wss rechtstreeks vanop de code).
We kunnen hier immers vele kanten mee uit

5:42 pm

Wim Vermeulen

Al opgelost? Indien niet, dan mag je een "UnimplementedException" throwen (voorbeeldjes via ctrl-shift-find) Als je echt wil om deze te implementeren, dan kan deze boring best voorgesteld worden do...

Al opgelost?
Indien niet, dan mag je een "UnimplementedException" throwen (voorbeeldjes via ctrl-shift-find)
Als je echt wil om deze te implementeren, dan kan deze boring best voorgesteld worden door de combinatie van een "interne zijboring" + een "externe zijboring".