If smartless is invoked with a single [-h|-\?|--help|-v|-V|--version], act accordingly

Signed-off-by: Stavros Ntentos <[email protected]>

Author: Stavros Ntentos

Diff for: .github/workflows/test.yml

@@ -0,0 +1,36 @@
+name: Test
+
+on:
+  push:
+    branches:
+      - master
+      - 'dev/**'
+    tags:
+      - '*'
+  pull_request:
+    branches:
+      - master
+
+permissions:
+  contents: read
+
+env:
+  PYTHON_VERSION: 3.9
+
+jobs:
+  tests:
+    name: Core Test
+    if: "!contains(github.event.head_commit.message, '[CI SKIP]')"
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - name: Set up Python ${{ env.PYTHON_VERSION }}
+      uses: actions/setup-python@v4
+      with:
+        python-version: '${{ env.PYTHON_VERSION }}'
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade --requirement requirements-dev.txt
+    - name: Run tests
+      run: |
+        pytest

Diff for: smartless

@@ -119,6 +119,31 @@ for var ; do
   fi
 done
 
+if [ "${#args[@]}" -eq 1 ] && [ "${SMARTLESS_PAGER}" == 'less' ] ; then
+  case "${args[0]}" in
+    -h|-\?|--help)
+      head -7 "${BASH_SOURCE[0]}"
+      if [ "${args[0]}" == '-h' ] ; then
+        echo
+        echo "less (v551) has no '-h' argument; but invoking help"
+      fi
+      less -?
+      exit $?
+    ;;
+    -v|-V|--version)
+      head -7 "${BASH_SOURCE[0]}"
+      echo
+      if [ "${args[0]}" == '-v' ] ; then
+        echo "less (v551) has no '-v' argument; but invoking version"
+        echo
+      fi
+      less -V
+      exit $?
+    ;;
+    *);;
+  esac
+fi
+
 # if more than one file provided on cmdline, revert to original less ...
 if (( isfile > 1 )); then
   exec "${SMARTLESS_PAGER}" "${SMARTLESS_PAGER_ARGUMENTS}" "$@"

Diff for: tests/__init__.py

@@ -0,0 +1,50 @@
+#!/usr/bin/env python3
+
+import os
+import pty
+import sys
+from pathlib import Path
+
+import pytest
+
+from tests.structures import FixtureFD
+
+SMARTLESS_OPT = "--smartless"
+
+
+@pytest.fixture
+def smartless(request):
+    smartless_argv = [str(request.config.getoption(SMARTLESS_OPT))]
+
+    debug = getattr(request.config, "use_pdb", False)
+    debug |= getattr(request.config, "use_pudb", False)
+    debug |= sys.modules.get("_jb_runner_tools") is not None
+
+    if debug:
+        smartless_argv = ["bash", "-x"] + smartless_argv
+
+    yield smartless_argv
+
+
+@pytest.fixture
+def pty_factory():
+    master_fd, slave_fd = pty.openpty()
+
+    yield FixtureFD(parent=master_fd, child=slave_fd)
+
+    os.close(master_fd)
+    os.close(slave_fd)
+
+
+pty_stdin = pty_factory
+pty_stdout = pty_factory
+pty_stderr = pty_factory
+
+
+def pytest_addoption(parser):
+    parser.addoption(
+        SMARTLESS_OPT,
+        action="store",
+        default=Path(__file__).parent.parent / "smartless",
+        help="smartless path",
+    )

Diff for: tests/conftest.py

@@ -0,0 +1,16 @@
+from __future__ import annotations
+
+from typing import TYPE_CHECKING, NamedTuple
+
+if TYPE_CHECKING:
+    from _typeshed import FileDescriptorLike
+
+
+class Dimensions(NamedTuple):
+    rows: int
+    columns: int
+
+
+class FixtureFD(NamedTuple):
+    parent: FileDescriptorLike
+    child: FileDescriptorLike

Diff for: tests/structures.py

@@ -0,0 +1,53 @@
+#!/usr/bin/env python3
+
+import os
+import re
+import subprocess
+
+import pytest
+
+
+@pytest.mark.parametrize("help_flag", ["-h", "--help", "-?"])
+def test_help(smartless, help_flag):
+    with subprocess.Popen(
+        [*smartless, help_flag],
+        text=True,
+        stdout=subprocess.PIPE,
+    ) as process:
+        stdout, _ = process.communicate(None)
+
+    with open("smartless", encoding="utf-8") as f:
+        smartless_head = "".join([next(f) for _ in range(7)])
+
+    assert smartless_head in stdout
+    assert process.wait(5) == os.EX_OK
+
+    if help_flag == "-h":
+        assert "less (v551) has no '-h' argument; but invoking help" in stdout
+
+    assert "Commands marked with * may be preceded by a number," in stdout
+
+
+@pytest.mark.parametrize("version_flag", ["-v", "-V", "--version"])
+def test_version(smartless, version_flag):
+    with subprocess.Popen(
+        [*smartless, "-v"], text=True, stdout=subprocess.PIPE
+    ) as process:
+        stdout, _ = process.communicate()
+
+    with open("smartless", encoding="utf-8") as f:
+        smartless_head = "".join([next(f) for _ in range(7)])
+
+    assert smartless_head in stdout
+    assert process.wait(5) == os.EX_OK
+
+    if version_flag == "-v":
+        assert "less (v551) has no '-v' argument; but invoking version" in stdout
+
+    assert re.search(r"less \d+ \(GNU regular expressions\)", stdout)
+
+
+if __name__ == "__main__":
+    import sys
+
+    sys.exit(pytest.main())