run go fmt on unit test commit (#79)

Generating code on the main branch has a bunch of linting changes. Missed in unit test PR since there was no code generation and `go fmt` runs in build-controller script after code is generated

Testing

```
make test
```

Author: surajkota

apis/v1alpha1/ack-generate-metadata.yaml

@@ -1,14 +1,14 @@
 ack_generate_info:
-  build_date: "2021-07-30T19:09:55Z"
+  build_date: "2021-08-12T00:32:37Z"
   build_hash: c77aa9c75d944952dee198029ba9822691cd82b0
-  go_version: go1.16.4 darwin/amd64
+  go_version: go1.16.4 linux/amd64
   version: v0.6.0
 api_directory_checksum: c8f263a8d1a2e21f53c85b1ca5e2be3687045dd0
 api_version: v1alpha1
-aws_sdk_go_version: ""
+aws_sdk_go_version: v1.38.11
 generator_config_info:
   file_checksum: 795f6cf17e64a254ed063ee36e489b2949a031ef
   original_file_name: generator.yaml
 last_modification:
   reason: API generation
-  timestamp: 2021-07-30 19:10:09.128773 +0000 UTC
+  timestamp: 2021-08-12 00:32:40.304705695 +0000 UTC

pkg/resource/data_quality_job_definition/manager_test_suite_test.go

@@ -16,20 +16,20 @@ package data_quality_job_definition
 import (
 	"errors"
 	"fmt"
-	
-	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
+
 	ackv1alpha1 "github.com/aws-controllers-k8s/runtime/apis/core/v1alpha1"
-	"github.com/ghodss/yaml"
-	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
 	acktypes "github.com/aws-controllers-k8s/runtime/pkg/types"
+	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	svcsdk "github.com/aws/aws-sdk-go/service/sagemaker"
+	"github.com/ghodss/yaml"
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
+	"go.uber.org/zap/zapcore"
 	"path/filepath"
-	"testing"
 	ctrlrtzap "sigs.k8s.io/controller-runtime/pkg/log/zap"
-	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
-	"go.uber.org/zap/zapcore"
+	"testing"
 )
 
 // provideResourceManagerWithMockSDKAPI accepts MockSageMakerAPI and returns pointer to resourceManager
@@ -53,12 +53,12 @@ func provideResourceManagerWithMockSDKAPI(mockSageMakerAPI *mocksvcsdkapi.SageMa
 
 // TestDataQualityJobDefinitionTestSuite runs the test suite for data quality job definition
 func TestDataQualityJobDefinitionTestSuite(t *testing.T) {
-     	defer func() {
-     	   if r := recover(); r != nil {
-     	      fmt.Println(testutil.RecoverPanicString, r)
-	      t.Fail()
-     	   }
-        }()
+	defer func() {
+		if r := recover(); r != nil {
+			fmt.Println(testutil.RecoverPanicString, r)
+			t.Fail()
+		}
+	}()
 	var ts = testutil.TestSuite{}
 	testutil.LoadFromFixture(filepath.Join("testdata", "test_suite.yaml"), &ts)
 	var delegate = testRunnerDelegate{t: t}
@@ -91,13 +91,13 @@ func (d *testRunnerDelegate) EmptyServiceAPIOutput(apiName string) (interface{},
 	switch apiName {
 	case "CreateDataQualityJobDefinitionWithContext":
 		var output svcsdk.CreateDataQualityJobDefinitionOutput
-	    	return &output, nil
+		return &output, nil
 	case "DescribeDataQualityJobDefinitionWithContext":
 		var output svcsdk.DescribeDataQualityJobDefinitionOutput
-	    	return &output, nil
+		return &output, nil
 	case "DeleteDataQualityJobDefinitionWithContext":
 		var output svcsdk.DeleteDataQualityJobDefinitionOutput
-	    	return &output, nil
+		return &output, nil
 	}
 	return nil, errors.New(fmt.Sprintf("no matching API name found for: %s", apiName))
 }

pkg/resource/endpoint/manager_test_suite_test.go

@@ -16,20 +16,20 @@ package endpoint
 import (
 	"errors"
 	"fmt"
-	
-	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
+
 	ackv1alpha1 "github.com/aws-controllers-k8s/runtime/apis/core/v1alpha1"
-	"github.com/ghodss/yaml"
-	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
 	acktypes "github.com/aws-controllers-k8s/runtime/pkg/types"
+	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	svcsdk "github.com/aws/aws-sdk-go/service/sagemaker"
+	"github.com/ghodss/yaml"
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
+	"go.uber.org/zap/zapcore"
 	"path/filepath"
-	"testing"
 	ctrlrtzap "sigs.k8s.io/controller-runtime/pkg/log/zap"
-	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
-	"go.uber.org/zap/zapcore"
+	"testing"
 )
 
 // provideResourceManagerWithMockSDKAPI accepts MockSageMakerAPI and returns pointer to resourceManager
@@ -53,12 +53,12 @@ func provideResourceManagerWithMockSDKAPI(mockSageMakerAPI *mocksvcsdkapi.SageMa
 
 // TestEndpointTestSuite runs the test suite for endpoint
 func TestEndpointTestSuite(t *testing.T) {
-     	defer func() {
-     	   if r := recover(); r != nil {
-     	      fmt.Println(testutil.RecoverPanicString, r)
-	      t.Fail()
-     	   }
-        }()
+	defer func() {
+		if r := recover(); r != nil {
+			fmt.Println(testutil.RecoverPanicString, r)
+			t.Fail()
+		}
+	}()
 	var ts = testutil.TestSuite{}
 	testutil.LoadFromFixture(filepath.Join("testdata", "test_suite.yaml"), &ts)
 	var delegate = testRunnerDelegate{t: t}
@@ -91,7 +91,7 @@ func (d *testRunnerDelegate) EmptyServiceAPIOutput(apiName string) (interface{},
 	switch apiName {
 	case "CreateEndpointWithContext":
 		var output svcsdk.CreateEndpointOutput
-	    	return &output, nil
+		return &output, nil
 	case "DeleteEndpointWithContext":
 		var output svcsdk.DeleteEndpointOutput
 		return &output, nil

pkg/resource/endpoint_config/manager_test_suite_test.go

@@ -17,38 +17,38 @@ import (
 	"errors"
 	"fmt"
 
-	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	ackv1alpha1 "github.com/aws-controllers-k8s/runtime/apis/core/v1alpha1"
-	"github.com/ghodss/yaml"
-	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
 	acktypes "github.com/aws-controllers-k8s/runtime/pkg/types"
+	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	svcsdk "github.com/aws/aws-sdk-go/service/sagemaker"
+	"github.com/ghodss/yaml"
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
+	"go.uber.org/zap/zapcore"
 	"path/filepath"
-	"testing"
 	ctrlrtzap "sigs.k8s.io/controller-runtime/pkg/log/zap"
-	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
-	"go.uber.org/zap/zapcore"
+	"testing"
 )
 
 // provideResourceManagerWithMockSDKAPI accepts MockSageMakerAPI and returns pointer to resourceManager
 // the returned resourceManager is configured to use mockapi api.
 func provideResourceManagerWithMockSDKAPI(mockSageMakerAPI *mocksvcsdkapi.SageMakerAPI) *resourceManager {
-     zapOptions := ctrlrtzap.Options{
-     	    Development: true,
-	    Level:       zapcore.InfoLevel,
-     }
-     fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
-     return &resourceManager{
-     	    rr:           nil,
-	    awsAccountID: "",
-	    awsRegion:    "",
-	    sess:         nil,
-	    sdkapi:       mockSageMakerAPI,
-	    log:          fakeLogger,
-	    metrics:      ackmetrics.NewMetrics("sagemaker"),
-     }
+	zapOptions := ctrlrtzap.Options{
+		Development: true,
+		Level:       zapcore.InfoLevel,
+	}
+	fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
+	return &resourceManager{
+		rr:           nil,
+		awsAccountID: "",
+		awsRegion:    "",
+		sess:         nil,
+		sdkapi:       mockSageMakerAPI,
+		log:          fakeLogger,
+		metrics:      ackmetrics.NewMetrics("sagemaker"),
+	}
 }
 
 // TestEndpointConfigTestSuite runs the test suite for endpoint config.

pkg/resource/feature_group/manager_test_suite_test.go

@@ -18,47 +18,47 @@ import (
 	"fmt"
 
 	ackv1alpha1 "github.com/aws-controllers-k8s/runtime/apis/core/v1alpha1"
-	"github.com/ghodss/yaml"
-	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
-	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
 	acktypes "github.com/aws-controllers-k8s/runtime/pkg/types"
+	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	svcsdk "github.com/aws/aws-sdk-go/service/sagemaker"
+	"github.com/ghodss/yaml"
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
+	"go.uber.org/zap/zapcore"
 	"path/filepath"
-	"testing"
 	ctrlrtzap "sigs.k8s.io/controller-runtime/pkg/log/zap"
-	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
-	"go.uber.org/zap/zapcore"
+	"testing"
 )
 
 // provideResourceManagerWithMockSDKAPI accepts MockSageMakerAPI and returns pointer to resourceManager
 // the returned resourceManager is configured to use mockapi api.
 func provideResourceManagerWithMockSDKAPI(mockSageMakerAPI *mocksvcsdkapi.SageMakerAPI) *resourceManager {
-     zapOptions := ctrlrtzap.Options{
-     	    Development: true,
-	    Level:       zapcore.InfoLevel,
-     }
-     fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
-     return &resourceManager{
-     	    rr:           nil,
-	    awsAccountID: "",
-	    awsRegion:    "",
-	    sess:         nil,
-	    sdkapi:       mockSageMakerAPI,
-	    log:          fakeLogger,
-	    metrics:      ackmetrics.NewMetrics("sagemaker"),
-     }
+	zapOptions := ctrlrtzap.Options{
+		Development: true,
+		Level:       zapcore.InfoLevel,
+	}
+	fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
+	return &resourceManager{
+		rr:           nil,
+		awsAccountID: "",
+		awsRegion:    "",
+		sess:         nil,
+		sdkapi:       mockSageMakerAPI,
+		log:          fakeLogger,
+		metrics:      ackmetrics.NewMetrics("sagemaker"),
+	}
 }
 
 // TestFeatureGroupTestSuite runs the test suite for feature group
 func TestFeatureGroupTestSuite(t *testing.T) {
-     	defer func() {
-     	   if r := recover(); r != nil {
-     	      fmt.Println(testutil.RecoverPanicString, r)
-	      t.Fail()
-     	   }
-        }()
+	defer func() {
+		if r := recover(); r != nil {
+			fmt.Println(testutil.RecoverPanicString, r)
+			t.Fail()
+		}
+	}()
 	var ts = testutil.TestSuite{}
 	testutil.LoadFromFixture(filepath.Join("testdata", "test_suite.yaml"), &ts)
 	var delegate = testRunnerDelegate{t: t}

pkg/resource/hyper_parameter_tuning_job/manager_test_suite_test.go

@@ -17,38 +17,38 @@ import (
 	"errors"
 	"fmt"
 
-	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	ackv1alpha1 "github.com/aws-controllers-k8s/runtime/apis/core/v1alpha1"
-	"github.com/ghodss/yaml"
-	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
 	acktypes "github.com/aws-controllers-k8s/runtime/pkg/types"
+	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	svcsdk "github.com/aws/aws-sdk-go/service/sagemaker"
+	"github.com/ghodss/yaml"
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
+	"go.uber.org/zap/zapcore"
 	"path/filepath"
-	"testing"
 	ctrlrtzap "sigs.k8s.io/controller-runtime/pkg/log/zap"
-	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
-	"go.uber.org/zap/zapcore"
+	"testing"
 )
 
 // provideResourceManagerWithMockSDKAPI accepts MockSageMakerAPI and returns pointer to resourceManager
 // the returned resourceManager is configured to use mockapi api.
 func provideResourceManagerWithMockSDKAPI(mockSageMakerAPI *mocksvcsdkapi.SageMakerAPI) *resourceManager {
-     zapOptions := ctrlrtzap.Options{
-     	    Development: true,
-	    Level:       zapcore.InfoLevel,
-     }
-     fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
-     return &resourceManager{
-     	    rr:           nil,
-	    awsAccountID: "",
-	    awsRegion:    "",
-	    sess:         nil,
-	    sdkapi:       mockSageMakerAPI,
-	    log:          fakeLogger,
-	    metrics:      ackmetrics.NewMetrics("sagemaker"),
-     }
+	zapOptions := ctrlrtzap.Options{
+		Development: true,
+		Level:       zapcore.InfoLevel,
+	}
+	fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
+	return &resourceManager{
+		rr:           nil,
+		awsAccountID: "",
+		awsRegion:    "",
+		sess:         nil,
+		sdkapi:       mockSageMakerAPI,
+		log:          fakeLogger,
+		metrics:      ackmetrics.NewMetrics("sagemaker"),
+	}
 }
 
 // TestHyperParameterTuningJobTestSuite runs the test suite for hyper parameter tuning job.

pkg/resource/model/manager_test_suite_test.go

@@ -17,38 +17,38 @@ import (
 	"errors"
 	"fmt"
 
-	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	ackv1alpha1 "github.com/aws-controllers-k8s/runtime/apis/core/v1alpha1"
-	"github.com/ghodss/yaml"
-	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
 	acktypes "github.com/aws-controllers-k8s/runtime/pkg/types"
+	"github.com/aws-controllers-k8s/sagemaker-controller/pkg/testutil"
+	mocksvcsdkapi "github.com/aws-controllers-k8s/sagemaker-controller/test/mocks/aws-sdk-go/sagemaker"
 	svcsdk "github.com/aws/aws-sdk-go/service/sagemaker"
+	"github.com/ghodss/yaml"
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
+	"go.uber.org/zap/zapcore"
 	"path/filepath"
-	"testing"
 	ctrlrtzap "sigs.k8s.io/controller-runtime/pkg/log/zap"
-	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
-	"go.uber.org/zap/zapcore"
+	"testing"
 )
 
 // provideResourceManagerWithMockSDKAPI accepts MockSageMakerAPI and returns pointer to resourceManager
 // the returned resourceManager is configured to use mockapi api.
 func provideResourceManagerWithMockSDKAPI(mockSageMakerAPI *mocksvcsdkapi.SageMakerAPI) *resourceManager {
-     zapOptions := ctrlrtzap.Options{
-     	    Development: true,
-	    Level:       zapcore.InfoLevel,
-     }
-     fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
-     return &resourceManager{
-     	    rr:           nil,
-	    awsAccountID: "",
-	    awsRegion:    "",
-	    sess:         nil,
-	    sdkapi:       mockSageMakerAPI,
-	    log:          fakeLogger,
-	    metrics:      ackmetrics.NewMetrics("sagemaker"),
-     }
+	zapOptions := ctrlrtzap.Options{
+		Development: true,
+		Level:       zapcore.InfoLevel,
+	}
+	fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
+	return &resourceManager{
+		rr:           nil,
+		awsAccountID: "",
+		awsRegion:    "",
+		sess:         nil,
+		sdkapi:       mockSageMakerAPI,
+		log:          fakeLogger,
+		metrics:      ackmetrics.NewMetrics("sagemaker"),
+	}
 }
 
 // TestModelTestSuite runs the test suite for model