chore(tests): added and adjusted tests for project command

api/outdated/commands.py

@@ -6,7 +6,7 @@
 
 from outdated.outdated.models import Project
 
-if TYPE_CHECKING: # pragma: no cover
+if TYPE_CHECKING:  # pragma: no cover
     from django.core.management.base import CommandParser
 
 
@@ -44,5 +44,5 @@ def handle(self, *args, **options):
         for project in projects:
             self._handle(project)
 
-    def _handle(self, project: Project): # pragma: no cover
+    def _handle(self, project: Project):  # pragma: no cover
         raise NotImplementedError()

api/outdated/outdated/tests/test_syncproject.py

@@ -1,26 +1,15 @@
+from unittest.mock import ANY
+
 import pytest
 from django.core.management import call_command
 
+from outdated.outdated.synchroniser import Synchroniser
+
 
 @pytest.mark.vcr()
 @pytest.mark.django_db(transaction=True)
-def test_syncproject(project_factory):
-    call_command("syncproject", "foo")
-
+def test_syncproject(project_factory, mocker):
     project = project_factory.create(repo="https://github.com/projectcaluma/caluma")
-
+    sync_init_mocker = mocker.spy(Synchroniser, "__init__")
     call_command("syncproject", project.name)
-    assert project.versioned_dependencies.count() > 0
-
-
-@pytest.mark.vcr()
-@pytest.mark.django_db(transaction=True)
-def test_syncprojects(project_factory):
-    projects = [
-        project_factory(repo=f"https://github.com/adfinis/{project}")
-        for project in ["outdated", "mysagw"]
-    ]
-
-    call_command("syncprojects")
-    for project in projects:
-        assert project.versioned_dependencies.count() > 0
+    sync_init_mocker.assert_called_once_with(ANY, project)

api/outdated/tests/test_commands.py

@@ -0,0 +1,37 @@
+from unittest.mock import call
+
+import pytest
+
+from outdated.commands import ProjectCommand
+
+
+@pytest.mark.parametrize("all", [True, False])
+def test_command_handle(transactional_db, project_factory, all, mocker):
+    projects = project_factory.create_batch(5)
+    argv = ["", "project-command-test"]
+    handle_mocker = mocker.patch.object(ProjectCommand, "_handle")
+    ProjectCommand().run_from_argv(
+        [*argv, *(["--all"] if all else [project.name for project in projects])]
+    )
+    handle_mocker.assert_has_calls([call(project) for project in projects], True)
+
+
+@pytest.mark.parametrize(
+    "existing_projects",
+    [
+        [],
+        ["foo"],
+        ["Foo"],
+        ["foo", "foobar"],
+    ],
+)
+@pytest.mark.parametrize("nonexistant_projects", [["bar"], ["bar", "baz"]])
+def test_project_command(
+    transactional_db, project_factory, capsys, nonexistant_projects, existing_projects
+):
+    argv = ["", "project-command-test"]
+    for project in existing_projects:
+        project_factory(name=project)
+    ProjectCommand().run_from_argv([*argv, *nonexistant_projects, *existing_projects])
+    _, stderr = capsys.readouterr()
+    assert stderr == f"Projects with names {nonexistant_projects} do not exist\n"

api/pyproject.toml

@@ -38,6 +38,7 @@ pytest-cov = "^4.0.0"
 pytest-vcr = "^1.0.2"
 pdbpp = "^0.10.3"
 requests-mock = "^1.10.0"
+pytest-mock = "^3.11.1"
 
 
 [tool.isort]