Issue 5377 in pharo: Put a busy icon while running tests from nautilus

Status: Accepted
Owner: marianopeck
CC: [hidden email]
Labels: Type-Nautilus

New issue 5377 by marianopeck: Put a busy icon while running tests from  
nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

So that people don't expect that test run in background and you can do  
something else..... it would be more clear if we put a busy icon.


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

Comment #1 on issue 5377 by [hidden email]: Put a busy icon while  
running tests from nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

i have counterproposal: why not run test in background?


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

Comment #2 on issue 5377 by marianopeck: Put a busy icon while running  
tests from nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

That's an excellent question. I asked myself this several, several times.  
Even the same question when running from TestRunner. I always thought it  
was no possilbe or that there could be a problem with the priorities. I  
guess some UI tests may be affected if I do stuff at the same time?


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

Comment #3 on issue 5377 by [hidden email]: Put a busy icon while  
running tests from nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

yes, there are some tests which should run in UI thread.
as for others, i think it even better if they will run in separate thread  
at random priority, because it may help reveal issues which we were unaware  
of.

After all this is just tests. If they fail, we fix. If they break system,  
we fix tests/system.
There's nothing to fear about.


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

Comment #4 on issue 5377 by [hidden email]: Put a busy icon while  
running tests from nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

just an idea, we say that any TestCase subclass can run at any priority.
and UITestCase will run at UI process priority, so then we can just change  
a superclass of related tests to be a UITestCase, while rest will be  
unchanged.

This of course requires more thought , like waiting till test complete etc.
But there's already facilities for that in TestCase itself.


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

Comment #5 on issue 5377 by [hidden email]: Put a busy icon while  
running tests from nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

oh.. and sorry for turning Nautilus related issue into TestCase related  
one.. maybe we should split them :)


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

Updates:
        Status: Integrated

Comment #6 on issue 5377 by [hidden email]: Put a busy icon while  
running tests from nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

I just tried to run tests in background, and it seems to work :)

I think that til you do not change a test methods in the midtime, there is  
no big deal :)

I will open another bug entry for the Test Runner part ;)


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

Updates:
        Cc: [hidden email]

Comment #7 on issue 5377 by marianopeck: Put a busy icon while running  
tests from nautilus
http://code.google.com/p/pharo/issues/detail?id=5377

I think I agree with Igor. Moreover, now I remember that AutoTest (from  
Laurent Laffont) also used to run the tests in a different thread.


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