Enforce code format for tests (#660)

* Enforce code format including tests

* Format existing tests

* gofumpt

* gofumpt

Author: zliang-akamai

.github/workflows/ci.yml

@@ -24,6 +24,11 @@ jobs:
       - name: Run tidy
         run: make tidy
 
+      - name: Run formatter
+        run: |
+          go install mvdan.cc/gofumpt@86bffd62437a3c437c0b84d5d5ab244824e762fc
+          gofumpt -l -w .
+
       - name: Fail if changes
         run: git diff-index --exit-code HEAD
 

test/integration/lke_clusters_test.go

@@ -343,7 +343,8 @@ func TestLKECluster_APLEnabled_smoke(t *testing.T) {
 		func(createOpts *linodego.LKEClusterCreateOptions) {
 			// NOTE: g6-dedicated-4 is the minimum APL-compatible Linode type
 			createOpts.NodePools = []linodego.LKENodePoolCreateOptions{{Count: 3, Type: "g6-dedicated-4", Tags: []string{"test"}}}
-		}},
+		},
+	},
 		"fixtures/TestLKECluster_APLEnabled")
 	defer teardown()
 

test/unit/domain_test.go

@@ -3,9 +3,10 @@ package unit
 import (
 	"context"
 	"fmt"
-	"github.com/jarcoal/httpmock"
 	"testing"
 
+	"github.com/jarcoal/httpmock"
+
 	"github.com/linode/linodego"
 	"github.com/stretchr/testify/assert"
 )

test/unit/domainrecord_test.go

@@ -3,11 +3,12 @@ package unit
 import (
 	"context"
 	"fmt"
+	"testing"
+	"time"
+
 	"github.com/jarcoal/httpmock"
 	"github.com/linode/linodego"
 	"github.com/stretchr/testify/assert"
-	"testing"
-	"time"
 )
 
 func TestDomainRecord_List(t *testing.T) {

test/unit/longview_subscriptions_test.go

@@ -4,8 +4,8 @@ import (
 	"context"
 	"testing"
 
-	"github.com/stretchr/testify/assert"
 	"github.com/linode/linodego"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestListLongviewSubscriptions(t *testing.T) {

test/unit/longview_test.go

@@ -5,8 +5,8 @@ import (
 	"testing"
 	"time"
 
-	"github.com/stretchr/testify/assert"
 	"github.com/linode/linodego"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestListLongviewClients(t *testing.T) {

test/unit/mysql_test.go

@@ -2,9 +2,10 @@ package unit
 
 import (
 	"context"
-	"github.com/jarcoal/httpmock"
 	"testing"
 
+	"github.com/jarcoal/httpmock"
+
 	"github.com/linode/linodego"
 	"github.com/stretchr/testify/assert"
 )

test/unit/postgres_test.go

@@ -2,10 +2,11 @@ package unit
 
 import (
 	"context"
+	"testing"
+
 	"github.com/jarcoal/httpmock"
 	"github.com/linode/linodego"
 	"github.com/stretchr/testify/assert"
-	"testing"
 )
 
 func TestListDatabasePostgreSQL_List(t *testing.T) {

test/unit/stackscripts_test.go

@@ -5,34 +5,34 @@ import (
 	"fmt"
 	"testing"
 
-	"github.com/stretchr/testify/assert"
 	"github.com/linode/linodego"
+	"github.com/stretchr/testify/assert"
 	"golang.org/x/exp/slices"
 )
 
 func TestListStackscripts(t *testing.T) {
-    // Mock the API response to match the expected structure for a paginated response
-    fixtureData, err := fixtures.GetFixture("stackscripts_list")
-    assert.NoError(t, err)
+	// Mock the API response to match the expected structure for a paginated response
+	fixtureData, err := fixtures.GetFixture("stackscripts_list")
+	assert.NoError(t, err)
 
-    var base ClientBaseCase
-    base.SetUp(t)
-    defer base.TearDown(t)
+	var base ClientBaseCase
+	base.SetUp(t)
+	defer base.TearDown(t)
 
-    // Mock the request with a correct paginated structure
-    base.MockGet("linode/stackscripts", fixtureData)
+	// Mock the request with a correct paginated structure
+	base.MockGet("linode/stackscripts", fixtureData)
 
-    stackscripts, err := base.Client.ListStackscripts(context.Background(), &linodego.ListOptions{})
-    assert.NoError(t, err)
+	stackscripts, err := base.Client.ListStackscripts(context.Background(), &linodego.ListOptions{})
+	assert.NoError(t, err)
 
-    assert.NotEmpty(t, stackscripts, "Expected non-empty stackscripts list")
+	assert.NotEmpty(t, stackscripts, "Expected non-empty stackscripts list")
 
-    // Check if a specific stackscript exists using slices.ContainsFunc
-    exists := slices.ContainsFunc(stackscripts, func(stackscript linodego.Stackscript) bool {
-        return stackscript.Label == "Test Stackscript"
-    })
+	// Check if a specific stackscript exists using slices.ContainsFunc
+	exists := slices.ContainsFunc(stackscripts, func(stackscript linodego.Stackscript) bool {
+		return stackscript.Label == "Test Stackscript"
+	})
 
-    assert.True(t, exists, "Expected stackscripts list to contain 'Test Stackscript'")
+	assert.True(t, exists, "Expected stackscripts list to contain 'Test Stackscript'")
 }
 
 func TestCreateStackscript(t *testing.T) {

test/unit/tag_test.go

@@ -34,7 +34,6 @@ func TestListTags(t *testing.T) {
 	assert.True(t, exists, "Expected tag list to contain 'example-tag'")
 }
 
-
 func TestCreateTag(t *testing.T) {
 	// Load the fixture data for tag creation
 	fixtureData, err := fixtures.GetFixture("tag_create")
@@ -57,7 +56,6 @@ func TestCreateTag(t *testing.T) {
 	assert.Equal(t, "new-tag", tag.Label, "Expected created tag label to match input")
 }
 
-
 func TestDeleteTag(t *testing.T) {
 	var base ClientBaseCase
 	base.SetUp(t)
@@ -98,7 +96,6 @@ func TestListTaggedObjects(t *testing.T) {
 	}
 }
 
-
 func TestSortedObjects(t *testing.T) {
 	// Load the fixture data for tagged objects
 	fixtureData, err := fixtures.GetFixture("tagged_objects_list")