fix: logfile quoting and scancel error handling (#140)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
- Enhanced handling of output paths in SLURM job commands to better
accommodate spaces and special characters.
- Improved error reporting for job cancellations, providing clearer
feedback on failures.

- **Bug Fixes**
- Added robust exception handling for job cancellation processes to
capture and report errors effectively.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Christian Meesters <[email protected]>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

snakemake_executor_plugin_slurm/__init__.py

@@ -137,7 +137,7 @@ def run_job(self, job: JobExecutorInterface):
             f"sbatch "
             f"--parsable "
             f"--job-name {self.run_uuid} "
-            f"--output {slurm_logfile} "
+            f"--output '{slurm_logfile}' "
             f"--export=ALL "
             f"--comment {comment_str}"
         )
@@ -408,6 +408,14 @@ def cancel_jobs(self, active_jobs: List[SubmittedJobInfo]):
                 )
             except subprocess.TimeoutExpired:
                 self.logger.warning("Unable to cancel jobs within a minute.")
+            except subprocess.CalledProcessError as e:
+                msg = e.stderr.strip()
+                if msg:
+                    msg = f": {msg}"
+                raise WorkflowError(
+                    "Unable to cancel jobs with scancel "
+                    f"(exit code {e.returncode}){msg}"
+                ) from e
 
     async def job_stati(self, command):
         """Obtain SLURM job status of all submitted jobs with sacct