test: Use virStoragePoolObjIsDuplicate for storage define/create

Avoid the chance that there could be a duplicate storage pool UUID
or Name from the test driver storage pool define/create functions.

diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 8adc2167b454b13bc90054038d36c7f2109d34cc..dc743b49883542bbe4c4e53925f0579011a2193c 100644 (file)
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -4440,14 +4440,8 @@ testStoragePoolCreateXML(virConnectPtr conn,
     if (!(newDef = virStoragePoolDefParseString(xml)))
         goto cleanup;
 
-    obj = virStoragePoolObjFindByUUID(privconn->pools, newDef->uuid);
-    if (!obj)
-        obj = virStoragePoolObjFindByName(privconn->pools, newDef->name);
-    if (obj) {
-        virReportError(VIR_ERR_INTERNAL_ERROR,
-                       "%s", _("storage pool already exists"));
+    if (virStoragePoolObjIsDuplicate(privconn->pools, newDef, true) < 0)
         goto cleanup;
-    }
 
     if (!(obj = virStoragePoolObjAssignDef(privconn->pools, newDef)))
         goto cleanup;
@@ -4520,6 +4514,9 @@ testStoragePoolDefineXML(virConnectPtr conn,
     newDef->allocation = defaultPoolAlloc;
     newDef->available = defaultPoolCap - defaultPoolAlloc;
 
+    if (virStoragePoolObjIsDuplicate(privconn->pools, newDef, false) < 0)
+        goto cleanup;
+
     if (!(obj = virStoragePoolObjAssignDef(privconn->pools, newDef)))
         goto cleanup;
     newDef = NULL;