CacheAdminA_trunk2010/APPS/PM/Maatwerk/impl

IngegevenKenmerkTIPONUitfaseringVisitor.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 21 Mar 17
Commits this week: 0
Total Lines of Code (LoC): 42
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
Wednesday 17 May 2017
8:29 am

Kurt Callebaut

Friday 12 May 2017
8:02 pm

Wim Vermeulen

  • More
  • CR-217
  • finished reviewing
8:02 pm

Wim Vermeulen

In dit soort unittest steeds een positieve case en een negatieve case behouden (dit is een algemene regel) Hier komt dit neer op : minstens één testmethod ZONDER exceptie en één MET exceptie. Ge m...

In dit soort unittest steeds een positieve case en een negatieve case behouden (dit is een algemene regel)
Hier komt dit neer op : minstens één testmethod ZONDER exceptie en één MET exceptie.

Ge moet in deze test niks meer aanpassen, aangezien TIPON volledig uitgefaseerd wordt.
dus : Review OK

7:55 pm

Wim Vermeulen

[UW355] Uitfaseren ladediepte 300
[UW355] Uitfaseren ladediepte 300
Thursday 30 Mar 2017
8:37 am

Kurt Callebaut

Wednesday 29 Mar 2017
5:09 pm

Wim Vermeulen

  • More
  • CR-143
  • finished reviewing
5:08 pm

Wim Vermeulen

Qua naamgeving kan dit duidelijker : de klasse is UitfaserenTipon en de method is GeefLadeDiepteList --> zijn dit de "nog beschikbare" of de "reeds uitgefaseerde"? Ikzelf weet het antwoord, maar ie...

Qua naamgeving kan dit duidelijker :
de klasse is UitfaserenTipon en de method is GeefLadeDiepteList --> zijn dit de "nog beschikbare" of de "reeds uitgefaseerde"? Ikzelf weet het antwoord, maar iemand anders zou nog kunnen twijfelen :-P

Laat voorlopig maar zo staan, hoor

5:05 pm

Wim Vermeulen

Een basisprincipe van Unittesten is dat je een positief en een negatief scenario moet afdekken (in dit geval : een correcte assert en een verwachte exceptie) Deze test deed dit effectief goed. Maar...

Een basisprincipe van Unittesten is dat je een positief en een negatief scenario moet afdekken (in dit geval : een correcte assert en een verwachte exceptie)
Deze test deed dit effectief goed.
Maar door het uitfaseren van Tipon 300 faalt uw test.
Mijn idee is dat je een andere ladediepte zou kiezen (om het positief scenario te behouden), i.p.v. er een verwachte exceptie van te maken. Want nu heb je 2 negatieve scenario's en geen positieve.

Tuesday 28 Mar 2017
10:35 am

Kurt Callebaut

[UW355] Uitfaseren ladediepte 300
[UW355] Uitfaseren ladediepte 300
Tuesday 21 Mar 2017
2:39 pm

KurtC

[UW355] Uitfaseren ladediepte 300

- Het moet ook LBX zijn...

Options
    • -1
    • +1
    ./IngegevenKenmerkTIPONUitfaseringVisitor.cls.xml
  2. … 5 more files in changeset.
1:45 pm

KurtC

[UW355] Uitfaseren ladediepte 300

- herwerken naar lijst

Options
    • -1
    • +1
    ./IngegevenKenmerkTIPONUitfaseringVisitor.cls.xml
  2. … 4 more files in changeset.
Friday 17 Mar 2017
3:29 pm

Kurt Callebaut

2:00 pm

Jo Claes

  • More
  • CR-129
  • finished reviewing
Tuesday 14 Mar 2017
2:47 pm

Kurt Callebaut

[BUS601] LBX TIPON bij bestellen tegenhouden (gefaseerd)
[BUS601] LBX TIPON bij bestellen tegenhouden (gefaseerd)
2:25 pm

KurtC

[BUS601] LBX TIPON bij bestellen tegenhouden (gefaseerd)

- TIPONUitfasering visitors -> LBX, Maatwerklijnen, TIPON, 350,500,650 + test ondanks dit maar tijdelijk is.

- Todo Melding / vertaling.

Options
    • -0
    • +42
    ./IngegevenKenmerkTIPONUitfaseringVisitor.cls.xml
  2. … 3 more files in changeset.