Unit Tests/Rueckgabewert Funktion bei Exception

[Eclipticon]

Hi,

bei dem am Treffen vorgestellten Beispielen zu Unit Tests war auch ein Test enthalten, bei dem der Funktionsaufruf eine Exception ausloesen sollte. In weiterer Folge wurde in try ... except mit AssertEquals geprueft, ob der Rueckgabewert der Funktion auch 0 ist. (m.schnell, bist du einverstanden, wenn ich das Beispiel aus calculationservicetests hier poste?)

Hierzu zwei Fragen:

* Ist der Rueckgabewert einer Funktion ueberhaupt definiert, wenn sie eine unbehandelte Exception ausloest?

* Welchen Sinn macht es, diesen Wert zu pruefen?

Danke

[m.fuchs]

(m.schnell, bist du einverstanden, wenn ich das Beispiel aus calculationservicetests hier poste?)

Ich nehme mal an, dass du mich meinst: ja, das ist kein Problem. Ist ja ein Beispiel.

* Ist der Rueckgabewert einer Funktion ueberhaupt definiert, wenn sie eine unbehandelte Exception ausloest?

Der Rückgabewert ist durchaus definiert, man kann ihn ja jederzeit in die Result-Variable schreiben. ABER: der Rückgabewert wird nirgends zugewiesen, weil da ja schon die Exception vorher auftaucht. Woraus folgt...

* Welchen Sinn macht es, diesen Wert zu pruefen?

...eigentlich ist das sinnfrei. Interessant wäre so eine Prüfung höchstens außerhalb des except-Blocks, um auch den Fall abzufangen dass keine Exception geworfen wurde. Andererseits wird dieser Fehlerfall ja auch schon durch das

Code: Alles auswählen

Fail('No exception raised.');

abgedeckt. Von daher, die Prüfung auf 0 kann raus.

[Eclipticon]

Ja, ich mein Dich ... sorry, offenbar sollte ich um diese Uhrzeit keine Postings mehr schreiben, kommt nur Bloedsinn dabei raus.

Hier noch das betreffende Stueck Code ...

Code: Alles auswählen

procedure TCalculationServiceTests.ExceptionOnNegativeKapitalTest;
var
  FunctionResult: Currency = 0;
begin
  try
    FunctionResult := Service.BerechneZinsen(-1, 3.4, 5);
    Fail('No exception raised.');
  except
    on e:ECalculate do begin
      Assert(e.ExceptionType = cetKapitalNegativ);
      AssertEquals(0, FunctionResult); // Die Zeile hier war gemeint
    end else
      Fail('Wrong exception raised.');
  end;
end;
 

... und danke!