chore: fix golangci-lint warning (#3326)

Author: remyleone

.golangci.yml

@@ -88,7 +88,7 @@ linters:
     - usetesting # Reports uses of functions with replacement inside the testing package. [auto-fix]
     - wastedassign # wastedassign finds wasted assignment statements. [fast: false, auto-fix: false]
     - whitespace # Tool for detection of leading and trailing whitespace [fast: true, auto-fix: true]
-    - wsl # Whitespace Linter - Forces you to use empty lines! [fast: true, auto-fix: false]
+    - wsl_v5 # Whitespace Linter - Forces you to use empty lines! [fast: true, auto-fix: false]
     - zerologlint # Detects the wrong usage of `zerolog` that a user forgets to dispatch with `Send` or `Msg` [fast: false, auto-fix: false]
 
   disable:
@@ -141,6 +141,11 @@ linters:
       reject:
         - anon
 
+    wsl_v5:
+      allow-first-in-block: true
+      allow-whole-block: false
+      branch-max-lines: 2
+
   exclusions:
     rules:
       - path: _test\.go

internal/services/account/project_test.go

@@ -20,6 +20,7 @@ var DestroyWaitTimeout = 3 * time.Minute
 func TestAccProject_Basic(t *testing.T) {
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		ProviderFactories: tt.ProviderFactories,
 		CheckDestroy:      isProjectDestroyed(tt),
@@ -57,6 +58,7 @@ func TestAccProject_Basic(t *testing.T) {
 func TestAccProject_NoUpdate(t *testing.T) {
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		ProviderFactories: tt.ProviderFactories,
 		CheckDestroy:      isProjectDestroyed(tt),

internal/services/applesilicon/server_test.go

@@ -15,6 +15,7 @@ import (
 func TestAccServer_Basic(t *testing.T) {
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
 		ProviderFactories: tt.ProviderFactories,
@@ -67,6 +68,7 @@ func TestAccServer_Basic(t *testing.T) {
 func TestAccServer_EnableDisabledVPC(t *testing.T) {
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
 		ProviderFactories: tt.ProviderFactories,
@@ -121,6 +123,7 @@ func TestAccServer_EnableDisabledVPC(t *testing.T) {
 func TestAccServer_EnableVPC(t *testing.T) {
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
 		ProviderFactories: tt.ProviderFactories,
@@ -265,6 +268,7 @@ func TestAccServer_Commitment(t *testing.T) {
 
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
 		ProviderFactories: tt.ProviderFactories,

internal/services/autoscaling/instance_group_test.go

@@ -154,7 +154,6 @@ func testAccCheckInstanceGroupDestroy(tt *acctest.TestTools) resource.TestCheckF
 				InstanceGroupID: id,
 				Zone:            zone,
 			})
-
 			if err == nil {
 				return fmt.Errorf("autoscaling instance group (%s) still exists", rs.Primary.ID)
 			}

internal/services/autoscaling/instance_policy_test.go

@@ -170,7 +170,6 @@ func testAccCheckInstancePolicyDestroy(tt *acctest.TestTools) resource.TestCheck
 				PolicyID: id,
 				Zone:     zone,
 			})
-
 			if err == nil {
 				return fmt.Errorf("autoscaling instance policy (%s) still exists", rs.Primary.ID)
 			}

internal/services/autoscaling/instance_template_test.go

@@ -166,7 +166,6 @@ func testAccCheckInstanceTemplateDestroy(tt *acctest.TestTools) resource.TestChe
 				TemplateID: id,
 				Zone:       zone,
 			})
-
 			if err == nil {
 				return fmt.Errorf("autoscaling instance template (%s) still exists", rs.Primary.ID)
 			}

internal/services/az/availability_zones_data_source_test.go

@@ -10,6 +10,7 @@ import (
 func TestAccDataSourceAvailabilityZones_Basic(t *testing.T) {
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
 		ProviderFactories: tt.ProviderFactories,

internal/services/baremetal/server.go

@@ -606,6 +606,7 @@ func ResourceServerUpdate(ctx context.Context, d *schema.ResourceData, m any) di
 	}
 
 	var serverGetOptionIDs []*baremetal.ServerOption
+
 	serverGetOptionIDs = append(serverGetOptionIDs, server.Options...)
 
 	if d.HasChange("offer") {

internal/services/block/snapshot_data_source_test.go

@@ -11,6 +11,7 @@ import (
 func TestAccDataSourceSnapshot_Basic(t *testing.T) {
 	tt := acctest.NewTestTools(t)
 	defer tt.Cleanup()
+
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
 		ProviderFactories: tt.ProviderFactories,

internal/services/block/testfuncs/checks.go

@@ -75,7 +75,6 @@ func IsVolumeDestroyed(tt *acctest.TestTools) resource.TestCheckFunc {
 				VolumeID: id,
 				Zone:     zone,
 			})
-
 			if err == nil {
 				return fmt.Errorf("block volume (%s) still exists", rs.Primary.ID)
 			}
@@ -105,7 +104,6 @@ func IsSnapshotDestroyed(tt *acctest.TestTools) resource.TestCheckFunc {
 				SnapshotID: id,
 				Zone:       zone,
 			})
-
 			if err == nil {
 				return fmt.Errorf("block snapshot (%s) still exists", rs.Primary.ID)
 			}