fix: issue from golangci-lint about formatted, maps.Copy and embedded field

Signed-off-by: Shouren Yang <yangshouren@gmail.com>

Author: Shouren

pkg/device/devices.go

@@ -20,6 +20,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
+	"maps"
 	"slices"
 	"strconv"
 	"strings"
@@ -215,9 +216,7 @@ func (d *DeviceUsage) DeepCopy() *DeviceUsage {
 
 	if d.CustomInfo != nil {
 		dup.CustomInfo = make(map[string]any, len(d.CustomInfo))
-		for k, v := range d.CustomInfo {
-			dup.CustomInfo[k] = v
-		}
+		maps.Copy(dup.CustomInfo, d.CustomInfo)
 	}
 
 	return dup

pkg/device/devices_test.go

@@ -1191,7 +1191,6 @@ func TestCheckUUID(t *testing.T) {
 	}
 }
 
-
 func TestDeviceUsageDeepCopy(t *testing.T) {
 	tests := []struct {
 		name     string
@@ -1228,9 +1227,9 @@ func TestDeviceUsageDeepCopy(t *testing.T) {
 						{Name: "1g.5gb", Core: 1, Memory: 5, InUse: false},
 					},
 				},
-				Numa:     0,
-				Type:     "NVIDIA",
-				Health:   true,
+				Numa:   0,
+				Type:   "NVIDIA",
+				Health: true,
 				PodInfos: []*PodInfo{
 					{
 						Pod: &corev1.Pod{

pkg/device/pod_test.go

@@ -377,7 +377,6 @@ func TestUpdatePod(t *testing.T) {
 	}
 }
 
-
 func TestPodInfoDeepCopy(t *testing.T) {
 	tests := []struct {
 		name     string
@@ -430,14 +429,14 @@ func TestPodInfoDeepCopy(t *testing.T) {
 			assert.Equal(t, tt.original.CtrIDs, copy.CtrIDs)
 			assert.Equal(t, tt.original.Devices, copy.Devices)
 			if tt.original.Pod != nil {
-				assert.Equal(t, tt.original.Pod.Name, copy.Pod.Name)
+				assert.Equal(t, tt.original.Name, copy.Name)
 			}
 
 			// 2. Mutating the copy must not affect the original.
 			if copy.Pod != nil {
-				originalPodName := tt.original.Pod.Name
+				originalPodName := tt.original.Name
 				copy.Pod.Name = "mutated-pod"
-				assert.Equal(t, tt.original.Pod.Name, originalPodName)
+				assert.Equal(t, tt.original.Name, originalPodName)
 			}
 			originalNodeID := tt.original.NodeID
 			copy.NodeID = "mutated-node"

pkg/device/pods.go

@@ -211,9 +211,7 @@ func (c ContainerDevice) DeepCopy() ContainerDevice {
 	}
 	if c.CustomInfo != nil {
 		dup.CustomInfo = make(map[string]any, len(c.CustomInfo))
-		for k, v := range c.CustomInfo {
-			dup.CustomInfo[k] = v
-		}
+		maps.Copy(dup.CustomInfo, c.CustomInfo)
 	}
 	return dup
 }