move.py: check copy_result_tb return

copy_result_tb returns the response from the PUT, and
does not raise an exception for HTTP errrors; so we
now check the return value and raise an exception
on error.

lib/pbench/cli/agent/commands/results/move.py

@@ -104,9 +104,11 @@ def execute(self, single_threaded: bool, delete: bool = True) -> int:
                         self.config,
                         self.logger,
                     )
-                    crt.copy_result_tb(
+                    res = crt.copy_result_tb(
                         self.context.token, self.context.access, self.context.metadata
                     )
+                    if not res.ok:
+                        raise CopyResultTb.FileUploadError
                 except Exception as exc:
                     if isinstance(exc, (CopyResultTb.FileUploadError, RuntimeError)):
                         msg = "Error uploading file"