Remove the executor named parameter from constructor

internal/factsengine/gatherers/saphostctrl.go

@@ -9,12 +9,12 @@ const (
 )
 
 type SapHostCtrlGatherer struct {
-	executor CommandExecutor
+	CommandExecutor
 }
 
 func NewSapHostCtrlGatherer() *SapHostCtrlGatherer {
 	return &SapHostCtrlGatherer{
-		executor: Executor{},
+		Executor{},
 	}
 }
 
@@ -24,7 +24,7 @@ func (g *SapHostCtrlGatherer) Gather(factsRequests []FactRequest) ([]Fact, error
 
 	for _, factReq := range factsRequests {
 		// TODO: The instance number "00" could be an additional argument in the future
-		version, err := g.executor.Exec(
+		version, err := g.Exec(
 			"/usr/sap/hostctrl/exe/saphostctrl", "-nr", "00", "-function", factReq.Argument)
 		if err != nil {
 			// TODO: Decide together with Wanda how to deal with errors. `err` field in the fact result?

internal/factsengine/gatherers/saphostctrl_test.go

@@ -25,7 +25,7 @@ func (suite *SapHostCtrlTestSuite) TestSapHostCtrlGather() {
 		[]byte("ERROR"), nil)
 
 	p := &SapHostCtrlGatherer{
-		executor: mockExecutor,
+		mockExecutor,
 	}
 
 	factRequests := []FactRequest{
@@ -71,7 +71,7 @@ func (suite *SapHostCtrlTestSuite) TestSapHostCtrlGatherError() {
 		[]byte("some error"), errors.New("some error"))
 
 	p := &SapHostCtrlGatherer{
-		executor: mockExecutor,
+		mockExecutor,
 	}
 
 	factRequests := []FactRequest{