[Seaside-dev] self shouldnt: [ whatever ] raise: Error.

[Philippe Marschall]

On Sun, Oct 20, 2013 at 4:17 PM, Johan Brichau <johan at inceptive.be> wrote:
> Indeed.
>
> In the case of the now failing tests in Seaside, it is actually meaningful to test if no error is thrown. The assertion is a documentation what is being tested.
> At first thought, I had the idea that it was easy to remove the assertion since the test will fail if an error is thrown anyway. However, without the assertion, the meaning of the test now has to be written in comments.

Exactly, it's the difference between a failure and an error.

> The spirit behind the idea seems good, but enforcing it like this is counter-productive in some cases.

Exactly, enterprise architect thinking.

Cheers
Philippe