Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2419] Matjes hebben geen afbeelding op documenten

- reviewopmerkingen

  1. … 1 more file in changeset.
Je hebt een aantal imagecodes gecontroleerd, dat is oké. Wanneer er nu een nieuwe MatKleur (of materiaal) bijkomt, dan zal deze test niet beginnen te falen. Logisch! Als we zoiets wel willen, dan m...

Je hebt een aantal imagecodes gecontroleerd, dat is oké.
Wanneer er nu een nieuwe MatKleur (of materiaal) bijkomt, dan zal deze test niet beginnen te falen. Logisch!
Als we zoiets wel willen, dan moeten we daarvoor een "SmokeTest" schrijven. Ik heb effe nagedacht, maar dat zou niet zo simpel zijn (gezien de combinatie van Materiaal en Kleur). Dus ik stel voor om het zo te laten, tenzij je zelf een briljant idee hebt.

Commentaar opkuisen :-P

Commentaar opkuisen :-P

... geeft lege AfbeeldingCode."

... geeft lege AfbeeldingCode."

Ter informatie (om misverstanden in de toekomst te vermijden) : Bij materiaal bestaat "antislip" en "antislip-Fiber" Voor deze testmethods (naam) maakt het echt ni uit (dus ge hoeft het ni per sé ...

Ter informatie (om misverstanden in de toekomst te vermijden) :
Bij materiaal bestaat "antislip" en "antislip-Fiber"

Voor deze testmethods (naam) maakt het echt ni uit (dus ge hoeft het ni per sé aan te passen) maar let wel op als ge met andere mensen spreekt over "Antislip" terwijl ge "AntislipFiber" zou bedoelen.

$$$Implementation( MATImageCodeBepaler (i.p.v. "SST...") en ook : in nieuwe code gebruiken we $$$Inject i.p.v. $$$implementation

$$$Implementation( MATImageCodeBepaler (i.p.v. "SST...")

en ook :
in nieuwe code gebruiken we $$$Inject i.p.v. $$$implementation

FormalSpec is copy-paste van LBX --> aanpassen naar MATImageCodeBepaler

FormalSpec is copy-paste van LBX --> aanpassen naar MATImageCodeBepaler

[ICT-2419] Matjes hebben geen afbeelding op documenten
[ICT-2419] Matjes hebben geen afbeelding op documenten
[ICT-2419] Matjes hebben geen afbeelding op documenten

- MatImageCodeBepaler toegevoegd

  1. ./impl/MATImageCodeBepaler/BepaalMatAfbeelding
  2. … 3 more files in changeset.
Kan je de laatste 5 svn-commits ook nog toevoegen?

Kan je de laatste 5 svn-commits ook nog toevoegen?

  • More
  • CR-571
  • finished reviewing
[EPIC52][UST2900] Afbeeldingcode bepaling
[EPIC52][UST2900] Afbeeldingcode bepaling
[EPIC52][UST2900] Afbeeldingcode bepaling:

- tests toegevoegd LBXNietVolledigeLadeImageCodeBepaler

  1. … 1 more file in changeset.
[UW179] - Locking in test framework

- Correct aanroepen van #super in OnBefore - OnAfter