Activity CacheAdminA_trunk2010/BL/Legacy/FLOWHALUX.cls.xml

4:13 pm

Tom Vermeulen

More
Re: CR-2266
commented on review

hier kunt ge ook AssertTrue (of AssertFalse) gebruiken.

1:30 pm

Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstre...

Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstreeks de ToeleveringService te laten gebruiken. Dan skippen we dit gewoon.

1:24 pm

Tom Vermeulen

More
Re: CR-2266
commented on review

ge hebt bij de oproepende klasse wel testen voorzien die checken of deze wordt opgeroepen, maar er zijn (nog) geen testen voor deze method zelf

11:25 am

Tom Vermeulen

More
Re: CR-2266
commented on review

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client g...

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client gaat doen bij een niet-geïnitialiseerde variabele. Aangezien je in dit concreet geval de client ook onder controle hebt, kan je ermee wegkomen. Persoonlijk zou ik in DnaCodeAfmeldenResponse een defaultvalue False voorzien voor de nieuwe property.

11:16 am