mirror of
https://github.com/kata-containers/kata-containers.git
synced 2025-05-01 13:14:33 +00:00
virtcontainers: Add msg to existing utils unit tests
Add message to existing unit tests that use arrays and loops for easier debugging. Signed-off-by: Chelsea Mafrica <chelsea.e.mafrica@intel.com>
This commit is contained in:
parent
c3fc09b95b
commit
fd13c93c6a
@ -6,6 +6,7 @@
|
|||||||
package utils
|
package utils
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"fmt"
|
||||||
"io/ioutil"
|
"io/ioutil"
|
||||||
"os"
|
"os"
|
||||||
"path/filepath"
|
"path/filepath"
|
||||||
@ -199,10 +200,11 @@ func TestGetVirtDriveName(t *testing.T) {
|
|||||||
{18277, "vdzzz"},
|
{18277, "vdzzz"},
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, test := range tests {
|
for i, test := range tests {
|
||||||
|
msg := fmt.Sprintf("test[%d]: %+v", i, test)
|
||||||
driveName, err := GetVirtDriveName(test.index)
|
driveName, err := GetVirtDriveName(test.index)
|
||||||
assert.NoError(err)
|
assert.NoError(err, msg)
|
||||||
assert.Equal(driveName, test.expectedDrive)
|
assert.Equal(driveName, test.expectedDrive, msg)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -225,11 +227,12 @@ func TestGetSCSIIdLun(t *testing.T) {
|
|||||||
{513, 2, 1},
|
{513, 2, 1},
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, test := range tests {
|
for i, test := range tests {
|
||||||
|
msg := fmt.Sprintf("test[%d]: %+v", i, test)
|
||||||
scsiID, lun, err := GetSCSIIdLun(test.index)
|
scsiID, lun, err := GetSCSIIdLun(test.index)
|
||||||
assert.NoError(err)
|
assert.NoError(err, msg)
|
||||||
assert.Equal(scsiID, test.expectedScsiID)
|
assert.Equal(scsiID, test.expectedScsiID, msg)
|
||||||
assert.Equal(lun, test.expectedLun)
|
assert.Equal(lun, test.expectedLun, msg)
|
||||||
}
|
}
|
||||||
|
|
||||||
_, _, err := GetSCSIIdLun(-1)
|
_, _, err := GetSCSIIdLun(-1)
|
||||||
@ -251,10 +254,11 @@ func TestGetSCSIAddress(t *testing.T) {
|
|||||||
{512, "2:0"},
|
{512, "2:0"},
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, test := range tests {
|
for i, test := range tests {
|
||||||
|
msg := fmt.Sprintf("test[%d]: %+v", i, test)
|
||||||
scsiAddr, err := GetSCSIAddress(test.index)
|
scsiAddr, err := GetSCSIAddress(test.index)
|
||||||
assert.NoError(err)
|
assert.NoError(err, msg)
|
||||||
assert.Equal(scsiAddr, test.expectedSCSIAddress)
|
assert.Equal(scsiAddr, test.expectedSCSIAddress, msg)
|
||||||
}
|
}
|
||||||
|
|
||||||
_, err := GetSCSIAddress(-1)
|
_, err := GetSCSIAddress(-1)
|
||||||
@ -299,15 +303,16 @@ func TestBuildSocketPath(t *testing.T) {
|
|||||||
|
|
||||||
for i, d := range data {
|
for i, d := range data {
|
||||||
result, err := BuildSocketPath(d.elems...)
|
result, err := BuildSocketPath(d.elems...)
|
||||||
|
msg := fmt.Sprintf("test[%d]: %+v", i, d)
|
||||||
|
|
||||||
if d.valid {
|
if d.valid {
|
||||||
assert.NoErrorf(err, "test %d, data %+v", i, d)
|
assert.NoErrorf(err, "test %d, data %+v", i, d, msg)
|
||||||
} else {
|
} else {
|
||||||
assert.Errorf(err, "test %d, data %+v", i, d)
|
assert.Errorf(err, "test %d, data %+v", i, d, msg)
|
||||||
}
|
}
|
||||||
|
|
||||||
assert.NotNil(result)
|
assert.NotNil(result, msg)
|
||||||
assert.Equal(d.expected, result)
|
assert.Equal(d.expected, result, msg)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user