Author: koen.aers(a)jboss.com Date: 2011-07-05 12:26:51 -0400 (Tue, 05 Jul 2011) New Revision: 32618 Modified: trunk/forge/plugins/org.jboss.tools.forge.core/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntime.java trunk/forge/tests/org.jboss.tools.forge.core.test/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntimeTest.java Log: starting notification should happen when process is already initialized Modified: trunk/forge/plugins/org.jboss.tools.forge.core/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntime.java =================================================================== --- trunk/forge/plugins/org.jboss.tools.forge.core/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntime.java 2011-07-05 16:19:02 UTC (rev 32617) +++ trunk/forge/plugins/org.jboss.tools.forge.core/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntime.java 2011-07-05 16:26:51 UTC (rev 32618) @@ -35,10 +35,10 @@ } try { progressMonitor.beginTask("Starting Forge", IProgressMonitor.UNKNOWN); + streamListener = new StartupListener(); + process = ForgeLaunchHelper.launch(getName(), getLocation()); state = STATE_STARTING; propertyChangeSupport.firePropertyChange(PROPERTY_STATE, STATE_NOT_RUNNING, STATE_STARTING); - streamListener = new StartupListener(); - process = ForgeLaunchHelper.launch(getName(), getLocation()); process.getStreamsProxy().getOutputStreamMonitor().addListener(streamListener); progressMonitor.worked(1); while (STATE_STARTING.equals(state)) { Modified: trunk/forge/tests/org.jboss.tools.forge.core.test/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntimeTest.java =================================================================== --- trunk/forge/tests/org.jboss.tools.forge.core.test/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntimeTest.java 2011-07-05 16:19:02 UTC (rev 32617) +++ trunk/forge/tests/org.jboss.tools.forge.core.test/src/org/jboss/tools/forge/core/process/ForgeAbstractRuntimeTest.java 2011-07-05 16:26:51 UTC (rev 32618) @@ -132,6 +132,9 @@ private class TestPropertyChangeListener implements PropertyChangeListener { public void propertyChange(PropertyChangeEvent evt) { propertyChangeEvents.add(evt); + if (ForgeRuntime.STATE_STARTING.equals(evt.getNewValue())) { + assertNotNull(runtime.getProcess()); + } } }