In-place replacement in file creates and removes back-up file

Author: snkas

exputil/shell.py

@@ -252,12 +252,12 @@ def copy_file(self, source_file, target_path):
         return self.perfect_exec("scp %s %s" % (source_file, target_path))
 
     def sed_replace_in_file_plain(self, target_file, search_term, replace_term):
-        return self.perfect_exec("sed -i "
+        return self.perfect_exec("sed -i'.original' "
                                  "'s/"
                                  + re.escape(search_term).replace("/", "\\/")
                                  + "/"
                                  + replace_term.replace("\\", "\\\\").replace("/", "\\/").replace("&", "\\&")
-                                 + "/g' " + target_file)
+                                 + "/g' " + target_file + "; rm " + target_file + ".original")
 
     def path_exists(self, path):
         res = self.valid_exec("if [ -d \"%s\" ]; then exit 0; else exit 1; fi" % path.replace('"', '\\"'))

setup.py

@@ -18,7 +18,7 @@
 
 setup(
     name='exputil',
-    version='1.5',
+    version='1.6',
     description="Experiment utilities",
     license="Apache 2.0",
     python_requires='>=3.7',

tests/test_shell_file_dir.py

@@ -30,6 +30,8 @@ def test_file(self):
         local_shell.write_file("temp/test.txt", "Test")
         self.assertTrue(local_shell.file_exists("temp/test.txt"))
         self.assertEqual("Test", local_shell.read_file("temp/test.txt").strip())
+        local_shell.sed_replace_in_file_plain("temp/test.txt", "Tes", "ABc")
+        self.assertEqual("ABct", local_shell.read_file("temp/test.txt").strip())
         with open("temp/test.txt", "w+") as f:
             writer = InstantWriter(f)
             writer.write("Ano/.ther testno/.ther")
@@ -49,6 +51,16 @@ def test_file(self):
             self.assertTrue(True)
         local_shell.remove_recursive("temp")
 
+    def test_sed_replace(self):
+        local_shell = LocalShell()
+        local_shell.remove_force("temp")
+        local_shell.make_full_dir("temp")
+        local_shell.write_file("temp/testA.txt", "This is a blue world blue world")
+        local_shell.sed_replace_in_file_plain("temp/testA.txt", "blue world", "green planet")
+        self.assertEqual("This is a green planet green planet", local_shell.read_file("temp/testA.txt").strip())
+        local_shell.remove("temp/testA.txt")
+        local_shell.remove_recursive("temp")
+
     def test_dir(self):
         local_shell = LocalShell()
         local_shell.make_full_dir("temp")