test: Adjust cleanup/error paths for nodedev test APIs
test: Adjust cleanup/error paths for nodedev test APIs

 - In testDestroyVport rather than use a cleanup label, just return -1
   immediately since nothing else is needed.

 - In testStoragePoolDestroy, if !privpool, then just return -1 since
   nothing else will happen anyway.

 - Rather than "goto cleanup;" on failure to virNodeDeviceObjFindByName
   an @obj, just return directly.  This then allows the cleanup: label code
   to not have to check "if (obj)" before calling virNodeDeviceObjUnlock.
   This also simplifies some exit logic...

 - In testNodeDeviceObjFindByName use an error: label to handle the failure
   and don't do the ncaps++ within the VIR_STRDUP() source target index.
   Only increment ncaps after success. Easier on eyes at error label too.

 - In testNodeDeviceDestroy use "cleanup" rather than "out" for the goto

Signed-off-by: John Ferlan <87558058f6f829e5ec976c8ef960720af4ff9c7d@redhat.com>

datto/libvirt,nertpinx/libvirt,fabianfreyer/libvirt,eskultety/libvirt,datto/libvirt,crobinso/libvirt,jardasgit/libvirt,crobinso/libvirt,crobinso/libvirt,jardasgit/libvirt,nertpinx/libvirt,olafhering/libvirt,fabianfreyer/libvirt,datto/libvirt,eskultety/libvirt,fabianfreyer/libvirt,libvirt/libvirt,libvirt/libvirt,andreabolognani/libvirt,jardasgit/libvirt,olafhering/libvirt,zippy2/libvirt,jfehlig/libvirt,olafhering/libvirt,jfehlig/libvirt,olafhering/libvirt,datto/libvirt,jfehlig/libvirt,nertpinx/libvirt,libvirt/libvirt,andreabolognani/libvirt,fabianfreyer/libvirt,andreabolognani/libvirt,eskultety/libvirt,jardasgit/libvirt,libvirt/libvirt,datto/libvirt,andreabolognani/libvirt,crobinso/libvirt,nertpinx/libvirt,nertpinx/libvirt,fabianfreyer/libvirt,eskultety/libvirt,jfehlig/libvirt,zippy2/libvirt,eskultety/libvirt,zippy2/libvirt,zippy2/libvirt,jardasgit/libvirt,andreabolognani/libvirt