Commit 9f955142 by Roberto Guzman Committed by GitHub

Refactor packages variables into dedicated fixture in lockfile_repository test (#6686)

Co-authored-by: Bartosz Sokorski <b.sokorski@gmail.com>
parent b310a9a4
......@@ -2,27 +2,34 @@ from __future__ import annotations
from copy import deepcopy
import pytest
from poetry.core.packages.package import Package
from poetry.repositories.lockfile_repository import LockfileRepository
def test_has_package() -> None:
@pytest.fixture(scope="module")
def packages() -> list[Package]:
return [
Package("a", "1.0", source_type="url", source_url="https://example.org/a.whl"),
Package("a", "1.0"),
Package(
"a", "1.0", source_type="url", source_url="https://example.org/a-1.whl"
),
]
def test_has_package(packages: list[Package]) -> None:
url_package, pypi_package, url_package_2 = packages
repo = LockfileRepository()
url_package = Package(
"a", "1.0", source_type="url", source_url="https://example.org/a.whl"
)
assert not repo.has_package(url_package)
repo.add_package(url_package)
pypi_package = Package("a", "1.0")
assert not repo.has_package(pypi_package)
repo.add_package(pypi_package)
url_package_2 = Package(
"a", "1.0", source_type="url", source_url="https://example.org/a-1.whl"
)
assert not repo.has_package(url_package_2)
repo.add_package(url_package_2)
......@@ -32,14 +39,8 @@ def test_has_package() -> None:
assert repo.has_package(deepcopy(url_package_2))
def test_remove_package() -> None:
url_package = Package(
"a", "1.0", source_type="url", source_url="https://example.org/a.whl"
)
pypi_package = Package("a", "1.0")
url_package_2 = Package(
"a", "1.0", source_type="url", source_url="https://example.org/a-1.whl"
)
def test_remove_package(packages: list[Package]) -> None:
url_package, pypi_package, url_package_2 = packages
repo = LockfileRepository()
repo.add_package(url_package)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment