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

api/outdated/commands.py

@@ -2,7 +2,6 @@
 
 from outdated.outdated.models import Project
 
-
 class ProjectCommand(BaseCommand):
  def add_arguments(self, parser: CommandParser):
  projects = parser.add_mutually_exclusive_group(required=True)
@@ -37,5 +36,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

@@ -35,6 +35,7 @@ pytest = "7.3.1"
 isort = "5.11.4"
 pytest-factoryboy = "2.5.1"
 pytest-cov = "^4.0.0"
+pytest-mock = "^3.11.1"
 pytest-vcr = "^1.0.2"
 pdbpp = "^0.10.3"
 requests-mock = "^1.10.0"