Commit 2aaeefee authored by Cristian Pop's avatar Cristian Pop

Updated TestExtractDeviceAndPrefix and added TestExtractIface to reflect the…

Updated TestExtractDeviceAndPrefix and added TestExtractIface to reflect the changes brought by the #39202 fix.
parent b0d285c7
...@@ -57,10 +57,20 @@ func TestGetDevicePrefixRefCount(t *testing.T) { ...@@ -57,10 +57,20 @@ func TestGetDevicePrefixRefCount(t *testing.T) {
func TestExtractDeviceAndPrefix(t *testing.T) { func TestExtractDeviceAndPrefix(t *testing.T) {
devicePath := "127.0.0.1:3260-iqn.2014-12.com.example:test.tgt00" devicePath := "127.0.0.1:3260-iqn.2014-12.com.example:test.tgt00"
mountPrefix := "/var/lib/kubelet/plugins/kubernetes.io/iscsi/default/" + devicePath
lun := "-lun-0" lun := "-lun-0"
device, prefix, err := extractDeviceAndPrefix("/var/lib/kubelet/plugins/kubernetes.io/iscsi/" + devicePath + lun) device, prefix, err := extractDeviceAndPrefix(mountPrefix + lun)
if err != nil || device != (devicePath+lun) || prefix != devicePath { if err != nil || device != (devicePath+lun) || prefix != mountPrefix {
t.Errorf("extractDeviceAndPrefix: expected %s and %s, got %v %s and %s", devicePath+lun, devicePath, err, device, prefix) t.Errorf("extractDeviceAndPrefix: expected %s and %s, got %v %s and %s", devicePath+lun, mountPrefix, err, device, prefix)
}
}
func TestExtractIface(t *testing.T) {
ifaceName := "default"
devicePath := "127.0.0.1:3260-iqn.2014-12.com.example:test.tgt00-lun-0"
iface, err := extractIface("/var/lib/kubelet/plugins/kubernetes.io/iscsi/" + ifaceName + "/" + devicePath)
if err != nil || iface != ifaceName {
t.Errorf("extractIface: expected %s, got %v %s", ifaceName, err, iface)
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment