Bug 210792 - Test suites killed by timeout must appear red
Summary: Test suites killed by timeout must appear red
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 3.4   Edit
Hardware: All All
: P2 major (vote)
Target Milestone: ---   Edit
Assignee: Sravan Kumar Lakkimsetti CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
: 296060 (view as bug list)
Depends on:
Blocks: 423561
  Show dependency tree
 
Reported: 2007-11-23 11:16 EST by Dani Megert CLA
Modified: 2020-11-24 01:31 EST (History)
5 users (show)

See Also:


Attachments
Consolelog with timeout (11.07 KB, text/plain)
2007-11-23 11:37 EST, Markus Keller CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2007-11-23 11:16:05 EST
I20071120-1300

Test suites killed by timeout should appear red.

Due to a bug on the Mac one of our tests hug and hence made the tests wait for 2hrs holding off the final results for everybody. Unfortunately we did not notice this so far because all was green.

We should asap make killed test suites appear red.

Idea: create a simple JUnit test that fails saying 'test suite $testSuiteName killed after timeout of $timeoutVal'. If a test suite is killed then run this dummy JUnit tests via ant task and make sure the output file has the expected name.

I marked this as 'major' as this can cause the whole Eclipse test run to wait for 2hrs without being noticed.
Comment 1 Markus Keller CLA 2007-11-23 11:37:47 EST
Created attachment 83657 [details]
Consolelog with timeout

Here's the relevant extract of a consolelog.txt for a test that failed with time out.
Comment 2 Kim Moir CLA 2007-11-26 16:30:08 EST
Another alternative would be use an ant listener to implement this, I'm investigating.
Comment 3 Mike Wilson CLA 2008-09-11 16:54:25 EDT
Can we fix this for 3.5?

Comment 4 Dani Megert CLA 2009-08-10 05:21:16 EDT
Ping. Kim, it would be good to get this fixed.
Comment 5 Markus Keller CLA 2009-11-25 10:11:56 EST
*** Bug 296060 has been marked as a duplicate of this bug. ***
Comment 6 Dani Megert CLA 2010-06-22 03:33:51 EDT
Ping ;-)
Comment 7 Dani Megert CLA 2010-08-10 06:13:40 EDT
I have to mark this as blocker since the consequences of this bug are very bad: I just found out that there's another test (suite) which gets killed in every build including 3.5. This means that this always contributes 2 hours to each test run without producing anything useful.
Comment 8 John Arthorne CLA 2011-05-27 17:01:15 EDT
Is this really a blocker.. it's hard to see how a bug that is four years old blocks our development.
Comment 9 Dani Megert CLA 2011-08-16 08:47:58 EDT
(In reply to comment #8)
> Is this really a blocker.. it's hard to see how a bug that is four years old
> blocks our development.

Well, it blocked the results from appearing for 2 hours for every  build over more than a year.

The bad part is that this can happen again at any time and we won't notice it.
Comment 10 David Williams CLA 2013-05-09 13:59:49 EDT
Looks like this has been deferred several times already ... and, I will once again. But, will look to improve in Luna ... if nothing else, distinguish the different types of "DNFs" (for which there is another bug).
Comment 11 Markus Keller CLA 2013-09-27 12:54:11 EDT
Interesting observation from bug 417928 / bug 418101:

The test suite should appear read if
- the <java> Ant task killed the test run OR
- org.eclipse.test.EclipseTestRunner#startStackDumpTimoutTimer() calls dump(0)

In the bug 418101 scenario, the call to dump(0) causes a call to display#syncExec(..), which wakes the Display from its 2h-epsilon sleep. The 5s before the second call to dump(..) are enough to make the whole test suite finish and appear in a shiny green. In the end, the test is NOT killed by the Ant task timeout, but turned green by the observer -- a perfect heisenbug.
Comment 12 David Williams CLA 2013-09-29 18:21:09 EDT
(In reply to Markus Keller from comment #11)
> Interesting observation from bug 417928 / bug 418101:
> 
> The test suite should appear read if
> - the <java> Ant task killed the test run OR
> - org.eclipse.test.EclipseTestRunner#startStackDumpTimoutTimer() calls
> dump(0)
> 

Thanks for finding and documenting this issue ... I was content to leave it as a mystery. :)  I fear that the first condition above ("Ant task killed the test run") is currently simply "inferred" by the absence of test results (which is what leads to bug 399636) so my guess is we need to stop relying on "absence of results" for anything, and simply learn how to scan the log(s) for "timed out" (and other, serious) error messages.
Comment 13 Markus Keller CLA 2013-09-30 05:58:48 EDT
I think we really have to perform both checks. The "absence of results" could also happen for other reasons, e.g. VM crash, System.exit(), hardware failure, ...

> simply learn how to scan the
> log(s) for "timed out" (and other, serious) error messages.

Probably a better solution would be to write out a special file <testname>.timeout.txt to the consolelogs directory. That file could also contain the stacktraces, making them easier to find.
Comment 14 David Williams CLA 2014-05-29 04:39:31 EDT
Nothing planned for Luna SR0, so untargettng.
Comment 15 Sravan Kumar Lakkimsetti CLA 2017-05-25 07:00:44 EDT
Re targetting to 4.8
Comment 16 Sravan Kumar Lakkimsetti CLA 2018-08-23 02:49:18 EDT
There are plans for this. Will revisit in 4.10
Comment 17 Sravan Kumar Lakkimsetti CLA 2018-12-04 07:08:44 EST
Moving out of 4.10. Please re triage appropriately
Comment 18 Eclipse Genie CLA 2020-11-24 01:31:33 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.