quoting isn't that easy

matyasselmeci · May 22, 2019 · f663a58 · f663a58
1 parent d5a5f79
commit f663a58
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 10 deletions.
diff --git a/htcondor_jobs/handles.py b/htcondor_jobs/handles.py
@@ -194,8 +194,6 @@ def edit(self, attr: str, value: Union[str, int, float]) -> classad.ClassAd:
         logger.info(
             f"Executing edit {attr} = {value} against schedd {self.schedd} with constraint {cs}"
         )
-        if isinstance(value, str):
-            value = f'"{value}"'
         return self.schedd.edit(cs, attr, str(value))
 
 

diff --git a/tests/integration/test_handles/test_edit.py b/tests/integration/test_handles/test_edit.py
@@ -29,11 +29,3 @@ def test_change_request_memory(long_sleep):
     handle.edit("RequestMemory", 12345)
 
     assert get_job_attr(handle, "RequestMemory") == 12345
-
-
-def test_change_executable(long_sleep):
-    handle = jobs.submit(long_sleep, count=1)
-
-    handle.edit("Executable", "hello")
-
-    assert get_job_attr(handle, "Executable") == "hello"