increase code coverage

Author: Rahul Sharma

api/v1alpha2/linodevpc_types.go

@@ -103,7 +103,7 @@ type VPCSubnetCreateOptions struct {
 }
 
 // VPCSubnetCreateOptionsIPv6 defines the options for creating an IPv6 range in a VPC subnet.
-// Its copied from linodego.VPCSubnetCreateOptionsIPv6 and should be kept in sync.
+// It's copied from linodego.VPCSubnetCreateOptionsIPv6 and should be kept in sync.
 // Values supported by the linode API should be used here.
 // See https://techdocs.akamai.com/linode-api/reference/post-vpc-subnet for more details.
 type VPCSubnetCreateOptionsIPv6 struct {

config/crd/bases/infrastructure.cluster.x-k8s.io_linodevpcs.yaml

@@ -145,7 +145,7 @@ spec:
                       items:
                         description: |-
                           VPCSubnetCreateOptionsIPv6 defines the options for creating an IPv6 range in a VPC subnet.
-                          Its copied from linodego.VPCSubnetCreateOptionsIPv6 and should be kept in sync.
+                          It's copied from linodego.VPCSubnetCreateOptionsIPv6 and should be kept in sync.
                           Values supported by the linode API should be used here.
                           See https://techdocs.akamai.com/linode-api/reference/post-vpc-subnet for more details.
                         properties:

docs/src/reference/out.md

@@ -1270,7 +1270,7 @@ _Appears in:_
 
 
 VPCSubnetCreateOptionsIPv6 defines the options for creating an IPv6 range in a VPC subnet.
-Its copied from linodego.VPCSubnetCreateOptionsIPv6 and should be kept in sync.
+It's copied from linodego.VPCSubnetCreateOptionsIPv6 and should be kept in sync.
 Values supported by the linode API should be used here.
 See https://techdocs.akamai.com/linode-api/reference/post-vpc-subnet for more details.
 

internal/controller/linodemachine_controller_helpers.go

@@ -589,7 +589,7 @@ func getVPCInterfaceConfigFromDirectID(ctx context.Context, machineScope *scope.
 }
 
 // machineIPv6RangeConfig returns the IPv6 range configuration if subnet has IPv6 ranges.
-// for now, we support only a single IPv6 range for machine per subnet.
+// For now, we support only a single IPv6 range for machine per subnet.
 // If this changes, we may need to adjust this logic.
 func machineIPv6RangeConfig(numIPv6RangesInSubnet int) []linodego.InstanceConfigInterfaceCreateOptionsIPv6Range {
 	if numIPv6RangesInSubnet == 0 {

internal/controller/linodemachine_controller_helpers_test.go

@@ -409,6 +409,9 @@ func validateInterfaceExpectations(
 	if expectInterface {
 		require.NotNil(t, iface)
 		require.Equal(t, linodego.InterfacePurposeVPC, iface.Purpose)
+		if iface.IPv6 != nil {
+			require.Equal(t, defaultNodeIPv6CIDRRange, *iface.IPv6.Ranges[0].Range)
+		}
 		require.True(t, iface.Primary)
 		require.NotNil(t, iface.SubnetID)
 		require.Equal(t, expectSubnetID, *iface.SubnetID)
@@ -497,6 +500,11 @@ func TestGetVPCInterfaceConfigFromDirectID(t *testing.T) {
 						{
 							ID:    456,
 							Label: "subnet-1",
+							IPv6: []linodego.VPCIPv6Range{
+								{
+									Range: "2001:0db8::/56",
+								},
+							},
 						},
 					},
 				}, nil)
@@ -762,6 +770,11 @@ func TestConfigureVPCInterface(t *testing.T) {
 						{
 							ID:    456,
 							Label: "subnet-1",
+							IPv6: []linodego.VPCIPv6Range{
+								{
+									Range: "2001:0db8::/56",
+								},
+							},
 						},
 					},
 				}, nil)
@@ -944,6 +957,11 @@ func TestGetVPCInterfaceConfig(t *testing.T) {
 						{
 							SubnetID: 456,
 							Label:    "subnet-1",
+							IPv6: []linodego.VPCIPv6Range{
+								{
+									Range: "2001:0db8::/56",
+								},
+							},
 						},
 					}
 					return nil
@@ -1032,6 +1050,11 @@ func TestGetVPCInterfaceConfig(t *testing.T) {
 						{
 							SubnetID: 456,
 							Label:    "subnet-1",
+							IPv6: []linodego.VPCIPv6Range{
+								{
+									Range: "2001:0db8::/56",
+								},
+							},
 						},
 					}
 					return nil