19636484222

Committed 24 Nov 2025 01:45PM UTC coverage: 78.831% (-1.5%) from 80.283%

Build # 19636484222

Build Type

Pull #22917

github

Committed by

web-flow

Commit Message

Merge d61443516 into 612e19cd8

Pull Request Pull Request #22917: Mostly revert "Temporarily disable macos cache"

Run Details

74398 of 94376 relevant lines covered (78.83%)

2.9 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

71.43

/src/python/pants/engine/internals/target_adaptor.py

# Copyright 2020 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

from __future__ import annotations

import dataclasses
from dataclasses import dataclass
from typing import Any, final

from pants.build_graph.address import Address
from pants.engine.engine_aware import EngineAwareParameter
from pants.util.frozendict import FrozenDict
from pants.util.ordered_set import FrozenOrderedSet
from pants.vcs.hunk import TextBlock


@dataclass(frozen=True)
class SourceBlock:
    """Block of lines in a file.

    Lines are 1 indexed, `start` is inclusive, `end` is exclusive.

    SourceBlock is used to describe a set of source lines that are owned by a Target,
    thus it can't be empty, i.e. `start` must be less than `end`.
    """

    start: int
    end: int

    def __init__(self, start: int, end: int):
        object.__setattr__(self, "start", start)
        object.__setattr__(self, "end", end)

        self.__post_init__()

    def __post_init__(self):
        if self.start >= self.end:
            raise ValueError(f"{self.start=} must be less than {self.end=}")

    def __len__(self) -> int:
        return self.end - self.start

    def is_touched_by(self, o: TextBlock) -> bool:
        """Check if the TextBlock touches the SourceBlock.

        The function behaves similarly to range intersection check, but some edge cases are
        different. See test cases for details.
        """

        if o.count == 0:
            start = o.start + 1
            end = start
        else:
            start = o.start
            end = o.end

        if self.end < start:
            return False
        if end < self.start:
            return False
        return True

    @classmethod
    def from_text_block(cls, text_block: TextBlock) -> SourceBlock:
        """Convert (start, count) range to (start, end) range.

        Useful for unified diff conversion, see
        https://www.gnu.org/software/diffutils/manual/html_node/Detailed-Unified.html
        """
        return cls(start=text_block.start, end=text_block.start + text_block.count)


class SourceBlocks(FrozenOrderedSet[SourceBlock]):
    pass


@dataclass(frozen=True)
class TargetAdaptorRequest(EngineAwareParameter):
    """Lookup the TargetAdaptor for an Address."""

    address: Address
    description_of_origin: str = dataclasses.field(hash=False, compare=False)

    def debug_hint(self) -> str:
        return self.address.spec


@final
class TargetAdaptor:
    """A light-weight object to store target information before being converted into the Target
    API."""

    __slots__ = ("type_alias", "name", "kwargs", "description_of_origin", "origin_sources_blocks")

    def __init__(
        self,
        type_alias: str,
        name: str | None,
        __description_of_origin__: str,
        __origin_sources_blocks__: FrozenDict[str, SourceBlocks] = FrozenDict(),
        **kwargs: Any,
    ) -> None:
        self.type_alias = type_alias
        self.name = name
        try:
            self.kwargs = FrozenDict.deep_freeze(kwargs)
        except TypeError as e:
            # FrozenDict's ctor eagerly computes its hash. If some kwarg is unhashable it will
            # raise a TypeError, which we enhance with the description of origin.
            raise TypeError(f"In {__description_of_origin__}: {e}")
        self.description_of_origin = __description_of_origin__
        self.origin_sources_blocks = __origin_sources_blocks__

    def with_new_kwargs(self, **kwargs) -> TargetAdaptor:
        return TargetAdaptor(
            type_alias=self.type_alias,
            name=self.name,
            __description_of_origin__=self.description_of_origin,
            __origin_sources_blocks__=self.origin_sources_blocks,
            **kwargs,
        )

    def __repr__(self) -> str:
        maybe_blocks = f", {self.origin_sources_blocks}" if self.origin_sources_blocks else ""
        return f"TargetAdaptor(type_alias={self.type_alias}, name={self.name}, origin={self.description_of_origin}{maybe_blocks})"

    def __eq__(self, other: Any | TargetAdaptor) -> bool:
        if not isinstance(other, TargetAdaptor):
            return NotImplemented
        return (
            self.type_alias == other.type_alias
            and self.name == other.name
            and self.kwargs == other.kwargs
        )

    def __hash__(self) -> int:
        return hash((self.type_alias, self.name, self.kwargs))

    @property
    def name_explicitly_set(self) -> bool:
        return self.name is not None

