From b17acfe64d06f5aa3760861f1f9691086111d51c Mon Sep 17 00:00:00 2001
From: hanen mizouni <hanen.mizouni@outscale.com>
Date: Wed, 12 Oct 2022 15:35:16 +0000
Subject: [PATCH] add logs for mostRecent test

Signed-off-by: hanen mizouni <hanen.mizouni@outscale.com>
---
 builder/osc/common/step_source_omi_info_test.go | 12 +++++++-----
 example/most_recent_test.pkr.hcl                |  2 +-
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/builder/osc/common/step_source_omi_info_test.go b/builder/osc/common/step_source_omi_info_test.go
index 1445399..084af78 100644
--- a/builder/osc/common/step_source_omi_info_test.go
+++ b/builder/osc/common/step_source_omi_info_test.go
@@ -1,6 +1,7 @@
 package common
 
 import (
+	"fmt"
 	"testing"
 
 	"github.com/hashicorp/packer-plugin-sdk/multistep"
@@ -13,14 +14,15 @@ import (
 )
 
 // Create statebag for running test
-func getState() multistep.StateBag {
+func getState() (multistep.StateBag, error) {
 	state := new(multistep.BasicStateBag)
 	accessConfig := &AccessConfig{}
 	accessConfig.RawRegion = "eu-west-2"
 	var oscConn *osc.APIClient
 	var err error
 	if oscConn, err = accessConfig.NewOSCClient(); err != nil {
-		return nil
+		err := fmt.Errorf("error in creating osc Client: %s", err.Error())
+		return nil, err
 	}
 	state.Put("osc", oscConn)
 	state.Put("ui", &packersdk.BasicUi{
@@ -28,7 +30,7 @@ func getState() multistep.StateBag {
 		Writer: new(bytes.Buffer),
 	})
 	state.Put("accessConfig", accessConfig)
-	return state
+	return state, err
 }
 
 func TestMostRecentOmiFilter(t *testing.T) {
@@ -38,9 +40,9 @@ func TestMostRecentOmiFilter(t *testing.T) {
 			MostRecent: true,
 		},
 	}
-	state := getState()
+	state, err := getState()
 	if state == nil {
-		t.Fatalf("error retrieving state, but")
+		t.Fatalf("error retrieving state %s", err.Error())
 	}
 
 	action := stepSourceOMIInfo.Run(context.Background(), state)
diff --git a/example/most_recent_test.pkr.hcl b/example/most_recent_test.pkr.hcl
index 6353b9c..316a397 100644
--- a/example/most_recent_test.pkr.hcl
+++ b/example/most_recent_test.pkr.hcl
@@ -57,4 +57,4 @@ source "outscale-bsusurrogate" "test" {
 
 build {
      sources = [ "source.outscale-bsusurrogate.test" ]
-}
\ No newline at end of file
+}