This PR fixes oVirt#82 and check that vnic doesn't has same network

client_nic_create_test.go

@@ -59,7 +59,7 @@ func TestDuplicateVMNICCreationWithSameName(t *testing.T) {
 	assertNICCount(t, vm, 0)
 }
 
-func TestDuplicateVMNICCreationWithSameNameAndDiffNetwork(t *testing.T) {
+func TestDuplicateVMNICCreationWithSameNameAndDiffVNICProfile(t *testing.T) {
 	t.Parallel()
 	helper := getHelper(t)
 	nicName := "test_duplicate_name"
@@ -79,12 +79,21 @@ func TestDuplicateVMNICCreationWithSameNameAndDiffNetwork(t *testing.T) {
 		ovirtclient.CreateNICParams())
 	assertNICCount(t, vm, 1)
 	DiffVNICProfile, _ := assertCanFindDiffVNICProfile(helper, helper.GetVNICProfileID())
-	assertCannotCreateNICWithSameNameDiffNetwork(
-		t,
-		vm,
-		nicName,
-		DiffVNICProfile,
-		ovirtclient.CreateNICParams())
+	if DiffVNICProfile == "" {
+		assertCannotCreateNICWithSameName(
+			t,
+			helper,
+			vm,
+			nicName,
+			ovirtclient.CreateNICParams())
+	} else {
+		assertCannotCreateNICWithVNICProfile(
+			t,
+			vm,
+			nicName,
+			DiffVNICProfile,
+			ovirtclient.CreateNICParams())
+	}
 	assertNICCount(t, vm, 1)
 	assertCanRemoveNIC(t, nic1)
 	assertNICCount(t, vm, 0)

client_nic_test.go

@@ -41,22 +41,23 @@ func assertCannotCreateNICWithSameName(
 	name string,
 	params ovirtclient.BuildableNICParameters,
 ) {
-	nic, _ := vm.CreateNIC(name, helper.GetVNICProfileID(), params)
-	if nic != nil {
+
+	_, err := vm.CreateNIC(name, helper.GetVNICProfileID(), params)
+	if err == nil {
 		t.Fatalf("create 2 NICs with same name %s", name)
 	}
 }
 
-func assertCannotCreateNICWithSameNameDiffNetwork(
+func assertCannotCreateNICWithVNICProfile(
 	t *testing.T,
 	vm ovirtclient.VM,
 	name string,
 	diffVNICProfile string,
 	params ovirtclient.BuildableNICParameters,
 ) {
-	nic, _ := vm.CreateNIC(name, diffVNICProfile, params)
-	if nic != nil {
-		t.Fatalf("create 2 NICs with same name %s", name)
+	_, err := vm.CreateNIC(name, diffVNICProfile, params)
+	if err == nil {
+		t.Fatalf("create 2 NICs with same name %s and different VNICProfile (%v)", name, err)
 	}
 }
 

client_vnicprofile_test.go

@@ -68,5 +68,5 @@ func assertCanFindDiffVNICProfile(helper ovirtclient.TestHelper, vnicProfileID s
 			return vnicID, nil
 		}
 	}
-	return vnicProfileID, fmt.Errorf("failed to find different VNIC profile ID for testing, use the exiting one")
+	return "", fmt.Errorf("failed to find different VNIC profile ID for testing, use the exiting one")
 }