Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT851] TAXFP1: Activering intern:
[ICT851] TAXFP1: Activering intern:
[ICT851] TAXFP1: Activering intern:

- Nog wat testjes gefixt

  1. … 3 more files in changeset.
  • More
  • CR-239
  • finished reviewing
[BUS618] Probox vanaf 4 lades
[BUS618] Probox vanaf 4 lades
[BUS618] Probox vanaf 4 lades

- Overbodige opties en testjes weggezwierd

  1. … 3 more files in changeset.
[BUS618] Probox vanaf 4 lades

- 5->4 en testjes

  1. … 5 more files in changeset.
  • More
  • CR-203
  • finished reviewing
HIer niet beter de valuelist van de enumeratie gebruiken zodoende we duidelijk kunnen zien dat alle lade dieptes van toepassing ..

HIer niet beter de valuelist van de enumeratie gebruiken zodoende we duidelijk kunnen zien dat alle lade dieptes van toepassing ..

  • 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
[BUS615] Uitfaseren Tipon (volledig) bij LBX

- ladediepte 270 ook, staat al in prod.

  1. … 1 more file in changeset.
[BUS6515] - Uitfaseren Tipon (volledig) bij LBX
[BUS6515] - Uitfaseren Tipon (volledig) bij LBX
[BUS615] Uitfaseren Tipon (volledig) bij LBX

- stap 1 staat al in productie.

- straks knop weghalen en deze code mag dan verwijderd worden.

  1. … 1 more file in changeset.
update uitfasering tipon. testen tijdelijk niet nodig.
Uitfaseren ladediepte 600 Legrabox -(staat al in productie.)
  1. … 1 more file in changeset.
[UW355] Uitfaseren ladediepte 300

- alle positieve scenario's afdekken in de testen.

  • More
  • CR-95
  • finished reviewing
  • 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
LBX LadeDiepte 450 TIPON uitfaseren. (reeds op cache01 doorgevoerd.) - test