I think Adrian Kuhn did that in his SUnit work. I also remember he also introduced a difference between expectedFailures and expectedErrors.

Doru


On 21 Apr 2010, at 10:16, Stéphane Ducasse wrote:


On Apr 21, 2010, at 9:51 AM, Adrian Lienhard wrote:

Yea, I agree, the GUI is suboptimal.

I still think, though, that treating this case as a failure is correct. For instance, consider the case where you had added a workaround to a known bug and when the bug is fixed you need to remove the workaround again. Maybe it even leads to a wrong behavior now that the bug is gone. In this case you really want to know that the test does not fail anymore.

yes
Now I have the impression that expectedFailures should be like passes, failed, errors: a state of the tests.

Stef

In any case, I think that tagging methods as expected failures should be done with pragmas and not with #expectedFailures. Like this it would also be much easier to understand what's going on when you have a failure in this test although all assertions pass.

Adrian

On Apr 21, 2010, at 08:22 , Stéphane Ducasse wrote:


On Apr 20, 2010, at 11:20 PM, Adrian Lienhard wrote:

Yes, if a test that is expected to fail does not fail, this is treated as a failure. I think that makes sense.

well it depends about the scenario.
you put on expectedfailures something that gets in your way now, so after if it works even better. of course you should get notified that the test is green while expected it to failed.

Now it leads to a UI problem where you have a failure that passes so when you click on it nothing happens: no debugger. And you can wonder why the hell do I have a failure when my tests pass.

So I think that this implementation of expectedFailures is a hack.


Adrian

On Apr 20, 2010, at 21:57 , Stéphane Ducasse wrote:

Hi

I tagged some tests as expected failures and I got a strange behavior.
On the the tests which was passing was listed under the failures.
When I renamed the method without updating the expected failures my bar was green. So expected failures really expect that the tests failed? We cannto have green tests in there?

Stef
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo- project


_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

--
www.tudorgirba.com

"Live like you mean it."


_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Reply via email to