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

Bug 224131

Summary: InactiveProcessException when launching AGR tests using the Agent Controller.
Product: z_Archived Reporter: Paul Slauenwhite <paulslau>
Component: TPTPAssignee: Paul Slauenwhite <paulslau>
Status: CLOSED FIXED QA Contact:
Severity: blocker    
Priority: P1 CC: apnan, dmorris
Version: unspecifiedFlags: paulslau: review+
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch. none

Description Paul Slauenwhite CLA 2008-03-26 11:57:21 EDT 
InactiveProcessException when launching AGR tests using the Agent Controller.

Using the 4.5.0-200803241439 build (4.5 I6 TP1 candidate), the following exception is thrown when launching an AGR test suite using the Agent Controller:


!ENTRY org.eclipse.hyades.test.core 1 0 2008-03-26 12:23:59.078
!MESSAGE org.eclipse.hyades.internal.execution.local.control.InactiveProcessException
!STACK 0
org.eclipse.hyades.internal.execution.local.control.InactiveProcessException
	at org.eclipse.hyades.execution.harness.TestExecutionHarnessExecutorStub.launch(TestExecutionHarnessExecutorStub.java:422)
	at org.eclipse.hyades.execution.harness.TestExecutionHarness.launchTestExecution(TestExecutionHarness.java:2072)
	at org.eclipse.hyades.execution.harness.TestExecutionHarness.access$1(TestExecutionHarness.java:1804)
	at org.eclipse.hyades.execution.harness.TestExecutionHarness$2.run(TestExecutionHarness.java:724)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)

!ENTRY org.eclipse.hyades.test.core 1 0 2008-03-26 12:23:59.078
!MESSAGE java.lang.RuntimeException: org.eclipse.hyades.internal.execution.local.control.InactiveProcessException
!STACK 0
java.lang.RuntimeException: org.eclipse.hyades.internal.execution.local.control.InactiveProcessException
	at org.eclipse.hyades.execution.harness.TestExecutionHarnessExecutorStub.handleException(TestExecutionHarnessExecutorStub.java:347)
	at org.eclipse.hyades.execution.harness.TestExecutionHarnessExecutorStub.launch(TestExecutionHarnessExecutorStub.java:434)
	at org.eclipse.hyades.execution.harness.TestExecutionHarness.launchTestExecution(TestExecutionHarness.java:2072)
	at org.eclipse.hyades.execution.harness.TestExecutionHarness.access$1(TestExecutionHarness.java:1804)
	at org.eclipse.hyades.execution.harness.TestExecutionHarness$2.run(TestExecutionHarness.java:724)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)
Caused by: org.eclipse.hyades.internal.execution.local.control.InactiveProcessException
	at org.eclipse.hyades.execution.harness.TestExecutionHarnessExecutorStub.launch(TestExecutionHarnessExecutorStub.java:422)
	... 4 more

!ENTRY org.eclipse.hyades.test.core 4 0 2008-03-26 12:23:59.156
!MESSAGE IWAT4039E The Test Execution Harness reported the following problems:org.eclipse.hyades.internal.execution.local.control.InactiveProcessException
Comment 1 Paul Slauenwhite CLA 2008-03-26 11:58:00 EDT 
This defect is blocking 4.5 I6 TP1.
Comment 2 Paul Slauenwhite CLA 2008-03-28 16:02:49 EDT 
The problem is that the org.eclipse.tptp.test.auto.gui.internal.runner.AutoGUIExecObjAdapter.java does not send the implementor ID as the test ID to the org.eclipse.tptp.test.auto.gui.internal.runner.AutoGUIRunner.java.  The new encrypted datapool support (see defect #202695) expected the test runner to have a non-null test ID, causing a NPE on the concat() method in org.eclipse.hyades.test.common.testservices.resources.DatapoolPasswordProvider.getDatapoolPassword(String).
Comment 3 Paul Slauenwhite CLA 2008-03-28 16:12:01 EDT 
Created attachment 94060 [details]
Patch.
Comment 4 Paul Slauenwhite CLA 2008-03-28 16:14:28 EDT 
Note, Duwayne contributed several hours in the triage of this defect.

Updated hours worked by me on triage and the patch.

Joe, can you review the patch?
Comment 5 Alex Nan CLA 2008-03-28 16:42:43 EDT 
Patch looks good.
Comment 6 Paul Slauenwhite CLA 2008-03-28 17:22:21 EDT 
Patch approved by the PMC and checked in to CVS (HEAD).
Comment 7 Paul Slauenwhite CLA 2008-04-01 11:06:17 EDT 
Verified with the TPTP-4.5.0-200803310100 build.  

Closing.
Comment 8 Paul Slauenwhite CLA 2008-11-27 19:20:27 EST 
(In reply to comment #5)
> Patch looks good.
> 

Approved.