Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/poetry/config/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,14 +83,14 @@ def build_config_setting_normalizer(val: str) -> Mapping[str, str | Sequence[str
@dataclasses.dataclass
class PackageFilterPolicy:
policy: dataclasses.InitVar[str | list[str] | None]
packages: list[str] = dataclasses.field(init=False)
packages: frozenset[str] = dataclasses.field(init=False)

def __post_init__(self, policy: str | list[str] | None) -> None:
if not policy:
policy = []
elif isinstance(policy, str):
policy = self.normalize(policy)
self.packages = policy
self.packages = frozenset(policy)

def allows(self, package_name: str) -> bool:
if ":all:" in self.packages:
Expand Down
11 changes: 11 additions & 0 deletions tests/config/test_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
from deepdiff.diff import DeepDiff

from poetry.config.config import Config
from poetry.config.config import PackageFilterPolicy
from poetry.config.config import boolean_normalizer
from poetry.config.config import int_normalizer
from poetry.config.config import str_list_normalizer
Expand Down Expand Up @@ -50,6 +51,16 @@ def test_str_list_normalizer(value: str, expected: list[str]) -> None:
assert str_list_normalizer(value) == expected


def test_package_filter_policy_normalizes_and_matches_packages() -> None:
policy = PackageFilterPolicy("PyTest,black")

assert policy.packages == frozenset({"black", "pytest"})
assert policy.allows("Poetry")
assert not policy.allows("PyTest")
assert policy.has_exact_package("PyTest")
assert not policy.has_exact_package("Poetry")


@pytest.mark.parametrize(
("name", "value"),
[
Expand Down
Loading