fix: change the condition to consider a DV import failed

The test `TaskRun fails and DataVolume is modified but does not import
successfully` checks if a DV has failed by asserting several fields
on the DV. One of them is `conditions[cdiv1beta1.DataVolumeBound].Status
== v1.ConditionTrue`. In the current version of CDI, if a DV fails to
fetch the import link, which is the scenario in this test, the
underlining PVC is not bound. Therefore, the test assertion fails even
if the DV has failed to import.

Signed-off-by: Javier Cano Cano <jcanocan@redhat.com>

modules/tests/test/dataobject/operations.go

@@ -56,7 +56,7 @@ func HasDataVolumeFailedToImport(dataVolume *cdiv1beta1.DataVolume) bool {
 	conditions := getConditionMapDv(dataVolume)
 	return dataVolume.Status.Phase == cdiv1beta1.ImportInProgress &&
 		dataVolume.Status.RestartCount > constants.UnusualRestartCountThreshold &&
-		conditions[cdiv1beta1.DataVolumeBound].Status == v1.ConditionTrue &&
+		conditions[cdiv1beta1.DataVolumeBound].Status == v1.ConditionFalse &&
 		conditions[cdiv1beta1.DataVolumeRunning].Status == v1.ConditionFalse &&
 		conditions[cdiv1beta1.DataVolumeRunning].Reason == constants.ReasonError
 }