fast-dev-cli 0.14.1__tar.gz → 0.14.2__tar.gz

{fast_dev_cli-0.14.1 → fast_dev_cli-0.14.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: fast-dev-cli
-Version: 0.14.1
+Version: 0.14.2
 Summary: Python project development tool.
 Author-Email: Waket Zheng <waketzheng@gmail.com>>
 Classifier: Development Status :: 4 - Beta

fast_dev_cli-0.14.2/fast_dev_cli/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.14.2"

{fast_dev_cli-0.14.1 → fast_dev_cli-0.14.2}/fast_dev_cli/cli.py

@@ -38,6 +38,7 @@ else:  # pragma: no cover
 
 
 cli = typer.Typer()
+DryOption = Option(False, "--dry", help="Only print, not really run shell command")
 TOML_FILE = "pyproject.toml"
 ToolName = Literal["poetry", "pdm", "uv"]
 
@@ -348,7 +349,7 @@ def bump_version(
     commit: bool = Option(
         False, "--commit", "-c", help="Whether run `git commit` after version changed"
     ),
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Bump up version string in pyproject.toml"""
     return BumpUp(_ensure_bool(commit), getattr(part, "value", part), dry=dry).run()
@@ -612,7 +613,7 @@ class UpgradeDependencies(Project, DryRun):
 
 @cli.command()
 def upgrade(
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Upgrade dependencies in pyproject.toml to latest versions"""
     if (tool := Project.get_manage_tool()) == "uv":
@@ -666,7 +667,7 @@ class GitTag(DryRun):
 @cli.command()
 def tag(
     message: str = Option("", "-m", "--message"),
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Run shell command: git tag -a <current-version-in-pyproject.toml> -m {message}"""
     GitTag(message, dry=dry).run()
@@ -720,6 +721,8 @@ class LintCode(DryRun):
         skip_mypy: bool = False,
         use_dmypy: bool = False,
     ) -> str:
+        if paths != "." and all(i.endswith(".html") for i in paths.split()):
+            return f"prettier -w {paths}"
         cmd = ""
         tools = ["ruff format", "ruff check --extend-select=I,B,SIM --fix", "mypy"]
         if check_only:
@@ -802,7 +805,7 @@ def make_style(
     use_dmypy: bool = Option(
         False, "--dmypy", help="Use `dmypy run` instead of `mypy`"
     ),
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Run: ruff check/format to reformat code and then mypy to check"""
     if getattr(files, "default", files) is None:
@@ -822,7 +825,7 @@ def make_style(
 def only_check(
     bandit: bool = Option(False, "--bandit", help="Run `bandit -r <package_dir>`"),
     skip_mypy: bool = Option(False, "--skip-mypy"),
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Check code style without reformat"""
     check(dry=dry, bandit=bandit, skip_mypy=_ensure_bool(skip_mypy))
@@ -873,7 +876,7 @@ def sync(
     save: bool = Option(
         False, "--save", "-s", help="Whether save the requirement file"
     ),
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Export dependencies by poetry to a txt file then install by pip."""
     Sync(filename, extras, save, dry=dry).run()
@@ -907,7 +910,7 @@ def test(dry: bool, ignore_script=False) -> None:
 
 @cli.command(name="test")
 def coverage_test(
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
     ignore_script: bool = Option(False, "--ignore-script", "-i"),
 ) -> None:
     """Run unittest by pytest and report coverage"""
@@ -930,7 +933,7 @@ class Publish:
 
 @cli.command()
 def upload(
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Shortcut for package publish"""
     cmd = Publish.gen()
@@ -969,7 +972,7 @@ def runserver(
     file_or_port: Optional[str] = typer.Argument(default=None),
     port: Optional[int] = Option(None, "-p", "--port"),
     host: Optional[str] = Option(None, "-h", "--host"),
-    dry: bool = Option(False, "--dry", help="Only print, not really run shell command"),
+    dry: bool = DryOption,
 ) -> None:
     """Start a fastapi server(only for fastapi>=0.111.0)"""
     if getattr(file_or_port, "default", file_or_port):

{fast_dev_cli-0.14.1 → fast_dev_cli-0.14.2}/pyproject.toml

@@ -41,7 +41,7 @@ dependencies = [
     "pytest >=8.2.0,<9",
     "packaging>=20.5",
 ]
-version = "0.14.1"
+version = "0.14.2"
 
 [project.urls]
 Homepage = "https://github.com/waketzheng/fast-dev-cli"

{fast_dev_cli-0.14.1 → fast_dev_cli-0.14.2}/tests/test_bump.py

@@ -17,10 +17,11 @@ from fast_dev_cli.cli import (
     StrEnum,
     bump,
     bump_version,
+    capture_cmd_output,
     get_current_version,
 )
 
-from .utils import chdir, mock_sys_argv
+from .utils import chdir, mock_sys_argv, prepare_poetry_project
 
 
 def test_enum():
@@ -53,12 +54,14 @@ def test_bump_dry(mocker):
     mocker.patch("fast_dev_cli.cli.Project.manage_by_poetry", return_value=True)
     with pytest.raises(ShellCommandError):
         get_current_version()
-    # TODO:
-    # version = get_current_version()
-    # patch_without_commit, patch_with_commit, minor_with_commit = _bump_commands(version)
-    # assert BumpUp(part="patch", commit=False, dry=True).gen() == patch_without_commit
-    # assert BumpUp(part="patch", commit=True, dry=True).gen() == patch_with_commit
-    # assert BumpUp(part="minor", commit=True, dry=True).gen() == minor_with_commit
+    mocker.patch("fast_dev_cli.cli.Project.manage_by_poetry", return_value=False)
+    version = get_current_version()
+    patch_without_commit, patch_with_commit, minor_with_commit = _bump_commands(
+        version, filename="fast_dev_cli/__init__.py"
+    )
+    assert BumpUp(part="patch", commit=False, dry=True).gen() == patch_without_commit
+    assert BumpUp(part="patch", commit=True, dry=True).gen() == patch_with_commit
+    assert BumpUp(part="minor", commit=True, dry=True).gen() == minor_with_commit
 
 
 def test_bump(
@@ -145,36 +148,46 @@ def test_bump_with_emoji(mocker, tmp_path, monkeypatch):
     mocker.patch("fast_dev_cli.cli.Project.manage_by_poetry", return_value=True)
     with pytest.raises(ShellCommandError):
         get_current_version()
-    # TODO:
-    # version = get_current_version()
-    # patch_without_commit, patch_with_commit, minor_with_commit = _bump_commands(
-    #     version, emoji=True
-    # )
-    # last_commit = "📝 Update release notes"
-    # mocker.patch(
-    #     "fast_dev_cli.cli.BumpUp.get_last_commit_message",
-    #     return_value=last_commit,
-    # )
-    # assert BumpUp(part="patch", commit=False, dry=True).gen() == patch_without_commit
-    # assert BumpUp(part="patch", commit=True, dry=True).gen() == patch_with_commit
-    # assert BumpUp(part="minor", commit=True, dry=True).gen() == minor_with_commit
+    mocker.patch("fast_dev_cli.cli.Project.manage_by_poetry", return_value=False)
+    version = get_current_version()
+    patch_without_commit, patch_with_commit, minor_with_commit = _bump_commands(
+        version, filename="fast_dev_cli/__init__.py", emoji=True
+    )
+    last_commit = "📝 Update release notes"
+    mocker.patch(
+        "fast_dev_cli.cli.BumpUp.get_last_commit_message",
+        return_value=last_commit,
+    )
+    assert BumpUp(part="patch", commit=False, dry=True).gen() == patch_without_commit
+    assert BumpUp(part="patch", commit=True, dry=True).gen() == patch_with_commit
+    assert BumpUp(part="minor", commit=True, dry=True).gen() == minor_with_commit
+
+
+def test_bump_with_emoji_in_poetry_project(mocker, tmp_path, monkeypatch):
     # real bump
-    # with chdir(tmp_path):
-    #     project = "foo"
-    #     subprocess.run(["poetry", "new", project])
-    #     with chdir(tmp_path / project):
-    #         subprocess.run(["git", "init"])
-    #         subprocess.run(["git", "add", "."])
-    #         subprocess.run(["git", "commit", "-m", last_commit])
-    #         monkeypatch.setenv("DONT_GIT_PUSH", "1")
-    #         command = BumpUp(part="patch", commit=True).gen()
-    #         expected = patch_with_commit.split("&&")[0].strip().replace('""', '"0.1.0"')
-    #         assert expected == command
-    #         subprocess.run(["poetry", "run", "pip", "install", "bumpversion2"])
-    #         subprocess.run(["fast", "bump", "patch", "--commit"])
-    #         out = capture_cmd_output(["git", "log"])
-    # new_commit = "⬆️  Bump version: 0.1.0 → 0.1.1"
-    # assert new_commit in out
+    last_commit = "📝 Update release notes"
+    _, patch_with_commit, __ = _bump_commands("0.1.0", emoji=True)
+    with prepare_poetry_project(tmp_path):
+        subprocess.run(["git", "init"])
+        subprocess.run(["git", "add", "."])
+        subprocess.run(["git", "config", "user.name", "sb"])
+        subprocess.run(["git", "config", "user.email", "sb@foo.com"])
+        assert BumpUp.should_add_emoji() is False
+        subprocess.run(["git", "commit", "-m", last_commit])
+        monkeypatch.setenv("DONT_GIT_PUSH", "1")
+        command = BumpUp(part="patch", commit=True).gen()
+        expected = patch_with_commit.split("&&")[0].strip().replace('""', '"0.1.0"')
+        assert expected == command
+        subprocess.run(["poetry", "run", "pip", "install", "bumpversion2"])
+        subprocess.run(["fast", "bump", "patch", "--commit"])
+        out = capture_cmd_output(["git", "log"])
+        assert BumpUp.should_add_emoji()
+        Path("a.txt").touch()
+        subprocess.run(["git", "add", "."])
+        subprocess.run(["git", "commit", "-m", "no emoji"])
+        assert BumpUp.should_add_emoji() is False
+    new_commit = "⬆️  Bump version: 0.1.0 → 0.1.1"
+    assert new_commit in out
 
 
 def test_bump_with_uv(tmp_path):

{fast_dev_cli-0.14.1 → fast_dev_cli-0.14.2}/tests/test_functions.py

@@ -1,6 +1,7 @@
 import os
 from contextlib import redirect_stdout
 from io import StringIO
+from pathlib import Path
 
 import pytest
 import typer
@@ -16,6 +17,8 @@ from fast_dev_cli.cli import (
     run_and_echo,
 )
 
+from .utils import prepare_poetry_project
+
 
 def test_utils(capsys):
     # parse files
@@ -64,12 +67,6 @@ def test_run_shell():
     # current version
     with pytest.raises(ShellCommandError):
         get_current_version(True, is_poetry=True)
-    # TODO: add [tool.poetry] to pyproject.toml
-    # stream = StringIO()
-    # write_to_stream = redirect_stdout(stream)
-    # with write_to_stream:
-    #     get_current_version(True, is_poetry=True)
-    # assert "poetry version -s" in stream.getvalue()
 
     name = "TEST_EXIT_IF_RUN_FAILED"
     value = "foo"
@@ -92,6 +89,15 @@ def test_run_shell():
         A().run()
 
 
+def test_get_version_in_poetry_project(tmp_path: Path):
+    with prepare_poetry_project(tmp_path):
+        stream = StringIO()
+        write_to_stream = redirect_stdout(stream)
+        with write_to_stream:
+            get_current_version(True, is_poetry=True)
+    assert "poetry version -s" in stream.getvalue()
+
+
 def test_ensure_bool():
     assert _ensure_bool(True) is True
     assert _ensure_bool(False) is False

{fast_dev_cli-0.14.1 → fast_dev_cli-0.14.2}/tests/test_lint.py

@@ -65,7 +65,7 @@ def test_check(mock_no_dmypy, monkeypatch, mocker):
     for cmd in CHECK_CMD.split(SEP):
         assert cmd in command
     command2 = capture_cmd_output("fast check --bandit --dry")
-    assert command2 == command + " && bandit -r fast_dev_cli"
+    assert command2 == command + " && bandit -c pyproject.toml -r ."
     monkeypatch.setenv("FASTDEVCLI_BANDIT", "1")
     command3 = capture_cmd_output("fast check --dry")
     assert command3 == command2
@@ -84,6 +84,7 @@ def test_check_bandit(tmp_path):
         src_dir = src_dir.parent / package_path.name
     shutil.rmtree(src_dir)
     with chdir(package_path):
+        assert LintCode.get_package_name() == "."
         command = capture_cmd_output("fast check --bandit --dry")
     assert "bandit -r ." in command
 
@@ -132,6 +133,15 @@ def test_lint_cmd(mock_no_dmypy):
     )
 
 
+def test_lint_html():
+    run = "pdm run "
+    lint_cmd = f"{run}python fast_dev_cli/cli.py lint"
+    command = capture_cmd_output(f"{lint_cmd} index.html --dry")
+    assert "prettier -w index.html" in command
+    command = capture_cmd_output(f"{lint_cmd} index.html flv.html --dry")
+    assert "prettier -w index.html flv.html" in command
+
+
 def test_lint_by_global_fast():
     run = "pdm run "
     fast = Path.home() / ".local" / "bin" / "fast"

{fast_dev_cli-0.14.1 → fast_dev_cli-0.14.2}/tests/utils.py

@@ -1,4 +1,5 @@
 import os
+import subprocess
 import sys
 from contextlib import contextmanager, redirect_stdout
 from io import StringIO
@@ -66,3 +67,12 @@ def temp_file(name: str, text=""):
     yield
     if path.exists():
         path.unlink()
+
+
+@contextmanager
+def prepare_poetry_project(tmp_path: Path):
+    with chdir(tmp_path):
+        project = "foo"
+        subprocess.run(["poetry", "new", project])
+        with chdir(tmp_path / project):
+            yield

fast_dev_cli-0.14.1/fast_dev_cli/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.14.1"