Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 353987 - EDC tests timeout when files are missing rather than failing when lack of file detected
Summary: EDC tests timeout when files are missing rather than failing when lack of fil...
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-edc (show other bugs)
Version: 8.0   Edit
Hardware: All All
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Ken Ryall CLA
QA Contact: Ken Ryall CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-05 05:52 EDT by Daniel Thomas CLA
Modified: 2012-05-22 20:40 EDT (History)
2 users (show)

See Also:

Attachments
Patch to detect lack of file problems earlier and take appropriate action. (3.34 KB, patch)
2011-08-05 05:53 EDT, Daniel Thomas CLA 		cdtdoug: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Thomas CLA 2011-08-05 05:52:14 EDT 
Build Identifier: 

Some of the EDC tests fail by timing out when expected files don't exist rather than failing as soon as the lack of file is detected. This means that the tests may appear to hang or at least take a very long time.
Affected tests:
org.eclipse.cdt.debug.edc.debugger.tests.Concurrent
org.eclipse.cdt.debug.edc.debugger.tests.OpaqueTypeResolving
org.eclipse.cdt.debug.edc.debugger.tests.RegisterView

I have a patch which tests for the existence of files earlier or throws exceptions at the relevant point so that they fail when the lack of file is first detected.

Reproducible: Always

Steps to Reproduce:
1. Run AllEDCTests on x86_64 and see that Concurrent and RegisterView timeout
2. Run OpaqueTypeResolving on a case sensitive OS and watch it spend 2 minutes timing out. (separate bug and patch filed to fix the problem causing this timeout)
Comment 1 Daniel Thomas CLA 2011-08-05 05:53:30 EDT 
Created attachment 200965 [details]
Patch to detect lack of file problems earlier and take appropriate action.
Comment 2 Kirk Beitz CLA 2011-08-06 18:16:20 EDT 
patch applied and nominally tested locally; awaiting final review and final
merge to eclipse edc git repository by committer ken ryall.