add an ut

Author: paulyufan2

cni/network/network.go

@@ -856,6 +856,7 @@ func (plugin *NetPlugin) createEpInfo(opt *createEpInfoOpt) (*network.EndpointIn
 	}
 
 	setEndpointOptions(opt.cnsNetworkConfig, &endpointInfo, vethName)
+
 	logger.Info("Generated endpoint info from fields", zap.String("epInfo", endpointInfo.PrettyString()))
 
 	// now our ep info should have the full combined information from both the network and endpoint structs

cns/middlewares/k8sSwiftV2_windows.go

@@ -1,8 +1,6 @@
 package middlewares
 
 import (
-	"fmt"
-
 	"github.com/Azure/azure-container-networking/cns"
 	"github.com/Azure/azure-container-networking/cns/logger"
 	"github.com/Azure/azure-container-networking/cns/middlewares/utils"
@@ -17,7 +15,6 @@ func (k *K8sSWIFTv2Middleware) setRoutes(podIPInfo *cns.PodIpInfo) error {
 		logger.Printf("[SWIFTv2Middleware] skip setting default route on InfraNIC interface")
 		podIPInfo.SkipDefaultRoutes = true
 	}
-
 	return nil
 }
 
@@ -44,14 +41,11 @@ func (k *K8sSWIFTv2Middleware) assignSubnetPrefixLengthFields(podIPInfo *cns.Pod
 		GatewayIPAddress: interfaceInfo.GatewayIP,
 	}
 	// assign default route
-	defaultRoute := cns.Route{
-		IPAddress: fmt.Sprintf("%s/%d", interfaceInfo.GatewayIP, prefixLength),
-	}
 	route := cns.Route{
 		IPAddress:        "0.0.0.0/0",
 		GatewayIPAddress: interfaceInfo.GatewayIP,
 	}
-	podIPInfo.Routes = append(podIPInfo.Routes, defaultRoute, route)
+	podIPInfo.Routes = append(podIPInfo.Routes, route)
 
 	return nil
 }

cns/middlewares/k8sSwiftV2_windows_test.go

@@ -1,6 +1,7 @@
 package middlewares
 
 import (
+	"reflect"
 	"testing"
 
 	"github.com/Azure/azure-container-networking/cns"
@@ -53,16 +54,29 @@ func TestAssignSubnetPrefixSuccess(t *testing.T) {
 		MacAddress: "12:34:56:78:9a:bc",
 	}
 
+	gatewayIP := "20.240.1.1"
 	intInfo := v1alpha1.InterfaceInfo{
-		GatewayIP:          "20.240.1.1",
+		GatewayIP:          gatewayIP,
 		SubnetAddressSpace: "20.240.1.0/16",
 	}
 
+	routes := []cns.Route{
+		{
+			IPAddress:        "0.0.0.0/0",
+			GatewayIPAddress: gatewayIP,
+		},
+	}
+
 	ipInfo := podIPInfo
 	err := middleware.assignSubnetPrefixLengthFields(&ipInfo, intInfo, ipInfo.PodIPConfig.IPAddress)
 	assert.Equal(t, err, nil)
 	// assert that the function for windows modifies all the expected fields with prefix-length
 	assert.Equal(t, ipInfo.PodIPConfig.PrefixLength, uint8(16))
 	assert.Equal(t, ipInfo.HostPrimaryIPInfo.Gateway, intInfo.GatewayIP)
 	assert.Equal(t, ipInfo.HostPrimaryIPInfo.Subnet, intInfo.SubnetAddressSpace)
+
+	// compare two slices of routes
+	if !reflect.DeepEqual(ipInfo.Routes, routes) {
+		t.Errorf("got '%+v', expected '%+v'", ipInfo.Routes, routes)
+	}
 }