Text fixes

Author: henricasanova

.github/workflows/build.yml

@@ -34,7 +34,6 @@ jobs:
 
       - name: Check package install
         run: |
-          sudo sed -i 's|if os.path.exists(cpu_benchmark_path):|if True:|' ./setup.py
           pip install .
 
       - name: Run tests

tests/test_helpers.py

@@ -47,7 +47,6 @@ def _install_WfCommons_on_container(container):
                                            stderr=True)
     exit_code, output = container.exec_run("sudo /bin/rm -rf /tmp/WfCommons/bin/cpu-benchmark", stdout=True,
                                            stderr=True)
-    exit_code, output = container.exec_run(r"sudo sed -i 's|if os.path.exists(cpu_benchmark_path):|if True:|' /tmp/WfCommons/setup.py")
 
     # Install WfCommons on the container (to install wfbench and cpu-benchmark really)
     exit_code, output = container.exec_run("sudo python3 -m pip install . --break-system-packages",

tests/translators_loggers/test_translators_loggers.py

@@ -259,9 +259,11 @@ def test_translator(self, backend) -> None:
         translator = translator_classes[backend](benchmark.workflow)
         translator.translate(output_folder=dirpath)
 
-        # Make the directory that holds the translation world-writable,
-        # so that docker commands won't fail
-        os.chmod(dirpath, 0o777)
+        # # Make the directory that holds the translation world-writable,
+        # # so that docker commands won't fail
+        # TODO: Explore whether this below makes tests runnable on Linux due to
+        #       different Docker permission schemes, etc.
+        # os.chmod(dirpath, 0o777)
 
         # Start the Docker container
         container = _start_docker_container(backend, str_dirpath, str_dirpath, str_dirpath + "bin/")