Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3537] Langtekst LBX: Space Step configurator toont geen lade als de kleur CS-M is
[ICT-3537] Langtekst LBX: Space Step configurator toont geen lade als de kleur CS-M is
Beter om de return-waarde ook nog te controleren. Dus : ... DanReturn("UT_SpaceStepImage") $$$AssertEquals(result, "UT_SpaceStepImage") Reden: De SSTImageCodeBepaler is de zogenaamde "black box" d...

Beter om de return-waarde ook nog te controleren.
Dus :
... DanReturn("UT_SpaceStepImage")
$$$AssertEquals(result, "UT_SpaceStepImage")

Reden:
De SSTImageCodeBepaler is de zogenaamde "black box" die een (willekeurig) resultaat teruggeeft. Het is een quick-win om deze te controleren als result van de "method under test".
Andere insteek :
ik kan de code in AfbeeldingCodeBepaler zeer gemakkelijk kapot maken zonder dat deze test dit aan het licht brengt. Door het resultaat te controleren, wordt het heel wat moeilijker om ditzelfde kapot te maken.

[ICT-1728] Implementatie afbeeldingcodebepaler SpaceStep
[ICT-1728] Implementatie afbeeldingcodebepaler SpaceStep
[UST4096] LBX: MeenemerSet voor binnenlade: APPS-enum - default van leeg naar Zonder:
[UST4096] LBX: MeenemerSet voor binnenlade: APPS-enum - default van leeg naar Zonder:
Moet DOM enu zijn. en dan zal bijhorende test ook moeten aangepast worden. Zie mail.

Moet DOM enu zijn.
en dan zal bijhorende test ook moeten aangepast worden. Zie mail.

Ene commit niet toegevoegd (met enkel testbestanden => IntegratieTest => Maatwerk.Configurator), wegens beperking op maximum toegelaten toegevoegde files.

Ene commit niet toegevoegd (met enkel testbestanden => IntegratieTest => Maatwerk.Configurator), wegens beperking op maximum toegelaten toegevoegde files.

[UST4056] LBX: MeenemerSet voor binnenlade: APPS dto-kenmerk:
[UST4056] LBX: MeenemerSet voor binnenlade: APPS dto-kenmerk:
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
  • More
  • CR-594
  • finished reviewing
"Voor Wand ... " Het is jammer dat je de foute schrijfwijze hebt voortgezet. Ik weet dat je voorstander bent van consistente spelling, maar in dit geval (van nieuwe implementatie) had je gerust kun...

"Voor Wand ... "
Het is jammer dat je de foute schrijfwijze hebt voortgezet.
Ik weet dat je voorstander bent van consistente spelling, maar in dit geval (van nieuwe implementatie) had je gerust kunnen omschakelen.
Zeker als je weet dat een Klasse-rename bijna ONmogelijk is als het alleen om een hoofdletter gaat.
Met andere woorden, we zullen nooit meer van die "VoorWant" vanaf geraken

Anyway, het staat nu zo, dus laat maar staan. Geen onnodige tijd meer aan verliezen!
Verder is de volledige review OK, denk

[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):
[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):