use existing elasticsearch node in test framework could still spawn a new java process

Mpdreamz · Mpdreamz · commit 6a2122d262b2 · 2016-09-27T11:15:07.000+02:00
diff --git a/src/Tests/Framework/ManagedElasticsearch/Process/ElasticsearchNode.cs b/src/Tests/Framework/ManagedElasticsearch/Process/ElasticsearchNode.cs
@@ -185,7 +185,7 @@ private IObservable<ElasticsearchConsoleOut> UseAlreadyRunningInstance(XplatManu
 			this.WaitForClusterBootstrap(client, handle, alreadyRunningInstance: true);
 
 			this.ValidateLicense(client, handle);
-			return null;
+			return Observable.Empty<ElasticsearchConsoleOut>();
 		}
 
 		private bool ValidateRunningVersion(IElasticClient client, XplatManualResetEvent handle)

Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ private IObservable<ElasticsearchConsoleOut> UseAlreadyRunningInstance(XplatManu`
`185`	`185`	`this.WaitForClusterBootstrap(client, handle, alreadyRunningInstance: true);`
`186`	`186`
`187`	`187`	`this.ValidateLicense(client, handle);`
`188`		`- return null;`
	`188`	`+ return Observable.Empty<ElasticsearchConsoleOut>();`
`189`	`189`	`}`
`190`	`190`
`191`	`191`	`private bool ValidateRunningVersion(IElasticClient client, XplatManualResetEvent handle)`