runem 0.1.2__tar.gz → 0.2.0__tar.gz

{runem-0.1.2 → runem-0.2.0}/HISTORY.md

@@ -4,6 +4,33 @@ Changelog
 
 (unreleased)
 ------------
+- Merge pull request #63 from
+  lursight/dependabot/github_actions/actions/cache-4. [Frank Harrison]
+
+  chore(deps): bump actions/cache from 3 to 4
+- Chore(deps): bump actions/cache from 3 to 4. [dependabot[bot]]
+
+  Bumps [actions/cache](https://github.com/actions/cache) from 3 to 4.
+  - [Release notes](https://github.com/actions/cache/releases)
+  - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
+  - [Commits](https://github.com/actions/cache/compare/v3...v4)
+
+  ---
+  updated-dependencies:
+  - dependency-name: actions/cache
+    dependency-type: direct:production
+    update-type: version-update:semver-major
+  ...
+- Feat(simple-files): adds ability to pass file-lists to simple-
+  commands. [Frank Harrison]
+- Feat(simple-files): strongly types the simple job executor. [Frank
+  Harrison]
+- Feat(simple-files): fixes docstring. [Frank Harrison]
+
+
+0.1.2 (2024-11-18)
+------------------
+- Release: version 0.1.2 🚀 [Frank Harrison]
 - Chore(actions): simplifies code paths for actions. [Frank Harrison]
 - Chore(contrib): removes the Makefile commands that have been removed.
   [Frank Harrison]

{runem-0.1.2 → runem-0.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: runem
-Version: 0.1.2
+Version: 0.2.0
 Summary: Awesome runem created by lursight
 Home-page: https://github.com/lursight/runem/
 Author: lursight

runem-0.2.0/runem/VERSION

@@ -0,0 +1 @@
+0.2.0

runem-0.2.0/runem/job_runner_simple_command.py

@@ -0,0 +1,46 @@
+import shlex
+import typing
+
+from typing_extensions import Unpack
+
+from runem.run_command import run_command
+from runem.types.common import FilePathList
+from runem.types.runem_config import JobConfig
+from runem.types.types_jobs import AllKwargs
+
+
+def validate_simple_command(command_string: str) -> typing.List[str]:
+    """Use shlex to handle parsing of the command string, a non-trivial problem."""
+    split_command: typing.List[str] = shlex.split(command_string)
+    return split_command
+
+
+def job_runner_simple_command(
+    **kwargs: Unpack[AllKwargs],
+) -> None:
+    """Parses the command and tries to run it via the system.
+
+    Commands inherit the environment.
+    """
+    # assume we have the job.command entry, allowing KeyError to propagate up
+    job_config: JobConfig = kwargs["job"]
+    command_string: str = job_config["command"]
+
+    command_string_files: str = command_string
+    if "{file_list}" in command_string:
+        file_list: FilePathList = kwargs["file_list"]
+        file_list_with_quotes: typing.List[str] = [
+            f'"{str(file_path)}"' for file_path in file_list
+        ]
+        command_string_files = command_string.replace(
+            "{file_list}", " ".join(file_list_with_quotes)
+        )
+
+    # use shlex to handle parsing of the command string, a non-trivial problem.
+    cmd = validate_simple_command(command_string_files)
+
+    # preserve quotes for consistent handling of strings and avoid the "word
+    # splitting" problem for unix-like shells.
+    cmd_with_quotes = [f'"{token}"' if " " in token else token for token in cmd]
+
+    run_command(cmd=cmd_with_quotes, **kwargs)

{runem-0.1.2 → runem-0.2.0}/runem.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: runem
-Version: 0.1.2
+Version: 0.2.0
 Summary: Awesome runem created by lursight
 Home-page: https://github.com/lursight/runem/
 Author: lursight

runem-0.2.0/tests/test_job_runner_simple_command.py

@@ -0,0 +1,99 @@
+import io
+import typing
+from contextlib import redirect_stdout
+from pathlib import Path
+from unittest.mock import Mock, patch
+
+from runem.config_metadata import ConfigMetadata
+from runem.job import Job
+from runem.job_runner_simple_command import job_runner_simple_command
+from runem.types.runem_config import JobConfig
+from tests.test_runem import gen_dummy_config_metadata
+
+
+@patch(
+    "runem.job_runner_simple_command.run_command",
+    # return_value=None,
+)
+def test_job_runner_simple_command(mock_run_command: Mock) -> None:
+    """Tests the basics of job_runner_simple_command."""
+    job_config: JobConfig = {
+        "command": "echo 'testing job_runner_simple_command'",
+    }
+    config_metadata: ConfigMetadata = gen_dummy_config_metadata()
+    with io.StringIO() as buf, redirect_stdout(buf):
+        ret: None = job_runner_simple_command(  # type: ignore[func-returns-value]
+            options=config_metadata.options,  # type: ignore
+            file_list=[],
+            procs=config_metadata.args.procs,
+            root_path=Path("."),
+            verbose=True,  # config_metadata.args.verbose,
+            # unpack useful data points from the job_config
+            label=Job.get_job_name(job_config),
+            job=job_config,
+        )
+        run_command_stdout = buf.getvalue()
+
+    assert ret is None
+    assert run_command_stdout.split("\n") == [""]
+    mock_run_command.assert_called_once_with(
+        cmd=["echo", '"testing job_runner_simple_command"'],
+        file_list=[],
+        job={"command": "echo 'testing job_runner_simple_command'"},
+        label="echo 'testing job_runner_simple_command'",
+        options={},
+        procs=1,
+        root_path=Path("."),
+        verbose=True,
+    )
+
+
+@patch(
+    "runem.job_runner_simple_command.run_command",
+    # return_value=None,
+)
+def test_job_runner_simple_command_with_file_list(mock_run_command: Mock) -> None:
+    """Tests the basics of job_runner_simple_command."""
+    test_cmd_string: str = (
+        'echo "some option before files" {file_list} "some option after files"'
+    )
+    job_config: JobConfig = {
+        "command": test_cmd_string,
+    }
+    config_metadata: ConfigMetadata = gen_dummy_config_metadata()
+    file_list: typing.List[typing.Union[str, Path]] = [
+        Path("file1"),
+        "file2",
+        "file with spaces",
+    ]
+    with io.StringIO() as buf, redirect_stdout(buf):
+        ret: None = job_runner_simple_command(  # type: ignore[func-returns-value]
+            file_list=file_list,  # type: ignore[arg-type] # intentional type mismatch
+            job=job_config,
+            label=Job.get_job_name(job_config),
+            options=config_metadata.options,  # type: ignore
+            procs=config_metadata.args.procs,
+            root_path=Path("."),
+            verbose=True,  # config_metadata.args.verbose,
+        )
+        run_command_stdout = buf.getvalue()
+
+    assert ret is None
+    assert run_command_stdout.split("\n") == [""]
+    mock_run_command.assert_called_once_with(
+        cmd=[
+            "echo",
+            '"some option before files"',
+            "file1",
+            "file2",
+            '"file with spaces"',
+            '"some option after files"',
+        ],
+        file_list=file_list,
+        job=job_config,
+        label=test_cmd_string,
+        options={},
+        procs=1,
+        root_path=Path("."),
+        verbose=True,
+    )

{runem-0.1.2 → runem-0.2.0}/tests/test_runem.py

@@ -1780,7 +1780,7 @@ def test_progress_updater_with_false(show_spinner: bool) -> None:
         )
 
 
-def _dummy_config_metadata() -> ConfigMetadata:
+def gen_dummy_config_metadata() -> ConfigMetadata:
     config_metadata: ConfigMetadata = ConfigMetadata(
         cfg_filepath=pathlib.Path(__file__),
         phases=("dummy phase 1",),
@@ -1809,7 +1809,7 @@ def _dummy_config_metadata() -> ConfigMetadata:
 
 
 DUMMY_MAIN_RETURN: MainReturnType = (
-    _dummy_config_metadata(),  # ConfigMetadata,
+    gen_dummy_config_metadata(),  # ConfigMetadata,
     {},  # job_run_metadatas,
     IntentionalTestError(),
 )

runem-0.1.2/runem/VERSION

@@ -1 +0,0 @@
-0.1.2

runem-0.1.2/runem/job_runner_simple_command.py

@@ -1,32 +0,0 @@
-import shlex
-import typing
-
-from runem.run_command import run_command
-from runem.types.runem_config import JobConfig
-
-
-def validate_simple_command(command_string: str) -> typing.List[str]:
-    # use shlex to handle parsing of the command string, a non-trivial problem.
-    split_command: typing.List[str] = shlex.split(command_string)
-    return split_command
-
-
-def job_runner_simple_command(
-    **kwargs: typing.Any,
-) -> None:
-    """Parses the command and tries to run it via the system.
-
-    Commands inherit the environment.
-    """
-    # assume we have the job.command entry, allowing KeyError to propagate up
-    job_config: JobConfig = kwargs["job"]
-    command_string: str = job_config["command"]
-
-    # use shlex to handle parsing of the command string, a non-trivial problem.
-    result = validate_simple_command(command_string)
-
-    # preserve quotes for consistent handling of strings and avoid the "word
-    # splitting" problem for unix-like shells.
-    result_with_quotes = [f'"{token}"' if " " in token else token for token in result]
-
-    run_command(cmd=result_with_quotes, **kwargs)

runem-0.1.2/tests/test_job_runner_simple_command.py

@@ -1,39 +0,0 @@
-import io
-from contextlib import redirect_stdout
-from unittest.mock import Mock, patch
-
-from runem.job import Job
-from runem.job_runner_simple_command import job_runner_simple_command
-from runem.types.runem_config import JobConfig
-
-
-@patch(
-    "runem.job_runner_simple_command.run_command",
-    # return_value=None,
-)
-def test_job_runner_simple_command(mock_run_command: Mock) -> None:
-    """Tests the basics of job_runner_simple_command."""
-    job_config: JobConfig = {
-        "command": "echo 'testing job_runner_simple_command'",
-    }
-    with io.StringIO() as buf, redirect_stdout(buf):
-        ret: None = job_runner_simple_command(  # type: ignore[func-returns-value]
-            # options=config_metadata.options,  # type: ignore
-            # file_list=file_list,
-            # procs=config_metadata.args.procs,
-            # root_path=root_path,
-            verbose=True,  # config_metadata.args.verbose,
-            # unpack useful data points from the job_config
-            label=Job.get_job_name(job_config),
-            job=job_config,
-        )
-        run_command_stdout = buf.getvalue()
-
-    assert ret is None
-    assert run_command_stdout.split("\n") == [""]
-    mock_run_command.assert_called_once_with(
-        cmd=["echo", '"testing job_runner_simple_command"'],
-        verbose=True,
-        label="echo 'testing job_runner_simple_command'",
-        job={"command": "echo 'testing job_runner_simple_command'"},
-    )