Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 456738

Summary:	[junit] TraceFile tests cause failures
Product:	[Tools] CDT	Reporter:	Marc Khouzam <marc.khouzam>
Component:	cdt-debug-dsf-gdb	Assignee:	Marc Khouzam <marc.khouzam>
Status:	RESOLVED FIXED	QA Contact:	Marc Khouzam <marc.khouzam>
Severity:	normal
Priority:	P3	CC:	cdtdoug, nobody, pawel.1.piech
Version:	8.5
Target Milestone:	8.6.0
Hardware:	PC
OS:	Linux
Whiteboard:

Description Marc Khouzam

2015-01-05 21:59:04 EST

The Junit tests from TraceFileTest_7_4 are causing trouble for other tests.  The reason is that they create tracepoints but do not clean them up, so future breakpoints tests (when running AllSuitesRemote) fail because of these extra breakpoints.

Also, the tests in TraceFileTest_7_4 are dependent on each other but JUnit no longer enforces the order, so we have to do it ourselves.

Comment 1 Marc Khouzam

2015-01-05 22:04:36 EST

I pushed a fix to Gerrit:
  https://git.eclipse.org/r/39013

This runs the tests in order, and removes all breakpoints and tracepoint actions using @AfterClass.

I think that a better solution would be to make each test independent.  However, as I'm trying to get the AllSuitesRemote to work, I think this solution is sufficient for now.

Comment 2 Marc Khouzam

2015-01-07 14:53:04 EST

Committed to master after Mikhail's review.