chore: fix lint warnings

goto · Aug 15, 2023 · 4f30781 · 4f30781
1 parent 34755c0
commit 4f30781
Show file tree

Hide file tree

Showing 9 changed files with 9 additions and 10 deletions.
diff --git a/api/handler/v1beta1/appeal.go b/api/handler/v1beta1/appeal.go
@@ -3,6 +3,7 @@ package v1beta1
 import (
 	"context"
 	"errors"
+
 	guardianv1beta1 "github.com/goto/guardian/api/proto/gotocompany/guardian/v1beta1"
 	"github.com/goto/guardian/core/appeal"
 	"github.com/goto/guardian/domain"

diff --git a/api/handler/v1beta1/appeal_test.go b/api/handler/v1beta1/appeal_test.go
@@ -3,9 +3,10 @@ package v1beta1_test
 import (
 	"context"
 	"errors"
-	"github.com/google/uuid"
 	"time"
 
+	"github.com/google/uuid"
+
 	guardianv1beta1 "github.com/goto/guardian/api/proto/gotocompany/guardian/v1beta1"
 	"github.com/goto/guardian/core/appeal"
 	"github.com/goto/guardian/domain"
@@ -702,7 +703,6 @@ func (s *GrpcHandlersSuite) TestGetAppeal() {
 		s.Nil(res)
 		s.appealService.AssertExpectations(s.T())
 	})
-
 }
 
 func (s *GrpcHandlersSuite) TestCancelAppeal() {

diff --git a/core/appeal/service_test.go b/core/appeal/service_test.go
@@ -2395,7 +2395,6 @@ func (s *ServiceTestSuite) TestCancel() {
 		s.Nil(actualResult)
 		s.EqualError(actualErr, expectedErr.Error())
 	})
-
 }
 
 func (s *ServiceTestSuite) TestAddApprover() {

diff --git a/internal/store/postgres/approval_repository_test.go b/internal/store/postgres/approval_repository_test.go
@@ -118,7 +118,6 @@ func (s *ApprovalRepositoryTestSuite) TearDownSuite() {
 }
 
 func (s *ApprovalRepositoryTestSuite) TestListApprovals() {
-
 	pendingAppeal := &domain.Appeal{
 		ResourceID:    s.dummyResource.ID,
 		PolicyID:      s.dummyPolicy.ID,

diff --git a/pkg/evaluator/expression_test.go b/pkg/evaluator/expression_test.go
@@ -1,9 +1,10 @@
 package evaluator_test
 
 import (
-	"github.com/stretchr/testify/assert"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
+
 	"github.com/goto/guardian/pkg/evaluator"
 )
 

diff --git a/plugins/notifiers/slack/client_test.go b/plugins/notifiers/slack/client_test.go
@@ -70,7 +70,6 @@ func (s *ClientTestSuite) TestNotify() {
 
 		s.Equal(expectedErrs, actualErrs)
 	})
-
 }
 
 func (s *ClientTestSuite) TestParseMessage() {

diff --git a/plugins/providers/bigquery/model.go b/plugins/providers/bigquery/model.go
@@ -16,8 +16,6 @@ const (
 	ResourceTypeDataset = "dataset"
 	// ResourceTypeTable is the resource type name for BigQuery table
 	ResourceTypeTable = "table"
-
-	resourceTypeUnknown = "unknown"
 )
 
 // Dataset is a reference to a BigQuery dataset

diff --git a/plugins/providers/bigquery/model_test.go b/plugins/providers/bigquery/model_test.go
@@ -1,9 +1,10 @@
 package bigquery_test
 
 import (
-	"github.com/goto/guardian/core/resource"
 	"testing"
 
+	"github.com/goto/guardian/core/resource"
+
 	"github.com/goto/guardian/domain"
 	"github.com/goto/guardian/plugins/providers/bigquery"
 	"github.com/stretchr/testify/assert"

diff --git a/utils/utils_test.go b/utils/utils_test.go
@@ -1,8 +1,9 @@
 package utils
 
 import (
-	"github.com/stretchr/testify/assert"
 	"testing"
+
+	"github.com/stretchr/testify/assert"
 )
 
 func TestMapToSlice(t *testing.T) {