> On 13 Jan 2016, at 18:42, abdelghani ALIDRA <alidran...@yahoo.fr> wrote:
> 
> Thank you all for your responses.
> 
> I see your point about returning from a block. Because the return in this 
> example is very explicit. But what if I put (by mistake) a return in a block 
> in a method that is called by the test method (or even in a method that is 
> called by another method that is called by the test method) then I would have 
> no reason to doubt my test was successful whereas it would have failed if it 
> was run till the last assertion.
> 
> My point is, would it not be interesting to report tests that were not run 
> successfully till the end?

An empty test is successful.
A test only fails when an assertion fails.
Assertions must be written explicitly.

Your suggestion is not per se a bad idea, it is just not within the contract of 
SUnit, I think.

> Abdelghani
> 
> 
> De : Sven Van Caekenberghe <s...@stfx.eu>
> À : abdelghani ALIDRA <alidran...@yahoo.fr>; Any question about pharo is 
> welcome <pharo-users@lists.pharo.org> 
> Envoyé le : Mardi 12 janvier 2016 7h35
> Objet : Re: [Pharo-users] Failling Tests are green if BlockCannotReturn 
> exception
> 
> 
> 
> 
> > On 12 Jan 2016, at 00:26, abdelghani ALIDRA <alidran...@yahoo.fr> wrote:
> > 
> > Hi,
> > 
> > I observed this unexpected behavior in test classes.
> > In a test class define a method :
> > 
> > testBlock
> >    |aBlock|
> >    aBlock := [ ^1 ].
> >    aBlock value.
> >    self assert: false.
> > 
> > Althought the assertion is false at the end of the test, the test is green.
> > Actually, It does not matter what you put after aBlock value, the test 
> > always succedes (I tried to put a self halt, it does not execute)
> 
> The test is 'does an explicit return from a block return from the whole 
> method'. Of course it does. But if it would not, you would arrive at the self 
> assert: false and the test would fail. You see ?
> 
> BTW, this can also be written as self fail - search for the senders of that 
> message for more examples.
> 
> 
> > I tried this both in Pharo 4 and 5 under Windows and MacOS.
> > 
> > Any ideas?
> > 
> > Cheers 
> > Abdelghani
> > 
> > 
> 
> 


Reply via email to