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

[UW355] Uitfaseren ladediepte 300
[UW355] Uitfaseren ladediepte 300
  • More
  • CR-143
  • finished reviewing
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

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.

[UW355] Uitfaseren ladediepte 300
[UW355] Uitfaseren ladediepte 300
[UW355] Uitfaseren ladediepte 300

- Uitfaseren ladediepte 300 is toegevoegd aan lijst.

  1. … 1 more file in changeset.
[UW355] Uitfaseren ladediepte 300

- herwerken naar lijst

  1. … 4 more files in changeset.