1	# Copyright 2020 Pants project contributors (see CONTRIBUTORS.md).
2	# Licensed under the Apache License, Version 2.0 (see LICENSE).
3
4	from __future__ import annotations	10✔
5
6	import dataclasses	10✔
7	from dataclasses import dataclass	10✔
8	from typing import Any, final	10✔
9
10	from pants.build_graph.address import Address	10✔
11	from pants.engine.engine_aware import EngineAwareParameter	10✔
12	from pants.util.frozendict import FrozenDict	10✔
13	from pants.util.ordered_set import FrozenOrderedSet	10✔
14	from pants.vcs.hunk import TextBlock	10✔
15
16
17	@dataclass(frozen=True)	10✔
18	class SourceBlock:	10✔
19	"""Block of lines in a file.
20
21	Lines are 1 indexed, `start` is inclusive, `end` is exclusive.
22
23	SourceBlock is used to describe a set of source lines that are owned by a Target,
24	thus it can't be empty, i.e. `start` must be less than `end`.
25	"""
26
27	start: int	10✔
28	end: int	10✔
29
30	def __init__(self, start: int, end: int):	10✔
31	object.__setattr__(self, "start", start)	1✔
32	object.__setattr__(self, "end", end)	1✔
33
34	self.__post_init__()	1✔
35
36	def __post_init__(self):	10✔
37	if self.start >= self.end:	1✔
38	raise ValueError(f"{self.start=} must be less than {self.end=}")	×
39
40	def __len__(self) -> int:	10✔
41	return self.end - self.start	×
42
43	def is_touched_by(self, o: TextBlock) -> bool:	10✔
44	"""Check if the TextBlock touches the SourceBlock.
45
46	The function behaves similarly to range intersection check, but some edge cases are
47	different. See test cases for details.
48	"""
49
50	if o.count == 0:	×
51	start = o.start + 1	×
52	end = start	×
53	else:
54	start = o.start	×
55	end = o.end	×
56
57	if self.end < start:	×
58	return False	×
59	if end < self.start:	×
60	return False	×
61	return True	×
62
63	@classmethod	10✔
64	def from_text_block(cls, text_block: TextBlock) -> SourceBlock:	10✔
65	"""Convert (start, count) range to (start, end) range.
66
67	Useful for unified diff conversion, see
68	https://www.gnu.org/software/diffutils/manual/html_node/Detailed-Unified.html
69	"""
70	return cls(start=text_block.start, end=text_block.start + text_block.count)	×
71
72
73	class SourceBlocks(FrozenOrderedSet[SourceBlock]):	10✔
74	pass	10✔
75
76
77	@dataclass(frozen=True)	10✔
78	class TargetAdaptorRequest(EngineAwareParameter):	10✔
79	"""Lookup the TargetAdaptor for an Address."""
80
81	address: Address	10✔
82	description_of_origin: str = dataclasses.field(hash=False, compare=False)	10✔
83
84	def debug_hint(self) -> str:	10✔
85	return self.address.spec	×
86
87
88	@final	10✔
89	class TargetAdaptor:	10✔
90	"""A light-weight object to store target information before being converted into the Target
91	API."""
92
93	__slots__ = ("type_alias", "name", "kwargs", "description_of_origin", "origin_sources_blocks")	10✔
94
95	def __init__(	10✔
96	self,
97	type_alias: str,
98	name: str \| None,
99	__description_of_origin__: str,
100	__origin_sources_blocks__: FrozenDict[str, SourceBlocks] = FrozenDict(),
101	**kwargs: Any,
102	) -> None:
103	self.type_alias = type_alias	3✔
104	self.name = name	3✔
105	try:	3✔
106	self.kwargs = FrozenDict.deep_freeze(kwargs)	3✔
107	except TypeError as e:	×
108	# FrozenDict's ctor eagerly computes its hash. If some kwarg is unhashable it will
109	# raise a TypeError, which we enhance with the description of origin.
110	raise TypeError(f"In {__description_of_origin__}: {e}")	×
111	self.description_of_origin = __description_of_origin__	3✔
112	self.origin_sources_blocks = __origin_sources_blocks__	3✔
113
114	def with_new_kwargs(self, **kwargs) -> TargetAdaptor:	10✔
115	return TargetAdaptor(	1✔
116	type_alias=self.type_alias,
117	name=self.name,
118	__description_of_origin__=self.description_of_origin,
119	__origin_sources_blocks__=self.origin_sources_blocks,
120	**kwargs,
121	)
122
123	def __repr__(self) -> str:	10✔
124	maybe_blocks = f", {self.origin_sources_blocks}" if self.origin_sources_blocks else ""	×
125	return f"TargetAdaptor(type_alias={self.type_alias}, name={self.name}, origin={self.description_of_origin}{maybe_blocks})"	×
126
127	def __eq__(self, other: Any \| TargetAdaptor) -> bool:	10✔
128	if not isinstance(other, TargetAdaptor):	1✔
129	return NotImplemented	×
130	return (	1✔
131	self.type_alias == other.type_alias
132	and self.name == other.name
133	and self.kwargs == other.kwargs
134	)
135
136	def __hash__(self) -> int:	10✔
137	return hash((self.type_alias, self.name, self.kwargs))	1✔
138
139	@property	10✔
140	def name_explicitly_set(self) -> bool:	10✔
141	return self.name is not None	×

pantsbuild / pants / 19636484222

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous