Re-using the same runner instance when executing specs after starting…

… api.

This ensures runner is not started twice during execution. Fixes #34

gauge.go

@@ -260,18 +260,17 @@ func executeSpecs(inParallel bool) {
 		handleCriticalError(err)
 	}
 	err, apiHandler := startAPIService(0)
-	apiHandler.runner.kill(getCurrentLogger())
 	if err != nil {
+		apiHandler.runner.kill(getCurrentLogger())
 		handleCriticalError(errors.New(fmt.Sprintf("Failed to start gauge API. %s\n", err.Error())))
 	}
-
-	runner, runnerError := startRunnerAndMakeConnection(manifest, getCurrentLogger())
-	if runnerError != nil {
-		handleCriticalError(errors.New(fmt.Sprintf("Failed to start a runner. %s\n", runnerError.Error())))
+	if apiHandler.runner == nil {
+		handleCriticalError(errors.New("Failed to start a runner\n"))
 	}
-	validateSpecs(manifest, specsToExecute, runner, conceptsDictionary)
+
+	validateSpecs(manifest, specsToExecute, apiHandler.runner, conceptsDictionary)
 	pluginHandler := startPlugins(manifest)
-	execution := newExecution(manifest, specsToExecute, runner, pluginHandler, parallelInfo, getCurrentLogger())
+	execution := newExecution(manifest, specsToExecute, apiHandler.runner, pluginHandler, parallelInfo, getCurrentLogger())
 	result := execution.start()
 	execution.finish()
 	exitCode := printExecutionStatus(result, specsSkipped)

specDetails.go

@@ -264,11 +264,9 @@ func (specInfoGatherer *specInfoGatherer) getStepsFromRunner(runner *testRunner)
 		if connErr == nil {
 			steps = append(steps, requestForSteps(runner)...)
 			logger.ApiLog.Debug("Steps got from runner: %v", steps)
-		}
-		if connErr != nil {
+		} else {
 			logger.ApiLog.Error("Runner connection failed: %s", connErr)
 		}
-
 	} else {
 		steps = append(steps, requestForSteps(runner)...)
 		logger.ApiLog.Debug("Steps got from runner: %v", steps)